-Wsometimes-uninitialized in arch/arm64/include/asm/percpu.h

[nickdesaulniers]

Reported-by: Prasad Sodagudi psodagud@quicinc.com psodagud@codeaurora.org

./arch/arm64/include/asm/percpu.h:105:1: warning: variable 'ret' is used uninitialized whenever switch default is taken [-Wsometimes-uninitialized]

I think that you can initialize ret in the default case
besides calling BUILD_BUG_ON();.

[nathanchance]

Hmmm how is this warning being triggered? I just did a simple defconfig build for arm64 (which should show it since it's in a super generic header no?) and I didn't see it anywhere. Do I need a full allyesconfig?

I'll cook up a patch shortly and ship it along but I am curious how to get this show up in my build.

[nathanchance]

I'm also guessing this same situation applies? https://lore.kernel.org/lkml/CAKwvOdnjgEWDo8vjOiAS_TsEoJFZZagJgaqBAdW99L-Nkr9BxA@mail.gmail.com/

[nickdesaulniers]

allyesconfig. Yes on updating the reporter's email, too. Good catch.

[nathanchance]

Comes from drivers/staging/erofs in a few files, thanks for the clarification.

Patch sent: https://lore.kernel.org/lkml/20180925194459.13088-1-natechancellor@gmail.com/

[nathanchance]

I think that was the quickest patch acceptance I've ever had: https://git.kernel.org/pub/scm/linux/kernel/git/dennis/percpu.git/commit/?id=b5bb425871186303e6936fa2581521bdd1964a58

[nathanchance]

Merged into mainline: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=b5bb425871186303e6936fa2581521bdd1964a58