btrfs: usage error should not be logged into system log
authorAnand Jain <Anand.Jain@oracle.com>
Tue, 20 May 2014 06:38:11 +0000 (14:38 +0800)
committerChris Mason <clm@fb.com>
Tue, 10 Jun 2014 00:20:54 +0000 (17:20 -0700)
I have an opinion that system logs /var/log/messages are
valuable info to investigate the real system issues at
the data center. People handling data center issues
do spend a lot time and efforts analyzing messages
files. Having usage error logged into /var/log/messages
is something we should avoid.

Signed-off-by: Anand Jain <Anand.Jain@oracle.com>
Reviewed-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/sysfs.c

index ba2a645..c0dfda5 100644 (file)
@@ -374,11 +374,8 @@ static ssize_t btrfs_label_store(struct kobject *kobj,
        struct btrfs_root *root = fs_info->fs_root;
        int ret;
 
-       if (len >= BTRFS_LABEL_SIZE) {
-               pr_err("BTRFS: unable to set label with more than %d bytes\n",
-                      BTRFS_LABEL_SIZE - 1);
+       if (len >= BTRFS_LABEL_SIZE)
                return -EINVAL;
-       }
 
        trans = btrfs_start_transaction(root, 0);
        if (IS_ERR(trans))