>From Gui Hecheng <guim...@126.com>:

Gui Hecheng has uploaded this change for review. ( 
https://review.gerrithub.io/353310


Change subject: FSAL_RGW: fix rename causing mdcache to have conflict dentries
......................................................................

FSAL_RGW: fix rename causing mdcache to have conflict dentries

FSAL_RGW use {bucket,object} pair as a key for mdcache dentries,
so when we rename a file in FSAL_RGW, we change the actual key
inevitably.
But mdcache tries to do a mdcache_key_dup() from
old dentry to new dentry on rename, so we have a new dentry with
old {bucket,object} pair as key.
Upon next lookup for the new file, we should get EEXIST from mdcache.

To fix it, let FSAL_RGW's rename return a special code to signal
mdcache to drop the old dentry only without adding a new one.

Change-Id: I2c25748424f89cd4781805b193a40032c654cefd
Signed-off-by: Gui Hecheng <guihech...@cmss.chinamobile.com>
---
M src/FSAL/FSAL_RGW/handle.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c
2 files changed, 27 insertions(+), 3 deletions(-)



  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/10/353310/1
-- 
To view, visit https://review.gerrithub.io/353310
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2c25748424f89cd4781805b193a40032c654cefd
Gerrit-Change-Number: 353310
Gerrit-PatchSet: 1
Gerrit-Owner: Gui Hecheng <guim...@126.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to