diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-04-26 17:52:33 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-04-28 22:46:59 +0200 |
| commit | 6b87663fbe4a366e558d6566b3e6b6bc227da5a1 (patch) | |
| tree | 3a2e668fd6cb784974ae78dc353843c2f0fe6969 /net/tipc/subscr.c | |
| parent | tuntap: calculate rps hash only when needed (diff) | |
| download | linux-6b87663fbe4a366e558d6566b3e6b6bc227da5a1.tar.xz linux-6b87663fbe4a366e558d6566b3e6b6bc227da5a1.zip | |
net/mlx5e: avoid stack overflow in mlx5e_open_channels
struct mlx5e_channel_param is a large structure that is allocated
on the stack of mlx5e_open_channels, and with a recent change
it has grown beyond the warning size for the maximum stack
that a single function should use:
mellanox/mlx5/core/en_main.c: In function 'mlx5e_open_channels':
mellanox/mlx5/core/en_main.c:1325:1: error: the frame size of 1072 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]
The function is already using dynamic allocation and is not in
a fast path, so the easiest workaround is to use another kzalloc
for allocating the channel parameters.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: d3c9bc2743dc ("net/mlx5e: Added ICO SQs")
Acked-by: Saeed Mahameed <saeedm@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/subscr.c')
0 files changed, 0 insertions, 0 deletions