[Virtio-fs] [PATCH 5/6] fuse: Get rid of -EAGAIN logic in dmap free worker thread

Vivek Goyal vgoyal at redhat.com
Tue Jul 16 21:00:50 UTC 2019


Now we are not doing inode trylock anymore. That means we will not get
-EAGAIN at high frequency. So get rid of this logic. Its not needed anymore.

Signed-off-by: Vivek Goyal <vgoyal at redhat.com>
---
 fs/fuse/file.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/fs/fuse/file.c b/fs/fuse/file.c
index bf9903a858db..42b250d23888 100644
--- a/fs/fuse/file.c
+++ b/fs/fuse/file.c
@@ -4235,17 +4235,11 @@ static int try_to_free_dmap_chunks(struct fuse_conn *fc,
 
 		ret = lookup_and_reclaim_dmap(fc, inode, dmap_start);
 		iput(inode);
-		if (ret && ret != -EAGAIN) {
+		if (ret) {
 			printk("%s(window_offset=0x%llx) failed. err=%d\n",
 				__func__, window_offset, ret);
 			return ret;
 		}
-
-		/* Could not get inode lock. Try next element */
-		if (ret == -EAGAIN) {
-			cond_resched();
-			continue;
-		}
 		nr_freed++;
 	}
 	return 0;
-- 
2.20.1




More information about the Virtio-fs mailing list