summaryrefslogtreecommitdiff
path: root/include/linux/sunrpc/svc_xprt.h
diff options
context:
space:
mode:
authorChuck Lever <chuck.lever@oracle.com>2022-01-25 18:17:59 +0300
committerChuck Lever <chuck.lever@oracle.com>2022-02-28 18:26:39 +0300
commita9ff2e99e9fa501ec965da03c18a5422b37a2f44 (patch)
tree0d9598fc3eda25fa163c5ddc07bae3c99e83e98c /include/linux/sunrpc/svc_xprt.h
parent70a60cbfb613d8f6ffd1d9ade187d0a868066500 (diff)
downloadlinux-a9ff2e99e9fa501ec965da03c18a5422b37a2f44.tar.xz
SUNRPC: Remove the .svo_enqueue_xprt method
We have never been able to track down and address the underlying cause of the performance issues with workqueue-based service support. svo_enqueue_xprt is called multiple times per RPC, so it adds instruction path length, but always ends up at the same function: svc_xprt_do_enqueue(). We do not anticipate needing this flexibility for dynamic nfsd thread management support. As a micro-optimization, remove .svo_enqueue_xprt because Spectre/Meltdown makes virtual function calls more costly. This change essentially reverts commit b9e13cdfac70 ("nfsd/sunrpc: turn enqueueing a svc_xprt into a svc_serv operation"). Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'include/linux/sunrpc/svc_xprt.h')
-rw-r--r--include/linux/sunrpc/svc_xprt.h1
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/sunrpc/svc_xprt.h b/include/linux/sunrpc/svc_xprt.h
index 571f605bc91e..a3ba027fb4ba 100644
--- a/include/linux/sunrpc/svc_xprt.h
+++ b/include/linux/sunrpc/svc_xprt.h
@@ -131,7 +131,6 @@ int svc_create_xprt(struct svc_serv *, const char *, struct net *,
const int, const unsigned short, int,
const struct cred *);
void svc_xprt_received(struct svc_xprt *xprt);
-void svc_xprt_do_enqueue(struct svc_xprt *xprt);
void svc_xprt_enqueue(struct svc_xprt *xprt);
void svc_xprt_put(struct svc_xprt *xprt);
void svc_xprt_copy_addrs(struct svc_rqst *rqstp, struct svc_xprt *xprt);