[PATCH] btrfs: clean up some inconsistent indenting

[PATCH] btrfs: clean up some inconsistent indenting

[Jiapeng Chong]

No functional modification involved.

fs/btrfs/volumes.c:770 device_list_add() warn: inconsistent indenting.
fs/btrfs/volumes.c:1373 btrfs_scan_one_device() warn: inconsistent indenting.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8453
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 fs/btrfs/volumes.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
index a2d07fa3cfdf..caa3e83b0d6c 100644
--- a/fs/btrfs/volumes.c
+++ b/fs/btrfs/volumes.c
@@ -767,7 +767,7 @@ static noinline struct btrfs_device *device_list_add(const char *path,
 		if (same_fsid_diff_dev) {
 			generate_random_uuid(fs_devices->fsid);
 			fs_devices->temp_fsid = true;
-		pr_info("BTRFS: device %s (%d:%d) using temp-fsid %pU\n",
+			pr_info("BTRFS: device %s (%d:%d) using temp-fsid %pU\n",
 				path, MAJOR(path_devt), MINOR(path_devt),
 				fs_devices->fsid);
 		}
@@ -1370,8 +1370,9 @@ struct btrfs_device *btrfs_scan_one_device(const char *path, blk_mode_t flags,
 		else
 			btrfs_free_stale_devices(devt, NULL);
 
-	pr_debug("BTRFS: skip registering single non-seed device %s (%d:%d)\n",
-			path, MAJOR(devt), MINOR(devt));
+		pr_debug("BTRFS: skip registering single non-seed device %s (%d:%d)\n",
+			 path, MAJOR(devt), MINOR(devt));
+
 		device = NULL;
 		goto free_disk_super;
 	}
-- 
2.20.1.7.g153144c

Re: [PATCH] btrfs: clean up some inconsistent indenting

[Qu Wenruo]

在 2024/3/7 17:17, Jiapeng Chong 写道:
> No functional modification involved.
>
> fs/btrfs/volumes.c:770 device_list_add() warn: inconsistent indenting.
> fs/btrfs/volumes.c:1373 btrfs_scan_one_device() warn: inconsistent indenting.
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8453
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>

Can the bot be taught to watch the btrfs development branch
(https://github.com/btrfs/linux.git for-next) and send feedback directly
to the btrfs mailing list?
So that we can catch the problem immediately before reaching mainline.

I strongly doubt sending out such patches would leave any positive
impression on the bot.

Thanks,
Qu
> ---
>   fs/btrfs/volumes.c | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
> index a2d07fa3cfdf..caa3e83b0d6c 100644
> --- a/fs/btrfs/volumes.c
> +++ b/fs/btrfs/volumes.c
> @@ -767,7 +767,7 @@ static noinline struct btrfs_device *device_list_add(const char *path,
>   		if (same_fsid_diff_dev) {
>   			generate_random_uuid(fs_devices->fsid);
>   			fs_devices->temp_fsid = true;
> -		pr_info("BTRFS: device %s (%d:%d) using temp-fsid %pU\n",
> +			pr_info("BTRFS: device %s (%d:%d) using temp-fsid %pU\n",
>   				path, MAJOR(path_devt), MINOR(path_devt),
>   				fs_devices->fsid);
>   		}
> @@ -1370,8 +1370,9 @@ struct btrfs_device *btrfs_scan_one_device(const char *path, blk_mode_t flags,
>   		else
>   			btrfs_free_stale_devices(devt, NULL);
>
> -	pr_debug("BTRFS: skip registering single non-seed device %s (%d:%d)\n",
> -			path, MAJOR(devt), MINOR(devt));
> +		pr_debug("BTRFS: skip registering single non-seed device %s (%d:%d)\n",
> +			 path, MAJOR(devt), MINOR(devt));
> +
>   		device = NULL;
>   		goto free_disk_super;
>   	}