linux-kernel - Re: [PATCH 1/1] f2fs: update multi-dev metadata in resize

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 23 Sep 2019 09:55:53 +0800
From:   Chao Yu <yuchao0@...wei.com>
To:     sunqiuyang <sunqiuyang@...wei.com>, <linux-kernel@...r.kernel.org>,
        <linux-fsdevel@...r.kernel.org>,
        <linux-f2fs-devel@...ts.sourceforge.net>, <jaegeuk@...nel.org>
Subject: Re: [PATCH 1/1] f2fs: update multi-dev metadata in resize_fs

On 2019/9/18 20:51, sunqiuyang wrote:
> From: Qiuyang Sun <sunqiuyang@...wei.com>
> 
> Multi-device metadata should be updated in resize_fs as well.
> 
> Also, we check that the new FS size still reaches the last device.
> 
> Signed-off-by: Qiuyang Sun <sunqiuyang@...wei.com>
> ---
>  fs/f2fs/gc.c | 32 ++++++++++++++++++++++++++++++--
>  1 file changed, 30 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c
> index 5877bd7..a2b8cbe 100644
> --- a/fs/f2fs/gc.c
> +++ b/fs/f2fs/gc.c
> @@ -1431,26 +1431,46 @@ static void update_sb_metadata(struct f2fs_sb_info *sbi, int secs)
>  	int segment_count_main = le32_to_cpu(raw_sb->segment_count_main);
>  	long long block_count = le64_to_cpu(raw_sb->block_count);
>  	int segs = secs * sbi->segs_per_sec;
> +	int ndevs = sbi->s_ndevs;
>  
>  	raw_sb->section_count = cpu_to_le32(section_count + secs);
>  	raw_sb->segment_count = cpu_to_le32(segment_count + segs);
>  	raw_sb->segment_count_main = cpu_to_le32(segment_count_main + segs);
>  	raw_sb->block_count = cpu_to_le64(block_count +
>  					(long long)segs * sbi->blocks_per_seg);
> +	if (ndevs > 1) {

if (f2fs_is_multi_device(sbi)) {
	int last_ndev = sbi->s_ndevs - 1;


> +		int dev_segs =
> +			le32_to_cpu(raw_sb->devs[ndevs - 1].total_segments);
> +
> +		raw_sb->devs[ndevs - 1].total_segments =
> +						cpu_to_le32(dev_segs + segs);
> +	}
>  }
>  
>  static void update_fs_metadata(struct f2fs_sb_info *sbi, int secs)
>  {
>  	int segs = secs * sbi->segs_per_sec;
> +	long long blks = (long long)segs * sbi->blocks_per_seg;
>  	long long user_block_count =
>  				le64_to_cpu(F2FS_CKPT(sbi)->user_block_count);
> +	int ndevs = sbi->s_ndevs;
>  
>  	SM_I(sbi)->segment_count = (int)SM_I(sbi)->segment_count + segs;
>  	MAIN_SEGS(sbi) = (int)MAIN_SEGS(sbi) + segs;
>  	FREE_I(sbi)->free_sections = (int)FREE_I(sbi)->free_sections + secs;
>  	FREE_I(sbi)->free_segments = (int)FREE_I(sbi)->free_segments + segs;
> -	F2FS_CKPT(sbi)->user_block_count = cpu_to_le64(user_block_count +
> -					(long long)segs * sbi->blocks_per_seg);
> +	F2FS_CKPT(sbi)->user_block_count = cpu_to_le64(user_block_count + blks);
> +
> +	if (ndevs > 1) {

if (f2fs_is_multi_device(sbi)) {
	int last_ndev = sbi->s_ndevs - 1;

> +		FDEV(ndevs - 1).total_segments =
> +				(int)FDEV(ndevs - 1).total_segments + segs;
> +		FDEV(ndevs - 1).end_blk =
> +				(long long)FDEV(ndevs - 1).end_blk + blks;
> +#ifdef CONFIG_BLK_DEV_ZONED
> +		FDEV(ndevs - 1).nr_blkz = (int)FDEV(ndevs - 1).nr_blkz +
> +					(int)(blks >> sbi->log_blocks_per_blkz);
> +#endif
> +	}
>  }
>  
>  int f2fs_resize_fs(struct f2fs_sb_info *sbi, __u64 block_count)
> @@ -1465,6 +1485,14 @@ int f2fs_resize_fs(struct f2fs_sb_info *sbi, __u64 block_count)
>  	if (block_count > old_block_count)
>  		return -EINVAL;
>  
> +	if (sbi->s_ndevs > 1) {

if (f2fs_is_multi_device(sbi)) {
	int last_ndev = sbi->s_ndevs - 1;

Otherwise it looks good to me.

Thanks,

> +		__u64 last_segs = FDEV(sbi->s_ndevs - 1).total_segments;
> +
> +		if (block_count + last_segs * sbi->blocks_per_seg <=
> +								old_block_count)
> +			return -EINVAL;
> +	}
> +
>  	/* new fs size should align to section size */
>  	div_u64_rem(block_count, BLKS_PER_SEC(sbi), &rem);
>  	if (rem)
>