diff options
author | Jiffin Tony Thottan <jthottan@redhat.com> | 2018-04-27 15:22:50 +0530 |
---|---|---|
committer | Poornima G <pgurusid@redhat.com> | 2018-05-04 17:13:13 +0000 |
commit | 15866ac9773e89cd9e017e7d3bf8aa01a87edfd8 (patch) | |
tree | 9c9278c8eb7f0cfb6fed3a78f334892d248c7435 | |
parent | b04066721bf4a240f61b83bd87bbb27437c5fe4f (diff) |
feature/leases : fixing bugs found while testing glfs_test.t
Change-Id: Iee8f431601ecda184108a079f665e05902b0f78b
updates: #350
Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com>
-rw-r--r-- | xlators/features/leases/src/leases-internal.c | 14 | ||||
-rw-r--r-- | xlators/features/leases/src/leases.c | 54 | ||||
-rw-r--r-- | xlators/protocol/server/src/server-rpc-fops_v2.c | 2 |
3 files changed, 56 insertions, 14 deletions
diff --git a/xlators/features/leases/src/leases-internal.c b/xlators/features/leases/src/leases-internal.c index 1f79e554489..475576eac79 100644 --- a/xlators/features/leases/src/leases-internal.c +++ b/xlators/features/leases/src/leases-internal.c @@ -263,7 +263,7 @@ out: static inline gf_boolean_t __is_same_lease_id (const char *k1, const char *k2) { - if (memcmp(k1, k2, LEASE_ID_SIZE) == 0) + if (memcmp(k1, k2, strlen(k1)) == 0) return _gf_true; return _gf_false; @@ -642,7 +642,7 @@ __remove_lease (xlator_t *this, inode_t *inode, lease_inode_ctx_t *lease_ctx, leaseid_utoa (lease->lease_id)); lease_entry = __get_lease_id_entry (lease_ctx, lease->lease_id); - if (!lease_entry) { + if (!lease_entry || !(lease_entry->lease_type & lease->lease_type)) { gf_msg (this->name, GF_LOG_INFO, 0, LEASE_MSG_INVAL_UNLK_LEASE, "Got unlock lease request from client:%s, but has no " "corresponding lock", client_uid); @@ -666,7 +666,7 @@ __remove_lease (xlator_t *this, inode_t *inode, lease_inode_ctx_t *lease_ctx, if (lease_entry->lease_cnt == 0) { if (__is_clnt_lease_none (client_uid, lease_ctx)) { - gf_msg_debug (this->name, 0, "Client(%s) has no leases" + gf_msg_trace (this->name, 0, "Client(%s) has no leases" " on gfid (%s), hence removing the inode" " from the client cleanup list", client_uid, uuid_utoa (inode->gfid)); @@ -725,7 +725,8 @@ __is_lease_grantable (xlator_t *this, lease_inode_ctx_t *lease_ctx, * the same lease id is not checked for conflict, as it is * lease id based lease. */ - if (!__is_same_lease_id (fd_ctx->lease_id, lease->lease_id)) { + if (fd_ctx->client_uid != NULL + && !__is_same_lease_id (fd_ctx->lease_id, lease->lease_id)) { fd_count++; flags |= iter_fd->flags; } @@ -801,8 +802,7 @@ do_blocked_fops (xlator_t *this, lease_inode_ctx_t *lease_ctx) pthread_mutex_unlock (&lease_ctx->lock); gf_msg_trace (this->name, 0, "Executing the blocked stubs on gfid(%s)", - uuid_utoa (lease_ctx->inode->gfid)); - + uuid_utoa (lease_ctx->inode->gfid)); list_for_each_entry_safe (blk_fop, tmp, &wind_list, list) { list_del_init (&blk_fop->list); gf_msg_trace (this->name, 0, "Executing fop:%d", blk_fop->stub->fop); @@ -1085,7 +1085,7 @@ check_lease_conflict (call_frame_t *frame, inode_t *inode, gf_boolean_t is_blocking_fop = _gf_false; gf_boolean_t is_write_fop = _gf_false; gf_boolean_t conflicts = _gf_false; - int ret = -1; + int ret = WIND_FOP; lease_ctx = lease_ctx_get (inode, frame->this); if (!lease_ctx) { diff --git a/xlators/features/leases/src/leases.c b/xlators/features/leases/src/leases.c index ebee682a685..34143859c56 100644 --- a/xlators/features/leases/src/leases.c +++ b/xlators/features/leases/src/leases.c @@ -872,9 +872,11 @@ int leases_flush (call_frame_t *frame, xlator_t *this, fd_t *fd, dict_t *xdata) { - uint32_t fop_flags = 0; - char *lease_id = NULL; - int ret = 0; + uint32_t fop_flags = 0; + char *lease_id = NULL; + int ret = 0; + lease_fd_ctx_t *fd_ctx = NULL; + uint64_t ctx = 0; EXIT_IF_LEASES_OFF (this, out); @@ -895,6 +897,26 @@ block: return 0; out: + /* * + * currently release is not called after the close fop from the + * application. Hence lease fd ctx is resetted on here. + * This is actually not the right way, since flush can be called + * not only from the close op. + * TODO : + * - Either identify the flush is called from close call on fd from + * from the application. + * OR + * - Find why release is not called post the last close call + */ + ret = fd_ctx_get (fd, this, &ctx); + if (ret == 0) { + fd_ctx = (lease_fd_ctx_t *)(long)ctx; + if (fd_ctx->client_uid) { + GF_FREE (fd_ctx->client_uid); + fd_ctx->client_uid = NULL; + } + memset (fd_ctx->lease_id, 0, LEASE_ID_SIZE); + } STACK_WIND (frame, leases_flush_cbk, FIRST_CHILD(this), FIRST_CHILD(this)->fops->flush, fd, xdata); return 0; @@ -1065,8 +1087,30 @@ leases_forget (xlator_t *this, inode_t *inode) static int leases_release (xlator_t *this, fd_t *fd) { - /* TODO:cleanup fd_ctx */ - return 0; + int ret = -1; + uint64_t tmp = 0; + lease_fd_ctx_t *fd_ctx = NULL; + + if (fd == NULL) { + goto out; + } + + gf_log (this->name, GF_LOG_TRACE, + "Releasing all leases with fd %p", fd); + + ret = fd_ctx_del (fd, this, &tmp); + if (ret) { + gf_log (this->name, GF_LOG_DEBUG, + "Could not get fdctx"); + goto out; + } + + fd_ctx = (lease_fd_ctx_t *)(long)tmp; + if (fd_ctx) + GF_FREE (fd_ctx); +out: + return ret; + } static int diff --git a/xlators/protocol/server/src/server-rpc-fops_v2.c b/xlators/protocol/server/src/server-rpc-fops_v2.c index 1e0d9f12244..ac6fb86c8cd 100644 --- a/xlators/protocol/server/src/server-rpc-fops_v2.c +++ b/xlators/protocol/server/src/server-rpc-fops_v2.c @@ -200,11 +200,9 @@ server4_lease_cbk (call_frame_t *frame, void *cookie, xlator_t *this, uuid_utoa (state->resolve.gfid), STACK_CLIENT_NAME (frame->root), STACK_ERR_XL_NAME (frame->root)); - goto out; } server4_post_lease (&rsp, lease); -out: rsp.op_ret = op_ret; rsp.op_errno = gf_errno_to_error (op_errno); |