On Thu, May 31, 2018 at 6:56 PM, Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote: > 2018-05-31 12:53 GMT+09:00 Kees Cook <keescook@xxxxxxxxxxxx>: >> On Wed, May 30, 2018 at 6:26 PM, Kees Cook <keescook@xxxxxxxxxxxx> wrote: >>> On Wed, May 30, 2018 at 6:12 PM, Masahiro Yamada >>> <yamada.masahiro@xxxxxxxxxxxxx> wrote: >>>> Hi. >>>> (+CC Kees) >>>> >>>> 2018-05-31 7:40 GMT+09:00 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>: >>>>> Hi Masahiro, >>>>> >>>>> After merging the kbuild tree, today's linux-next build (x86_64 >>>>> allmodconfig) produced these warnings: >>>>> >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_workarounds_nphy_rev7': >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:16563:1: warning: the frame size of 3136 bytes is larger than 2048 bytes [-Wframe-larger-than=] >>>>> } >>>>> ^ >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_workarounds_nphy_rev3': >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:16905:1: warning: the frame size of 2872 bytes is larger than 2048 bytes [-Wframe-larger-than=] >>>>> } >>>>> ^ >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_cal_txiqlo_nphy': >>>>> drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:26033:1: warning: the frame size of 2432 bytes is larger than 2048 bytes [-Wframe-larger-than=] >>>>> } >>>>> ^ >>>>> >>>>> I have no idea what caused these warnings to appear ... nothing in those >>>>> functions looks too bad. >>>> >>>> >>>> This has been triggered by the following commit: >>>> >>>> >>>> commit 0e461945f3504e09b8ecf947b6398adce1287a28 >>>> Author: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> >>>> Date: Mon May 28 18:22:07 2018 +0900 >>>> >>>> gcc-plugins: allow to enable GCC_PLUGINS for COMPILE_TEST >>>> >>>> >>>> >>>> CONFIG_GCC_PLUGIN_STRUCTLEAK_BYREF_ALL was previously disabled >>>> for COMPILE_TEST, which is now enabled. For the moment, can you add "depends on !COMPILE_TEST" to CONFIG_GCC_PLUGIN_STRUCTLEAK_BYREF_ALL in your tree and I'll continue to figure out what's happening? Thanks! -Kees -- Kees Cook Pixel Security -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html