From d4468fa6ebdb59cb535e27cc0091663eb8f1874c Mon Sep 17 00:00:00 2001 From: Hanqing Wu Date: Tue, 6 Sep 2022 19:38:39 +0800 Subject: [PATCH] curve-fuse: fix miss use GetInodeAttr[Locked] Signed-off-by: Hanqing Wu --- curvefs/src/client/fuse_client.cpp | 6 +++--- curvefs/src/client/inode_cache_manager.cpp | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/curvefs/src/client/fuse_client.cpp b/curvefs/src/client/fuse_client.cpp index 796d2ce2b..1ac116433 100644 --- a/curvefs/src/client/fuse_client.cpp +++ b/curvefs/src/client/fuse_client.cpp @@ -680,7 +680,7 @@ CURVEFS_ERROR FuseClient::MakeNode(fuse_req_t req, fuse_ino_t parent, } InodeAttr attr; - inodeWrapper->GetInodeAttrLocked(&attr); + inodeWrapper->GetInodeAttr(&attr); GetDentryParamFromInodeAttr(option_, attr, e); return ret; } @@ -1276,7 +1276,7 @@ CURVEFS_ERROR FuseClient::FuseOpSymlink(fuse_req_t req, const char *link, } InodeAttr attr; - inodeWrapper->GetInodeAttrLocked(&attr); + inodeWrapper->GetInodeAttr(&attr); GetDentryParamFromInodeAttr(option_, attr, e); return ret; } @@ -1347,7 +1347,7 @@ CURVEFS_ERROR FuseClient::FuseOpLink(fuse_req_t req, fuse_ino_t ino, } InodeAttr attr; - inodeWrapper->GetInodeAttrLocked(&attr); + inodeWrapper->GetInodeAttr(&attr); GetDentryParamFromInodeAttr(option_, attr, e); return ret; } diff --git a/curvefs/src/client/inode_cache_manager.cpp b/curvefs/src/client/inode_cache_manager.cpp index 0939aef89..c92cfd3a5 100644 --- a/curvefs/src/client/inode_cache_manager.cpp +++ b/curvefs/src/client/inode_cache_manager.cpp @@ -176,7 +176,7 @@ CURVEFS_ERROR InodeCacheManagerImpl::GetInodeAttr(uint64_t inodeId, std::shared_ptr inodeWrapper; bool ok = iCache_->Get(inodeId, &inodeWrapper); if (ok) { - inodeWrapper->GetInodeAttrLocked(out); + inodeWrapper->GetInodeAttr(out); return CURVEFS_ERROR::OK; } @@ -214,7 +214,7 @@ CURVEFS_ERROR InodeCacheManagerImpl::BatchGetInodeAttr( bool ok = iCache_->Get(*iter, &inodeWrapper); if (ok) { InodeAttr tmpAttr; - inodeWrapper->GetInodeAttrLocked(&tmpAttr); + inodeWrapper->GetInodeAttr(&tmpAttr); attr->emplace_back(tmpAttr); iter = inodeIds->erase(iter); } else {