erofs: kill verbose debug info in erofs_fill_super
authorGao Xiang <gaoxiang25@huawei.com>
Wed, 4 Sep 2019 02:09:01 +0000 (10:09 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 5 Sep 2019 18:10:08 +0000 (20:10 +0200)
As Christoph said [1], "That is some very verbose
debug info.  We usually don't add that and let
people trace the function instead. "

[1] https://lore.kernel.org/r/20190829101545.GC20598@infradead.org/
Reported-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Gao Xiang <gaoxiang25@huawei.com>
Link: https://lore.kernel.org/r/20190904020912.63925-15-gaoxiang25@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/erofs/super.c

index 63cb17a..b64d69f 100644 (file)
@@ -384,9 +384,6 @@ static int erofs_fill_super(struct super_block *sb, void *data, int silent)
        struct erofs_sb_info *sbi;
        int err;
 
-       infoln("fill_super, device -> %s", sb->s_id);
-       infoln("options -> %s", (char *)data);
-
        sb->s_magic = EROFS_SUPER_MAGIC;
 
        if (!sb_set_blocksize(sb, EROFS_BLKSIZ)) {
@@ -419,9 +416,6 @@ static int erofs_fill_super(struct super_block *sb, void *data, int silent)
        if (err)
                return err;
 
-       if (!silent)
-               infoln("root inode @ nid %llu", ROOT_NID(sbi));
-
        if (test_opt(sbi, POSIX_ACL))
                sb->s_flags |= SB_POSIXACL;
        else
@@ -454,7 +448,8 @@ static int erofs_fill_super(struct super_block *sb, void *data, int silent)
                return err;
 
        if (!silent)
-               infoln("mounted on %s with opts: %s.", sb->s_id, (char *)data);
+               infoln("mounted on %s with opts: %s, root inode @ nid %llu.",
+                      sb->s_id, (char *)data, ROOT_NID(sbi));
        return 0;
 }