diff options
author | Arnd Bergmann <arnd@arndb.de> | 2019-06-28 14:37:48 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-07-14 08:11:22 +0200 |
commit | 70bae382b3dc54bfec309d09c5ac86ee62f47260 (patch) | |
tree | fac6bb7f612f3879a092f2c5561747bd08ce138d /include/linux/adfs_fs.h | |
parent | b46475ecd9304aef278bd12c0731b4955789b698 (diff) | |
download | linux-stable-70bae382b3dc54bfec309d09c5ac86ee62f47260.tar.gz linux-stable-70bae382b3dc54bfec309d09c5ac86ee62f47260.tar.bz2 linux-stable-70bae382b3dc54bfec309d09c5ac86ee62f47260.zip |
staging: rtl8712: reduce stack usage, again
commit fbd6b25009ac76b2034168cd21d5e01f8c2d83d1 upstream.
An earlier patch I sent reduced the stack usage enough to get
below the warning limit, and I could show this was safe, but with
GCC_PLUGIN_STRUCTLEAK_BYREF_ALL, it gets worse again because large stack
variables in the same function no longer overlap:
drivers/staging/rtl8712/rtl871x_ioctl_linux.c: In function 'translate_scan.isra.2':
drivers/staging/rtl8712/rtl871x_ioctl_linux.c:322:1: error: the frame size of 1200 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]
Split out the largest two blocks in the affected function into two
separate functions and mark those noinline_for_stack.
Fixes: 8c5af16f7953 ("staging: rtl8712: reduce stack usage")
Fixes: 81a56f6dcd20 ("gcc-plugins: structleak: Generalize to all variable types")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/linux/adfs_fs.h')
0 files changed, 0 insertions, 0 deletions