diff options
| author | Poornima G <pgurusid@redhat.com> | 2018-06-27 14:59:40 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-07-11 04:11:59 +0000 |
| commit | 69f77d28c3ecacba77fbae2f789b5110641347f3 (patch) | |
| tree | 746f1470476a981b6becaedb6c7c1e681f457f41 /xlators/cluster/dht | |
| parent | af6c6429f9743f287baaad68f8e3d56ed7390d1b (diff) | |
md-cache: Do not invalidate cache post set/remove xattr
Since setxattr and removexattr fops cbk do not carry poststat,
the stat cache was being invalidated in setxatr/remoxattr cbk.
Hence the further lookup wouldn't be served from cache.
To prevent this invalidation, md-cache is modified to get
the poststat in set/removexattr_cbk in dict.
Co-authored with Xavi Hernandez.
Change-Id: I6b946be2d20b807e2578825743c25ba5927a60b4
fixes: bz#1586018
Signed-off-by: Xavi Hernandez <xhernandez@redhat.com>
Signed-off-by: Poornima G <pgurusid@redhat.com>
Diffstat (limited to 'xlators/cluster/dht')
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index 900789ca8aa..fa73c33c5ab 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -5558,9 +5558,9 @@ dht_file_setxattr_cbk (call_frame_t *frame, void *cookie, xlator_t *this, out: if (local->fop == GF_FOP_SETXATTR) { - DHT_STACK_UNWIND (setxattr, frame, op_ret, op_errno, NULL); + DHT_STACK_UNWIND (setxattr, frame, op_ret, op_errno, xdata); } else { - DHT_STACK_UNWIND (fsetxattr, frame, op_ret, op_errno, NULL); + DHT_STACK_UNWIND (fsetxattr, frame, op_ret, op_errno, xdata); } return 0; |