On 2021/12/30 8:59, Antoine Viallon via Linux-f2fs-devel wrote:
Some variable naming mistaskes were made in the last patch,
here is an updated version.

---
  fsck/fsck.c | 9 +++++++++
  fsck/fsck.h | 1 +
  2 files changed, 10 insertions(+)

diff --git a/fsck/fsck.c b/fsck/fsck.c
index ecd87af..921db0f 100644
--- a/fsck/fsck.c
+++ b/fsck/fsck.c
@@ -546,6 +546,15 @@ int fsck_chk_node_blk(struct f2fs_sb_info *sbi, struct 
f2fs_inode *inode,
         node_blk = (struct f2fs_node *)calloc(BLOCK_SZ, 1);
         ASSERT(node_blk != NULL);
+ /* Progress report */
+       sbi->fsck->chk.checked_node_cnt++;
+
+       if (sbi->fsck->chk.checked_node_cnt % 1000 == 0)
+               printf("[FSCK] Check node %u / %u (%.2f%%)\n",
+                          sbi->fsck->chk.checked_node_cnt,
+                          sbi->total_valid_node_count,
+                          100 * (float)(sbi->fsck->chk.checked_node_cnt) / 
sbi->total_valid_node_count);

How about showing this message in debug mode? may be under level one?

Thanks,

+
         if (sanity_check_nid(sbi, nid, node_blk, ftype, ntype, &ni))
                 goto err;
diff --git a/fsck/fsck.h b/fsck/fsck.h
index 11846e1..535d6d5 100644
--- a/fsck/fsck.h
+++ b/fsck/fsck.h
@@ -93,6 +93,7 @@ struct f2fs_fsck {
         struct chk_result {
                 u64 valid_blk_cnt;
                 u32 valid_nat_entry_cnt;
+               u32 checked_node_cnt;
                 u32 valid_node_cnt;
                 u32 valid_inode_cnt;
                 u32 multi_hard_link_files;


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to