]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
net: sunrpc: mark expected switch fall-throughs
authorGustavo A. R. Silva <garsilva@embeddedor.com>
Fri, 20 Oct 2017 16:48:30 +0000 (11:48 -0500)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Fri, 17 Nov 2017 21:43:44 +0000 (16:43 -0500)
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.

Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
net/sunrpc/clnt.c
net/sunrpc/xprt.c
net/sunrpc/xprtsock.c

index 2ad827db270422ed7f2c84aae374c234f445e6ab..d25b077d326de8d517c04ff23b266ce79236d210 100644 (file)
@@ -1586,6 +1586,7 @@ call_reserveresult(struct rpc_task *task)
        switch (status) {
        case -ENOMEM:
                rpc_delay(task, HZ >> 2);
+               /* fall through */
        case -EAGAIN:   /* woken up; retry */
                task->tk_action = call_retry_reserve;
                return;
@@ -1647,10 +1648,13 @@ call_refreshresult(struct rpc_task *task)
                /* Use rate-limiting and a max number of retries if refresh
                 * had status 0 but failed to update the cred.
                 */
+               /* fall through */
        case -ETIMEDOUT:
                rpc_delay(task, 3*HZ);
+               /* fall through */
        case -EAGAIN:
                status = -EACCES;
+               /* fall through */
        case -EKEYEXPIRED:
                if (!task->tk_cred_retry)
                        break;
@@ -1911,6 +1915,7 @@ call_connect_status(struct rpc_task *task)
                        task->tk_action = call_bind;
                        return;
                }
+               /* fall through */
        case -ECONNRESET:
        case -ECONNABORTED:
        case -ENETUNREACH:
@@ -1924,6 +1929,7 @@ call_connect_status(struct rpc_task *task)
                        break;
                /* retry with existing socket, after a delay */
                rpc_delay(task, 3*HZ);
+               /* fall through */
        case -EAGAIN:
                /* Check for timeouts before looping back to call_bind */
        case -ETIMEDOUT:
@@ -2025,6 +2031,7 @@ call_transmit_status(struct rpc_task *task)
                        rpc_exit(task, task->tk_status);
                        break;
                }
+               /* fall through */
        case -ECONNRESET:
        case -ECONNABORTED:
        case -EADDRINUSE:
@@ -2145,6 +2152,7 @@ call_status(struct rpc_task *task)
                 * were a timeout.
                 */
                rpc_delay(task, 3*HZ);
+               /* fall through */
        case -ETIMEDOUT:
                task->tk_action = call_timeout;
                break;
@@ -2152,14 +2160,17 @@ call_status(struct rpc_task *task)
        case -ECONNRESET:
        case -ECONNABORTED:
                rpc_force_rebind(clnt);
+               /* fall through */
        case -EADDRINUSE:
                rpc_delay(task, 3*HZ);
+               /* fall through */
        case -EPIPE:
        case -ENOTCONN:
                task->tk_action = call_bind;
                break;
        case -ENOBUFS:
                rpc_delay(task, HZ>>2);
+               /* fall through */
        case -EAGAIN:
                task->tk_action = call_transmit;
                break;
index e741ec2b4d8e6ea5d08a8e942e3021e5a20e4c6a..02a9bacb239bbc969c946107c663da0a75c929e8 100644 (file)
@@ -1139,6 +1139,7 @@ void xprt_alloc_slot(struct rpc_xprt *xprt, struct rpc_task *task)
        case -EAGAIN:
                xprt_add_backlog(xprt, task);
                dprintk("RPC:       waiting for request slot\n");
+               /* fall through */
        default:
                task->tk_status = -EAGAIN;
        }
index c1841f234a717fdcf46413e25cb1e265912a2dfb..684e356b40e4c329a785909a8d27162896ccd938 100644 (file)
@@ -551,6 +551,7 @@ static int xs_local_send_request(struct rpc_task *task)
        default:
                dprintk("RPC:       sendmsg returned unrecognized error %d\n",
                        -status);
+               /* fall through */
        case -EPIPE:
                xs_close(xprt);
                status = -ENOTCONN;
@@ -1610,6 +1611,7 @@ static void xs_tcp_state_change(struct sock *sk)
                xprt->connect_cookie++;
                clear_bit(XPRT_CONNECTED, &xprt->state);
                xs_tcp_force_close(xprt);
+               /* fall through */
        case TCP_CLOSING:
                /*
                 * If the server closed down the connection, make sure that
@@ -2367,6 +2369,7 @@ static int xs_tcp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock)
        switch (ret) {
        case 0:
                xs_set_srcport(transport, sock);
+               /* fall through */
        case -EINPROGRESS:
                /* SYN_SENT! */
                if (xprt->reestablish_timeout < XS_TCP_INIT_REEST_TO)
@@ -2418,6 +2421,7 @@ static void xs_tcp_setup_socket(struct work_struct *work)
        default:
                printk("%s: connect returned unhandled error %d\n",
                        __func__, status);
+               /* fall through */
        case -EADDRNOTAVAIL:
                /* We're probably in TIME_WAIT. Get rid of existing socket,
                 * and retry