Hi Yunlei,
Could you send a v2 for this?
Thanks,
On Tue, Dec 29, 2015 at 08:58:24AM +0800, He YunLei wrote:
> On 2015/12/29 7:10, Jaegeuk Kim wrote:
> >Hi Yunlei,
> >
> >On Sat, Dec 26, 2015 at 10:55:45AM +0800, He YunLei wrote:
> >>On 2015/12/26 10:46, Yunlei He wrote:
> >>>This patch add a mismatch check between segment and section in
> >>>sanity_check_raw_super.
> >>>
> >>>Signed-off-by: Yunlei He <heyun...@huawei.com>
> >>>---
> >>> fs/f2fs/super.c | 11 +++++++++++
> >>> 1 file changed, 11 insertions(+)
> >>>
> >>>diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> >>>index c3070c1..b51a690 100644
> >>>--- a/fs/f2fs/super.c
> >>>+++ b/fs/f2fs/super.c
> >>>@@ -1036,6 +1036,17 @@ static int sanity_check_raw_super(struct
> >>>super_block *sb,
> >>> return 1;
> >>> }
> >>>
> >>>+ /* check seg # corresponding to sec # */
> >>>+ if (le32_to_cpu(raw_super->segment_count)
> >
> > raw_super->segment_count_main ?
> >
> >Thanks,
>
> yes, you are right, mistake also in f2fs_msg.
>
> >
> >>>+ * le32_to_cpu(raw_super->segs_per_sec)
> >>
> >>I am sorry that here is :
> >> / le32_to_cpu(raw_super->segs_per_sec)
> >>>+ != le32_to_cpu(raw_super->section_count)) {
> >>>+ f2fs_msg(sb, KERN_INFO,
> >>>+ "Mismatch seg num (%u) and sec num (%u) \n",
> >>>+ le32_to_cpu(raw_super->segment_count),
> here is also need change:
> le32_to_cpu(raw_super->segment_count_main),
> Thanks,
> >>>+ le32_to_cpu(raw_super->section_count));
> >>>+ return 1;
> >>>+ }
> >>>+
> >>> /* Currently, support 512/1024/2048/4096 bytes sector size */
> >>> if (le32_to_cpu(raw_super->log_sectorsize) >
> >>> F2FS_MAX_LOG_SECTOR_SIZE ||
> >>>
> >
> >.
> >
------------------------------------------------------------------------------
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
