On Tue, Jun 15, 2021 at 4:55 AM David Howells <dhowells@xxxxxxxxxx> wrote:
>
> From: Zheng Zengkai <zhengzengkai@xxxxxxxxxx>
>
> Add missing return to fix following compilation issue:
>
> fs/afs/dir.c: In function ‘afs_dir_set_page_dirty’:
> fs/afs/dir.c:51:1: error: no return statement in function
> returning non-void [-Werror=return-type]
This warning is actively wrong, and the patch is the wrong thing to do.
What compiler / architecture / config?
Because BUG() should have an "unreachable()", and the compiler should
know that a return statement isn't needed (and adding it shouldn't
make any difference).
And it's not warning for me when I build that code. So I really think
the real bug is entirely somewhere else, and this patch is papering
over the real problem.
Linus
