On 2015/12/29 9:46, Jaegeuk Kim wrote:
> Hi Yunlei,
>
> Could you send a v2 for this?
>
> Thanks,
Hi Kim,
OK! I have sent it.
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
