Message ID | de233b2c-8487-5be6-c189-7fca5d453048@suse.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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.
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)