Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v2 1/2] ext4: rename journal_dev to s_journal_dev inside
 ext4_sb_info
To:     brookxu <brookxu.cn@gmail.com>, tytso@mit.edu,
        adilger.kernel@dilger.ca
Cc:     linux-ext4@vger.kernel.org
References: <561ad829-542a-2ed2-349f-62ff0ac7fa19@gmail.com>
From:   Ritesh Harjani <riteshh@linux.ibm.com>
Date:   Thu, 13 Aug 2020 21:20:42 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.8.0
MIME-Version: 1.0
In-Reply-To: <561ad829-542a-2ed2-349f-62ff0ac7fa19@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Message-Id: <20200813155042.C2682A4051@d06av23.portsmouth.uk.ibm.com>
Sender: linux-ext4-owner@vger.kernel.org
Precedence: bulk


On 8/7/20 11:50 AM, brookxu wrote:
> Rename journal_dev to s_journal_dev inside ext4_sb_info, keep
> the naming rules consistent with other variables, which is
> convenient for code reading and writing.
> 
> Signed-off-by: Chunguang Xu <brookxu@tencent.com>

It seems the author email and Signed-off-by emails are different
and checkpatch complains. Highlighting for your FYI -

./scripts/checkpatch.pl 1.patch
WARNING: Missing Signed-off-by: line by nominal patch author 'brookxu 
<brookxu.cn@gmail.com>'

Other than that, threading ("--thread") on your patch series will help
add proper headers in the email so that it could be seen as threaded
emails.


Patch looks good to me. Please feel free to add,
Reviewed-by: Ritesh Harjani <riteshh@linux.ibm.com>


> ---
>   fs/ext4/ext4.h  |  2 +-
>   fs/ext4/fsmap.c |  8 ++++----
>   fs/ext4/super.c | 14 +++++++-------
>   3 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
> index 68e0ebe..8ca9adf 100644
> --- a/fs/ext4/ext4.h
> +++ b/fs/ext4/ext4.h
> @@ -1463,7 +1463,7 @@ struct ext4_sb_info {
>   	unsigned long s_commit_interval;
>   	u32 s_max_batch_time;
>   	u32 s_min_batch_time;
> -	struct block_device *journal_bdev;
> +	struct block_device *s_journal_bdev;
>   #ifdef CONFIG_QUOTA
>   	/* Names of quota files with journalled quota */
>   	char __rcu *s_qf_names[EXT4_MAXQUOTAS];
> diff --git a/fs/ext4/fsmap.c b/fs/ext4/fsmap.c
> index dbccf46..005c0ae 100644
> --- a/fs/ext4/fsmap.c
> +++ b/fs/ext4/fsmap.c
> @@ -571,8 +571,8 @@ static bool ext4_getfsmap_is_valid_device(struct super_block *sb,
>   	if (fm->fmr_device == 0 || fm->fmr_device == UINT_MAX ||
>   	    fm->fmr_device == new_encode_dev(sb->s_bdev->bd_dev))
>   		return true;
> -	if (EXT4_SB(sb)->journal_bdev &&
> -	    fm->fmr_device == new_encode_dev(EXT4_SB(sb)->journal_bdev->bd_dev))
> +	if (EXT4_SB(sb)->s_journal_bdev &&
> +	    fm->fmr_device == new_encode_dev(EXT4_SB(sb)->s_journal_bdev->bd_dev))
>   		return true;
>   	return false;
>   }
> @@ -642,9 +642,9 @@ int ext4_getfsmap(struct super_block *sb, struct ext4_fsmap_head *head,
>   	memset(handlers, 0, sizeof(handlers));
>   	handlers[0].gfd_dev = new_encode_dev(sb->s_bdev->bd_dev);
>   	handlers[0].gfd_fn = ext4_getfsmap_datadev;
> -	if (EXT4_SB(sb)->journal_bdev) {
> +	if (EXT4_SB(sb)->s_journal_bdev) {
>   		handlers[1].gfd_dev = new_encode_dev(
> -				EXT4_SB(sb)->journal_bdev->bd_dev);
> +				EXT4_SB(sb)->s_journal_bdev->bd_dev);
>   		handlers[1].gfd_fn = ext4_getfsmap_logdev;
>   	}
> 
> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> index 8ce61f3..f785aee7 100644
> --- a/fs/ext4/super.c
> +++ b/fs/ext4/super.c
> @@ -935,10 +935,10 @@ static void ext4_blkdev_put(struct block_device *bdev)
>   static void ext4_blkdev_remove(struct ext4_sb_info *sbi)
>   {
>   	struct block_device *bdev;
> -	bdev = sbi->journal_bdev;
> +	bdev = sbi->s_journal_bdev;
>   	if (bdev) {
>   		ext4_blkdev_put(bdev);
> -		sbi->journal_bdev = NULL;
> +		sbi->s_journal_bdev = NULL;
>   	}
>   }
> 
> @@ -1069,14 +1069,14 @@ static void ext4_put_super(struct super_block *sb)
> 
>   	sync_blockdev(sb->s_bdev);
>   	invalidate_bdev(sb->s_bdev);
> -	if (sbi->journal_bdev && sbi->journal_bdev != sb->s_bdev) {
> +	if (sbi->s_journal_bdev && sbi->s_journal_bdev != sb->s_bdev) {
>   		/*
>   		 * Invalidate the journal device's buffers.  We don't want them
>   		 * floating about in memory - the physical journal device may
>   		 * hotswapped, and it breaks the `ro-after' testing code.
>   		 */
> -		sync_blockdev(sbi->journal_bdev);
> -		invalidate_bdev(sbi->journal_bdev);
> +		sync_blockdev(sbi->s_journal_bdev);
> +		invalidate_bdev(sbi->s_journal_bdev);
>   		ext4_blkdev_remove(sbi);
>   	}
> 
> @@ -3712,7 +3712,7 @@ int ext4_calculate_overhead(struct super_block *sb)
>   	 * Add the internal journal blocks whether the journal has been
>   	 * loaded or not
>   	 */
> -	if (sbi->s_journal && !sbi->journal_bdev)
> +	if (sbi->s_journal && !sbi->s_journal_bdev)
>   		overhead += EXT4_NUM_B2C(sbi, sbi->s_journal->j_maxlen);
>   	else if (ext4_has_feature_journal(sb) && !sbi->s_journal && j_inum) {
>   		/* j_inum for internal journal is non-zero */
> @@ -5057,7 +5057,7 @@ static journal_t *ext4_get_dev_journal(struct super_block *sb,
>   			be32_to_cpu(journal->j_superblock->s_nr_users));
>   		goto out_journal;
>   	}
> -	EXT4_SB(sb)->journal_bdev = bdev;
> +	EXT4_SB(sb)->s_journal_bdev = bdev;
>   	ext4_init_journal_params(sb, journal);
>   	return journal;
>