[f2fs-dev] [PATCH] Check all the data segments against all node ones.

[f2fs-dev] [PATCH] Check all the data segments against all node ones.

[Surbhi Palande]

As a part of the sanity checking while mounting, distinct segment number
assignment to data and node segments is verified. Fixing a small bug in
this verification between node and data segments. We need to check all
the data segments with all the node segments.

Signed-off-by: Surbhi Palande <csurbhi@gmail.com>
---
 fs/f2fs/super.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index 6b959bb..1ec4d95 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -2684,11 +2684,11 @@ int f2fs_sanity_check_ckpt(struct f2fs_sb_info *sbi)
 		}
 	}
 	for (i = 0; i < NR_CURSEG_NODE_TYPE; i++) {
-		for (j = i; j < NR_CURSEG_DATA_TYPE; j++) {
+		for (j = 0; j < NR_CURSEG_DATA_TYPE; j++) {
 			if (le32_to_cpu(ckpt->cur_node_segno[i]) ==
 				le32_to_cpu(ckpt->cur_data_segno[j])) {
 				f2fs_msg(sbi->sb, KERN_ERR,
-					"Data segment (%u) and Data segment (%u)"
+					"Node segment (%u) and Data segment (%u)"
 					" has the same segno: %u", i, j,
 					le32_to_cpu(ckpt->cur_node_segno[i]));
 				return 1;
-- 
2.7.4



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

Re: [f2fs-dev] [PATCH] Check all the data segments against all node ones.

[Chao Yu]

On 2019-8-24 6:40, Surbhi Palande wrote:
> As a part of the sanity checking while mounting, distinct segment number
> assignment to data and node segments is verified. Fixing a small bug in
> this verification between node and data segments. We need to check all
> the data segments with all the node segments.
> 

Fixes: 042be0f849e5f ("f2fs: fix to do sanity check with current segment number")

> Signed-off-by: Surbhi Palande <csurbhi@gmail.com>

Reviewed-by: Chao Yu <yuchao0@huawei.com>

Thanks,


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

Re: [f2fs-dev] [PATCH] Check all the data segments against all node ones.

[Chao Yu]

Hi Surbhi,

I guess this patch is okay, please wait Jaegeuk to merge it.

On 2019/8/24 17:48, Chao Yu wrote:
> On 2019-8-24 6:40, Surbhi Palande wrote:
>> As a part of the sanity checking while mounting, distinct segment number
>> assignment to data and node segments is verified. Fixing a small bug in
>> this verification between node and data segments. We need to check all
>> the data segments with all the node segments.
>>
> 
> Fixes: 042be0f849e5f ("f2fs: fix to do sanity check with current segment number")

I meant it will be better to add this line to indicate which patch introduced
this bug, it can help LTS kernel maintainer to recognize and backport the
necessary fixing patches.

BTW, it needs to update commit tile as format of "f2fs: check all...."

Anyway, thanks for the patch!

Thanks,

> 
>> Signed-off-by: Surbhi Palande <csurbhi@gmail.com>
> 
> Reviewed-by: Chao Yu <yuchao0@huawei.com>
> 
> Thanks,
> 
> 
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
> .
> 


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

Re: [f2fs-dev] [PATCH] Check all the data segments against all node ones.

[Jaegeuk Kim]

Hi Surbhi,

Applied as "f2fs: check all the data segments against all node ones".

Thanks,

On 08/23, Surbhi Palande wrote:
> As a part of the sanity checking while mounting, distinct segment number
> assignment to data and node segments is verified. Fixing a small bug in
> this verification between node and data segments. We need to check all
> the data segments with all the node segments.
> 
> Signed-off-by: Surbhi Palande <csurbhi@gmail.com>
> ---
>  fs/f2fs/super.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> index 6b959bb..1ec4d95 100644
> --- a/fs/f2fs/super.c
> +++ b/fs/f2fs/super.c
> @@ -2684,11 +2684,11 @@ int f2fs_sanity_check_ckpt(struct f2fs_sb_info *sbi)
>  		}
>  	}
>  	for (i = 0; i < NR_CURSEG_NODE_TYPE; i++) {
> -		for (j = i; j < NR_CURSEG_DATA_TYPE; j++) {
> +		for (j = 0; j < NR_CURSEG_DATA_TYPE; j++) {
>  			if (le32_to_cpu(ckpt->cur_node_segno[i]) ==
>  				le32_to_cpu(ckpt->cur_data_segno[j])) {
>  				f2fs_msg(sbi->sb, KERN_ERR,
> -					"Data segment (%u) and Data segment (%u)"
> +					"Node segment (%u) and Data segment (%u)"
>  					" has the same segno: %u", i, j,
>  					le32_to_cpu(ckpt->cur_node_segno[i]));
>  				return 1;
> -- 
> 2.7.4


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