f2fs: stop allocating pinned sections if EAGAIN happens

Jaegeuk Kim · Jaegeuk Kim · commit 2e42b7f817ac · 2022-05-09T09:31:27.000-07:00
EAGAIN doesn't guarantee to have a free section. Let's report it.

Reviewed-by: Chao Yu &lt;chao@kernel.org&gt;
Signed-off-by: Jaegeuk Kim &lt;jaegeuk@kernel.org&gt;
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
@@ -1685,7 +1685,7 @@ static int expand_inode_data(struct inode *inode, loff_t offset,
 			GET_SEC_FROM_SEG(sbi, overprovision_segments(sbi)))) {
 			f2fs_down_write(&sbi->gc_lock);
 			err = f2fs_gc(sbi, true, false, false, NULL_SEGNO);
-			if (err && err != -ENODATA && err != -EAGAIN)
+			if (err && err != -ENODATA)
 				goto out_err;
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -1685,7 +1685,7 @@ static int expand_inode_data(struct inode *inode, loff_t offset,`
`1685`	`1685`	`GET_SEC_FROM_SEG(sbi, overprovision_segments(sbi)))) {`
`1686`	`1686`	`f2fs_down_write(&sbi->gc_lock);`
`1687`	`1687`	`err = f2fs_gc(sbi, true, false, false, NULL_SEGNO);`
`1688`		`- if (err && err != -ENODATA && err != -EAGAIN)`
	`1688`	`+ if (err && err != -ENODATA)`
`1689`	`1689`	`goto out_err;`
`1690`	`1690`	`}`
`1691`	`1691`