On 2024/12/24 19:46, zangyangyang1 via Linux-f2fs-devel wrote:
When f2fs_write_single_data_page fails, f2fs_write_cache_pages
will use the last 'submitted' value incorrectly, which will cause
'nwritten' and 'wbc->nr_to_write' calculation errors
> > Signed-off-by: zangyangyang1 <zangyangya...@xiaomi.com>
---
  fs/f2fs/data.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 94f7b084f601..083acd7f5420 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -3179,6 +3179,7 @@ static int f2fs_write_cache_pages(struct address_space 
*mapping,
                                 continue;
                         }
  #endif
+                       submitted = 0;

How about initializing "submitted" in f2fs_write_single_data_page()
to cover all cases in where we call f2fs_write_single_data_page()?

Thanks,

                         ret = f2fs_write_single_data_page(folio,
                                         &submitted, &bio, &last_block,
                                         wbc, io_type, 0, true);
--
2.43.2



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to