Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1468591

[PATCH v1 net-next 1/1] fs: gfs2: Check if the return value of rhashtable_walk_start is -EAGAIN

From fgao@ikuai8.com
Newsgroups linux.kernel
Subject [PATCH v1 net-next 1/1] fs: gfs2: Check if the return value of rhashtable_walk_start is -EAGAIN
Date 2016-08-23 16:30 +0200
Message-ID <s9kSd-5aF-7@gated-at.bofh.it> (permalink)
Organization linux.* mail to news gateway

Show all headers | View raw


From: Gao Feng <fgao@ikuai8.com>

When rhashtable_walk_start returns -EAGAIN, it means the resize event
happened but the iterator still could be used. So it should not be
treated as an error in gfs2_glock_seq_start.

Signed-off-by: Gao Feng <fgao@ikuai8.com>
---
 fs/gfs2/glock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c
index 3a90b2b..732721b 100644
--- a/fs/gfs2/glock.c
+++ b/fs/gfs2/glock.c
@@ -1818,7 +1818,7 @@ static void *gfs2_glock_seq_start(struct seq_file *seq, loff_t *pos)
 		n = (*pos - gi->last_pos);
 
 	ret = rhashtable_walk_start(&gi->hti);
-	if (ret)
+	if (ret && ret != -EAGAIN)
 		return NULL;
 
 	do {
-- 
1.9.1

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

[PATCH v1 net-next 1/1] fs: gfs2: Check if the return value of rhashtable_walk_start is -EAGAIN fgao@ikuai8.com - 2016-08-23 16:30 +0200

csiph-web