diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2017-09-22 21:29:18 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-10-12 09:51:24 (GMT) |
| commit | 57a77fffb0ffa8d9a56536ab8f6c1028abe83137 (patch) | |
| tree | c52502bb1da783925ed8545f95f96633105c5526 /include | |
| parent | 50b27486ae8a3b3d9dddbfeaa6d6be843d414de4 (diff) | |
| download | linux-57a77fffb0ffa8d9a56536ab8f6c1028abe83137.tar.xz | |
rocker: fix rocker_tlv_put_* functions for KASAN
commit 6098d7ddd62f532f80ee2a4b01aca500a8e4e9e4 upstream.
Inlining these functions creates lots of stack variables that each take
64 bytes when KASAN is enabled, leading to this warning about potential
stack overflow:
drivers/net/ethernet/rocker/rocker_ofdpa.c: In function 'ofdpa_cmd_flow_tbl_add':
drivers/net/ethernet/rocker/rocker_ofdpa.c:621:1: error: the frame size of 2752 bytes is larger than 1536 bytes [-Werror=frame-larger-than=]
gcc-8 can now consolidate the stack slots itself, but on older versions
we get the same behavior by using a temporary variable that holds a
copy of the inline function argument.
Link: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81715
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions