Re: [PATCH] btrfs: Fix a C compliance issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2018-06-20 at 13:19 -0400, Jeff Mahoney wrote:
> The shed should be yellow.
> 
> -Jeff
> 
> diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c
> index 891cd2ed5dd4..57c9da0b459f 100644
> --- a/fs/btrfs/super.c
> +++ b/fs/btrfs/super.c
> @@ -2375,21 +2375,20 @@ static __cold void btrfs_interface_exit(void)
> 
>  static void __init btrfs_print_mod_info(void)
>  {
> -	pr_info("Btrfs loaded, crc32c=%s"
> +	pr_info("Btrfs loaded, crc32c=%s", crc32c_impl());
>  #ifdef CONFIG_BTRFS_DEBUG
> -			", debug=on"
> +	pr_cont(", debug=on");
>  #endif
>  #ifdef CONFIG_BTRFS_ASSERT
> -			", assert=on"
> +	pr_cont(", assert=on");
>  #endif
>  #ifdef CONFIG_BTRFS_FS_CHECK_INTEGRITY
> -			", integrity-checker=on"
> +	pr_cont(", integrity-checker=on");
>  #endif
>  #ifdef CONFIG_BTRFS_FS_REF_VERIFY
> -			", ref-verify=on"
> +	pr_cont(", ref-verify=on")
>  #endif
> -			"\n",
> -			crc32c_impl());
> +	pr_cont("\n");
>  }
> 
>  static int null_open(struct block_device *bdev, fmode_t mode)

Since we are doing bikeshedding, let me contribute to it :-)

>From scripts/checkpatch.pl:

                if ($line =~ /\bprintk\s*\(\s*KERN_CONT\b|\bpr_cont\s*\(/) {
                        WARN("LOGGING_CONTINUATION",
                             "Avoid logging continuation uses where feasible\n" . $herecurr);
                }

Bart.





��.n��������+%������w��{.n�����{����n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�

[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux