From: "Gustavo A. R. Silva" <gustavo@embeddedor.com>
To: David Howells <dhowells@redhat.com>
Cc: linux-afs@lists.infradead.org, linux-kernel@vger.kernel.org,
"Gustavo A. R. Silva" <gustavo@embeddedor.com>
Subject: [PATCH] afs: Mark expected switch fall-throughs
Date: Tue, 31 Jul 2018 16:58:25 -0500 [thread overview]
Message-ID: <20180731215825.GA10268@embeddedor.com> (raw)
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Warning level 2 was used: -Wimplicit-fallthrough=2
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
---
fs/afs/cmservice.c | 10 ++++++++--
fs/afs/file.c | 2 ++
fs/afs/fsclient.c | 41 +++++++++++++++++++++++++++--------------
fs/afs/rxrpc.c | 1 +
fs/afs/vlclient.c | 29 ++++++++++++++++-------------
5 files changed, 54 insertions(+), 29 deletions(-)
diff --git a/fs/afs/cmservice.c b/fs/afs/cmservice.c
index 9e51d6f..c06fcda 100644
--- a/fs/afs/cmservice.c
+++ b/fs/afs/cmservice.c
@@ -179,7 +179,8 @@ static int afs_deliver_cb_callback(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the FID array and its count in two steps */
+ /* Fall through and extract the FID array and its count
+ * in two steps. */
case 1:
_debug("extract FID count");
ret = afs_extract_data(call, &call->tmp, 4, true);
@@ -198,6 +199,7 @@ static int afs_deliver_cb_callback(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+ /* Fall through */
case 2:
_debug("extract FID array");
ret = afs_extract_data(call, call->buffer,
@@ -224,7 +226,8 @@ static int afs_deliver_cb_callback(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the callback array and its count in two steps */
+ /* Fall through and extract the callback array and its count
+ * in two steps. */
case 3:
_debug("extract CB count");
ret = afs_extract_data(call, &call->tmp, 4, true);
@@ -238,6 +241,7 @@ static int afs_deliver_cb_callback(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+ /* Fall through */
case 4:
_debug("extract CB array");
ret = afs_extract_data(call, call->buffer,
@@ -336,6 +340,7 @@ static int afs_deliver_cb_init_call_back_state3(struct afs_call *call)
return -ENOMEM;
call->unmarshall++;
+ /* Fall through */
case 1:
_debug("extract UUID");
ret = afs_extract_data(call, call->buffer,
@@ -459,6 +464,7 @@ static int afs_deliver_cb_probe_uuid(struct afs_call *call)
return -ENOMEM;
call->unmarshall++;
+ /* Fall through */
case 1:
_debug("extract UUID");
ret = afs_extract_data(call, call->buffer,
diff --git a/fs/afs/file.c b/fs/afs/file.c
index 7d4f261..4a853ba 100644
--- a/fs/afs/file.c
+++ b/fs/afs/file.c
@@ -299,6 +299,8 @@ int afs_page_filler(void *data, struct page *page)
/* page will not be cached */
case -ENOBUFS:
_debug("cache said ENOBUFS");
+
+ /* fall through */
default:
go_on:
req = kzalloc(sizeof(struct afs_read) + sizeof(struct page *),
diff --git a/fs/afs/fsclient.c b/fs/afs/fsclient.c
index 50929cb..34fa791 100644
--- a/fs/afs/fsclient.c
+++ b/fs/afs/fsclient.c
@@ -485,9 +485,9 @@ static int afs_deliver_fs_fetch_data(struct afs_call *call)
goto no_msw;
}
- /* extract the upper part of the returned data length of an
- * FSFETCHDATA64 op (which should always be 0 using this
- * client) */
+ /* Fall through and extract the upper part of the returned data
+ * length of an FSFETCHDATA64 op (which should always be 0 using
+ * this client) */
case 1:
_debug("extract data length (MSW)");
ret = afs_extract_data(call, &call->tmp, 4, true);
@@ -499,6 +499,7 @@ static int afs_deliver_fs_fetch_data(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+ /* Fall through */
no_msw:
/* extract the returned data length */
case 2:
@@ -527,7 +528,7 @@ static int afs_deliver_fs_fetch_data(struct afs_call *call)
ASSERTCMP(call->count, <=, PAGE_SIZE);
req->remain -= size;
- /* extract the returned data */
+ /* Fall through and extract the returned data */
case 3:
_debug("extract data %llu/%llu %zu/%u",
req->remain, req->actual_len, call->offset, call->count);
@@ -572,7 +573,7 @@ static int afs_deliver_fs_fetch_data(struct afs_call *call)
call->offset = 0;
call->unmarshall = 5;
- /* extract the metadata */
+ /* Fall through and extract the metadata */
case 5:
ret = afs_extract_data(call, call->buffer,
(21 + 3 + 6) * 4, false);
@@ -1590,7 +1591,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the returned status record */
+ /* Fall through and extract the returned status record */
case 1:
_debug("extract status");
ret = afs_extract_data(call, call->buffer,
@@ -1603,7 +1604,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the volume name length */
+ /* Fall through extract the volume name length */
case 2:
ret = afs_extract_data(call, &call->tmp, 4, true);
if (ret < 0)
@@ -1616,7 +1617,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the volume name */
+ /* Fall through and extract the volume name */
case 3:
_debug("extract volname");
if (call->count > 0) {
@@ -1640,6 +1641,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
}
call->count = 4 - (call->count & 3);
+ /* Fall through */
case 4:
ret = afs_extract_data(call, call->buffer,
call->count, true);
@@ -1648,6 +1650,8 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+
+ /* Fall through */
no_volname_padding:
/* extract the offline message length */
@@ -1663,7 +1667,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the offline message */
+ /* Fall through and extract the offline message */
case 6:
_debug("extract offline");
if (call->count > 0) {
@@ -1687,6 +1691,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
}
call->count = 4 - (call->count & 3);
+ /* Fall through */
case 7:
ret = afs_extract_data(call, call->buffer,
call->count, true);
@@ -1695,6 +1700,8 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+
+ /* Fall through */
no_offline_padding:
/* extract the message of the day length */
@@ -1710,7 +1717,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* extract the message of the day */
+ /* Fall through and extract the message of the day */
case 9:
_debug("extract motd");
if (call->count > 0) {
@@ -1730,6 +1737,7 @@ static int afs_deliver_fs_get_volume_status(struct afs_call *call)
/* extract the message of the day padding */
call->count = (4 - (call->count & 3)) & 3;
+ /* Fall through */
case 10:
ret = afs_extract_data(call, call->buffer,
call->count, false);
@@ -2012,7 +2020,7 @@ static int afs_deliver_fs_get_capabilities(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract the capabilities word count */
+ /* Fall through and extract the capabilities word count */
case 1:
ret = afs_extract_data(call, &call->tmp,
1 * sizeof(__be32),
@@ -2027,7 +2035,7 @@ static int afs_deliver_fs_get_capabilities(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract capabilities words */
+ /* Fall through and extract capabilities words */
case 2:
count = min(call->count, 16U);
ret = afs_extract_data(call, call->buffer,
@@ -2196,7 +2204,8 @@ static int afs_deliver_fs_inline_bulk_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract the file status count and array in two steps */
+ /* Fall through and extract the file status count and array
+ * in two steps. */
case 1:
_debug("extract status count");
ret = afs_extract_data(call, &call->tmp, 4, true);
@@ -2213,6 +2222,7 @@ static int afs_deliver_fs_inline_bulk_status(struct afs_call *call)
more_counts:
call->offset = 0;
+ /* Fall through */
case 2:
_debug("extract status array %u", call->count);
ret = afs_extract_data(call, call->buffer, 21 * 4, true);
@@ -2234,7 +2244,8 @@ static int afs_deliver_fs_inline_bulk_status(struct afs_call *call)
call->unmarshall++;
call->offset = 0;
- /* Extract the callback count and array in two steps */
+ /* Fall through and extract the callback count and array in
+ * two steps. */
case 3:
_debug("extract CB count");
ret = afs_extract_data(call, &call->tmp, 4, true);
@@ -2250,6 +2261,7 @@ static int afs_deliver_fs_inline_bulk_status(struct afs_call *call)
more_cbs:
call->offset = 0;
+ /* Fall through */
case 4:
_debug("extract CB array");
ret = afs_extract_data(call, call->buffer, 3 * 4, true);
@@ -2272,6 +2284,7 @@ static int afs_deliver_fs_inline_bulk_status(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
+ /* Fall through */
case 5:
ret = afs_extract_data(call, call->buffer, 6 * 4, false);
if (ret < 0)
diff --git a/fs/afs/rxrpc.c b/fs/afs/rxrpc.c
index 183cc54..d0f915a 100644
--- a/fs/afs/rxrpc.c
+++ b/fs/afs/rxrpc.c
@@ -842,6 +842,7 @@ void afs_send_empty_reply(struct afs_call *call)
_debug("oom");
rxrpc_kernel_abort_call(net->socket, call->rxcall,
RX_USER_ABORT, -ENOMEM, "KOO");
+ /* Fall through */
default:
_leave(" [error]");
return;
diff --git a/fs/afs/vlclient.c b/fs/afs/vlclient.c
index c3b7408..e94735c 100644
--- a/fs/afs/vlclient.c
+++ b/fs/afs/vlclient.c
@@ -195,7 +195,9 @@ static int afs_deliver_vl_get_addrs_u(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract the returned uuid, uniquifier, nentries and blkaddrs size */
+
+ /* Fall through and extract the returned uuid, uniquifier,
+ * nentries and blkaddrs size. */
case 1:
ret = afs_extract_data(call, call->buffer,
sizeof(struct afs_uuid__xdr) + 3 * sizeof(__be32),
@@ -219,7 +221,7 @@ static int afs_deliver_vl_get_addrs_u(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract entries */
+ /* Fall through and extract entries */
case 2:
count = min(call->count, 4U);
ret = afs_extract_data(call, call->buffer,
@@ -326,7 +328,7 @@ static int afs_deliver_vl_get_capabilities(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract the capabilities word count */
+ /* Fall through and extract the capabilities word count */
case 1:
ret = afs_extract_data(call, &call->tmp,
1 * sizeof(__be32),
@@ -341,7 +343,7 @@ static int afs_deliver_vl_get_capabilities(struct afs_call *call)
call->offset = 0;
call->unmarshall++;
- /* Extract capabilities words */
+ /* Fall through and extract capabilities words */
case 2:
count = min(call->count, 16U);
ret = afs_extract_data(call, call->buffer,
@@ -434,9 +436,9 @@ static int afs_deliver_yfsvl_get_endpoints(struct afs_call *call)
call->offset = 0;
call->unmarshall = 1;
- /* Extract the returned uuid, uniquifier, fsEndpoints count and
- * either the first fsEndpoint type or the volEndpoints
- * count if there are no fsEndpoints. */
+ /* Fall through and extract the returned uuid, uniquifier,
+ * fsEndpoints count and either the first fsEndpoint type
+ * or the volEndpoints count if there are no fsEndpoints. */
case 1:
ret = afs_extract_data(call, call->buffer,
sizeof(uuid_t) +
@@ -465,7 +467,7 @@ static int afs_deliver_yfsvl_get_endpoints(struct afs_call *call)
call->unmarshall = 2;
- /* Extract fsEndpoints[] entries */
+ /* Fall through and extract fsEndpoints[] entries */
case 2:
switch (call->count2) {
case YFS_ENDPOINT_IPV4:
@@ -522,9 +524,10 @@ static int afs_deliver_yfsvl_get_endpoints(struct afs_call *call)
call->unmarshall = 3;
- /* Extract the type of volEndpoints[0]. Normally we would
- * extract the type of the next endpoint when we extract the
- * data of the current one, but this is the first...
+ /* Fall through and extract the type of volEndpoints[0].
+ * Normally we would extract the type of the next endpoint
+ * when we extract the data of the current one, but this is
+ * the first...
*/
case 3:
ret = afs_extract_data(call, call->buffer, sizeof(__be32), true);
@@ -536,7 +539,7 @@ static int afs_deliver_yfsvl_get_endpoints(struct afs_call *call)
call->offset = 0;
call->unmarshall = 4;
- /* Extract volEndpoints[] entries */
+ /* Fall through and extract volEndpoints[] entries */
case 4:
switch (call->count2) {
case YFS_ENDPOINT_IPV4:
@@ -584,7 +587,7 @@ static int afs_deliver_yfsvl_get_endpoints(struct afs_call *call)
end:
call->unmarshall = 5;
- /* Done */
+ /* Fall through - Done */
case 5:
ret = afs_extract_data(call, call->buffer, 0, false);
if (ret < 0)
--
2.7.4
next reply other threads:[~2018-07-31 22:21 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-31 21:58 Gustavo A. R. Silva [this message]
2018-08-02 9:12 ` [PATCH] afs: Mark expected switch fall-throughs David Howells
-- strict thread matches above, loose matches on Subject: below --
2018-10-17 11:36 Gustavo A. R. Silva
2019-01-10 22:02 Gustavo A. R. Silva
2019-04-09 0:10 ` Kees Cook
2019-04-09 9:28 ` David Howells
2019-04-09 15:18 ` Kees Cook
2019-04-16 13:35 ` David Howells
2019-04-09 9:34 ` David Howells
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180731215825.GA10268@embeddedor.com \
--to=gustavo@embeddedor.com \
--cc=dhowells@redhat.com \
--cc=linux-afs@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.