diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2012-05-19 14:49:21 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-05-23 06:38:24 -0700 |
| commit | 2606b87470e396e3e79269764e01f572da051e41 (patch) | |
| tree | 03fc1f3754eeb82f58b77c5ef513ae9c1c23a051 /xlators/performance/quick-read/src/quick-read-mem-types.h | |
| parent | bb8a0664ef36809d8b8e75fcb973a2089e5d08a6 (diff) | |
performance/quick-read: fix race-conditions in qr_unlink.
The list of fds on which open needs to be done as part of unlink,
was being modified at different places using different locks.
This resulted in a race-condition where open was marked as in-transit,
but fdctx was removed from the list of fds on which open was being
sent even before open was done. Because of this, open_in_transit would
be set forever (as an open was never actually sent, there would be no
open_cbk called and hence we could not reset the variable), blocking
all the future fd based fops on this fd.
Change-Id: Ie84a55bee578869a9a060a094ba28480e7643ae8
BUG: 819490
Signed-off-by: Raghavendra G <raghavendra@gluster.com>
Reviewed-on: http://review.gluster.com/3371
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'xlators/performance/quick-read/src/quick-read-mem-types.h')
| -rw-r--r-- | xlators/performance/quick-read/src/quick-read-mem-types.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/performance/quick-read/src/quick-read-mem-types.h b/xlators/performance/quick-read/src/quick-read-mem-types.h index 9a6be76b0e6..73c87c819d0 100644 --- a/xlators/performance/quick-read/src/quick-read-mem-types.h +++ b/xlators/performance/quick-read/src/quick-read-mem-types.h @@ -20,7 +20,7 @@ enum gf_qr_mem_types_ { gf_qr_mt_qr_conf_t, gf_qr_mt_qr_priority_t, gf_qr_mt_qr_private_t, - gf_qr_mt_qr_dentry_t, + gf_qr_mt_qr_unlink_ctx_t, gf_qr_mt_end }; #endif |