diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2023-09-11 17:38:51 +0300 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2023-10-16 19:44:03 +0300 |
commit | 6ed8cdf967f7e9fc96cd1c129719ef99db2f9afc (patch) | |
tree | 4b9ce6b63f6cad82cd9b9db5f1f6ac039a627c2a /net/sunrpc/xprtrdma/backchannel.c | |
parent | 7b31f4daebad296e3164602b8303c265ec4ac7dc (diff) | |
download | linux-6ed8cdf967f7e9fc96cd1c129719ef99db2f9afc.tar.xz |
SUNRPC: Clean up bc_svc_process()
The test robot complained that, in some build configurations, the
@error variable in bc_svc_process's only caller is set but never
used. This happens because dprintk() is the only consumer of that
value.
- Remove the dprintk() call sites in favor of the svc_process
tracepoint
- The @error variable and the return value of bc_svc_process() are
now unused, so get rid of them.
- The @serv parameter is set to rqstp->rq_serv by the only caller,
and bc_svc_process() then uses it only to set rqstp->rq_serv. It
can be removed.
- Rename bc_svc_process() according to the convention that
globally-visible RPC server functions have names that begin with
"svc_"; and because it is globally-visible, give it a proper
kdoc comment.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202308121314.HA8Rq2XG-lkp@intel.com/
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'net/sunrpc/xprtrdma/backchannel.c')
0 files changed, 0 insertions, 0 deletions