summaryrefslogtreecommitdiff
path: root/samples/livepatch
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2017-09-23 00:29:12 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-10-27 11:39:14 +0300
commitd26c0a8723f859c0a87ad989fb977f798fddc50f (patch)
tree2b761094c098ee3a03647b448284d79a582d0fd7 /samples/livepatch
parentafc62a9460f24eab1c91752b3fb08ffbe16ffcb0 (diff)
downloadlinux-d26c0a8723f859c0a87ad989fb977f798fddc50f.tar.xz
brcmsmac: make some local variables 'static const' to reduce stack size
commit c503dd38f850be28867ef7a42d9abe5ade81a9bd upstream. With KASAN and a couple of other patches applied, this driver is one of the few remaining ones that actually use more than 2048 bytes of kernel stack: broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_workarounds_nphy_gainctrl': broadcom/brcm80211/brcmsmac/phy/phy_n.c:16065:1: warning: the frame size of 3264 bytes is larger than 2048 bytes [-Wframe-larger-than=] broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function 'wlc_phy_workarounds_nphy': broadcom/brcm80211/brcmsmac/phy/phy_n.c:17138:1: warning: the frame size of 2864 bytes is larger than 2048 bytes [-Wframe-larger-than=] Here, I'm reducing the stack size by marking as many local variables as 'static const' as I can without changing the actual code. This is the first of three patches to improve the stack usage in this driver. It would be good to have this backported to stabl kernels to get all drivers in 'allmodconfig' below the 2048 byte limit so we can turn on the frame warning again globally, but I realize that the patch is larger than the normal limit for stable backports. The other two patches do not need to be backported. Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Kalle Valo <kvalo@codeaurora.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'samples/livepatch')
0 files changed, 0 insertions, 0 deletions