youngkingdom/s3fs-fuse
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Changed to serialize CacheFileStat after flushing a file

Browse Source
This commit is contained in:
Takeshi Nakatani
2025-08-26 03:08:47 +00:00
committed by Andrew Gaul
parent ecdcb4a836
commit bae0facba3
1 changed files with 18 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/fdcache_entity.cpp
View File

@ -1419,6 +1419,24 @@ int FdEntity::RowFlushHasLock(int fd, const char* tpath, bool force_sync)
FdManager::DeleteCacheFile(tpath);
}
// [NOTE]
// Normally, when client finishes editing a file and gets the file attributes,
// FUSE calls flush->relaase, and then getsattr.
// In other words, we expect that getattr will not be called between flush->release.
// However, because FUSE does not wait for the release process to be complete,
// the case of flush->getattr->release occurs.
// Therefore, we make sure to serialize the pagelist here.
//
if(0 == result && !cachepath.empty()){
ino_t cur_inode = GetInode();
if(0 != cur_inode && cur_inode == inode){
CacheFileStat cfstat(path.c_str());
if(!pagelist.Serialize(cfstat, inode)){
S3FS_PRN_WARN("failed to save cache stat file(%s).", path.c_str());
}
}
}
return result;
}