From 129ffcbd80adf2d88b151cbd47669c084f40f0b9 Mon Sep 17 00:00:00 2001 From: LI Qing Date: Mon, 27 Feb 2023 17:18:04 +0800 Subject: [PATCH] Do not decommit the page cahce when close file --- src/services/libs/jinux-std/src/fs/file_handle/mod.rs | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/src/services/libs/jinux-std/src/fs/file_handle/mod.rs b/src/services/libs/jinux-std/src/fs/file_handle/mod.rs index d588c3f41..e3682bc19 100644 --- a/src/services/libs/jinux-std/src/fs/file_handle/mod.rs +++ b/src/services/libs/jinux-std/src/fs/file_handle/mod.rs @@ -83,14 +83,8 @@ impl FileHandle { pub fn clean_for_close(&self) -> Result<()> { match &self.inner { - Inner::Inode(inode_handle) => { - let dentry = inode_handle.dentry(); - let ref_count = Arc::strong_count(dentry); - // The dentry is held by dentry cache and self - if ref_count == 2 { - let page_cache_size = dentry.vnode().pages().size(); - dentry.vnode().pages().decommit(0..page_cache_size)?; - } + Inner::Inode(_) => { + // Close does not guarantee that the data has been successfully saved to disk. } Inner::File(file) => file.flush()?, }