diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2020-10-02 00:59:18 +0200 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2020-10-02 15:37:41 +0200 |
commit | 84c138e78d8ec247737d2c9e8dba50f20eb01a60 (patch) | |
tree | 94ddfdaa455d7d86bff99f08572903a58bf89bc7 /fs/nfsd | |
parent | NFSD: Encoder and decoder functions are always present (diff) | |
download | linux-84c138e78d8ec247737d2c9e8dba50f20eb01a60.tar.xz linux-84c138e78d8ec247737d2c9e8dba50f20eb01a60.zip |
NFSD: Clean up switch statement in nfsd_dispatch()
Reorder the arms so the compiler places checks for the most frequent
case first.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd')
-rw-r--r-- | fs/nfsd/nfssvc.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c index b2d20920a998..3cdefb2294ce 100644 --- a/fs/nfsd/nfssvc.c +++ b/fs/nfsd/nfssvc.c @@ -1030,12 +1030,12 @@ nfsd_dispatch(struct svc_rqst *rqstp, __be32 *statp) /* Check whether we have this call in the cache. */ switch (nfsd_cache_lookup(rqstp)) { - case RC_DROPIT: - return 0; + case RC_DOIT: + break; case RC_REPLY: return 1; - case RC_DOIT:; - /* do it */ + case RC_DROPIT: + return 0; } /* need to grab the location to store the status, as |