diff mbox series

powerpc: net: bpf_jit_comp: Fix misuse of fallthrough

Message ID 20200928090023.38117-1-zhe.he@windriver.com (mailing list archive)
State Accepted
Commit 9cf51446e68607136e42a4e531a30c888c472463
Headers show
Series powerpc: net: bpf_jit_comp: Fix misuse of fallthrough | expand

Commit Message

He Zhe Sept. 28, 2020, 9 a.m. UTC 
From: He Zhe <zhe.he@windriver.com>

The user defined label following "fallthrough" is not considered by GCC
and causes build failure.

kernel-source/include/linux/compiler_attributes.h:208:41: error: attribute
'fallthrough' not preceding a case label or default label [-Werror]
 208   define fallthrough _attribute((fallthrough_))
                          ^~~~~~~~~~~~~

Signed-off-by: He Zhe <zhe.he@windriver.com>
---
 arch/powerpc/net/bpf_jit_comp.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Daniel Borkmann Sept. 29, 2020, 2:41 p.m. UTC | #1 
On 9/28/20 11:00 AM, zhe.he@windriver.com wrote:
> From: He Zhe <zhe.he@windriver.com>
> 
> The user defined label following "fallthrough" is not considered by GCC
> and causes build failure.
> 
> kernel-source/include/linux/compiler_attributes.h:208:41: error: attribute
> 'fallthrough' not preceding a case label or default label [-Werror]
>   208   define fallthrough _attribute((fallthrough_))
>                            ^~~~~~~~~~~~~
> 
> Signed-off-by: He Zhe <zhe.he@windriver.com>

Applied, thanks! I've also added Fixes tag with df561f6688fe ("treewide: Use fallthrough pseudo-keyword")
which added the bug.
patchwork-bot+bpf@kernel.org Sept. 29, 2020, 2:50 p.m. UTC | #2 
Hello:

This patch was applied to bpf/bpf.git (refs/heads/master):

On Mon, 28 Sep 2020 17:00:23 +0800 you wrote:
> From: He Zhe <zhe.he@windriver.com>
> 
> The user defined label following "fallthrough" is not considered by GCC
> and causes build failure.
> 
> kernel-source/include/linux/compiler_attributes.h:208:41: error: attribute
> 'fallthrough' not preceding a case label or default label [-Werror]
>  208   define fallthrough _attribute((fallthrough_))
>                           ^~~~~~~~~~~~~
> 
> [...]

Here is the summary with links:
  - powerpc: net: bpf_jit_comp: Fix misuse of fallthrough
    https://git.kernel.org/bpf/bpf/c/9cf51446e686

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/arch/powerpc/net/bpf_jit_comp.c b/arch/powerpc/net/bpf_jit_comp.c
index 78d61f97371e..e809cb5a1631 100644
--- a/arch/powerpc/net/bpf_jit_comp.c
+++ b/arch/powerpc/net/bpf_jit_comp.c
@@ -475,7 +475,6 @@  static int bpf_jit_build_body(struct bpf_prog *fp, u32 *image,
 		case BPF_JMP | BPF_JSET | BPF_K:
 		case BPF_JMP | BPF_JSET | BPF_X:
 			true_cond = COND_NE;
-			fallthrough;
 		cond_branch:
 			/* same targets, can avoid doing the test :) */
 			if (filter[i].jt == filter[i].jf) {