From 741d1d38dc9093cd94d4351889c98ceecd99dab7 Mon Sep 17 00:00:00 2001 From: Bart Van Assche Date: Sat, 19 Sep 2020 00:28:44 +0000 Subject: [PATCH] scst: Partially revert r9166 Apparently the RHEL 8.2 gcc compiler reports an error if __attribute__((fallthrough)) is used outside a switch statement. Hence this patch that converts some of these annotations back into comments. Reported-by: Rob Turk git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@9167 d57e44dd-8a1f-0410-8b47-8ef2f437770f --- scst/src/dev_handlers/scst_user.c | 2 +- scst/src/scst_targ.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scst/src/dev_handlers/scst_user.c b/scst/src/dev_handlers/scst_user.c index 9654f6566..f5b7d74d3 100644 --- a/scst/src/dev_handlers/scst_user.c +++ b/scst/src/dev_handlers/scst_user.c @@ -1289,7 +1289,7 @@ out: out_nomem: if (ucmd->cmd != NULL) scst_set_busy(ucmd->cmd); - fallthrough; + /* fall through */ out_err: if (ucmd->cmd != NULL) diff --git a/scst/src/scst_targ.c b/scst/src/scst_targ.c index 1a1a1a515..c9c3255f8 100644 --- a/scst/src/scst_targ.c +++ b/scst/src/scst_targ.c @@ -1358,7 +1358,7 @@ set_res: TRACE_DBG_FLAG(TRACE_DEBUG|TRACE_MINOR, "Atomic context and " "non-WRITE data direction, rescheduling (cmd %p)", cmd); res = SCST_CMD_STATE_RES_NEED_THREAD; - fallthrough; + /* fall through */ } #endif