nfsd: Log client tracking type log message as info instead of warning
authorPaul Menzel <pmenzel@molgen.mpg.de>
Fri, 12 Mar 2021 21:03:00 +0000 (22:03 +0100)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 22 Mar 2021 14:19:03 +0000 (10:19 -0400)
`printk()`, by default, uses the log level warning, which leaves the
user reading

    NFSD: Using UMH upcall client tracking operations.

wondering what to do about it (`dmesg --level=warn`).

Several client tracking methods are tried, and expected to fail. That’s
why a message is printed only on success. It might be interesting for
users to know the chosen method, so use info-level instead of
debug-level.

Cc: linux-nfs@vger.kernel.org
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4recover.c

index 891395c..6fedc49 100644 (file)
@@ -626,7 +626,7 @@ nfsd4_legacy_tracking_init(struct net *net)
        status = nfsd4_load_reboot_recovery_data(net);
        if (status)
                goto err;
-       printk("NFSD: Using legacy client tracking operations.\n");
+       pr_info("NFSD: Using legacy client tracking operations.\n");
        return 0;
 
 err:
@@ -1028,7 +1028,7 @@ nfsd4_init_cld_pipe(struct net *net)
 
        status = __nfsd4_init_cld_pipe(net);
        if (!status)
-               printk("NFSD: Using old nfsdcld client tracking operations.\n");
+               pr_info("NFSD: Using old nfsdcld client tracking operations.\n");
        return status;
 }
 
@@ -1605,7 +1605,7 @@ nfsd4_cld_tracking_init(struct net *net)
                nfs4_release_reclaim(nn);
                goto err_remove;
        } else
-               printk("NFSD: Using nfsdcld client tracking operations.\n");
+               pr_info("NFSD: Using nfsdcld client tracking operations.\n");
        return 0;
 
 err_remove:
@@ -1864,7 +1864,7 @@ nfsd4_umh_cltrack_init(struct net *net)
        ret = nfsd4_umh_cltrack_upcall("init", NULL, grace_start, NULL);
        kfree(grace_start);
        if (!ret)
-               printk("NFSD: Using UMH upcall client tracking operations.\n");
+               pr_info("NFSD: Using UMH upcall client tracking operations.\n");
        return ret;
 }