diff options
| author | Arnd Bergmann <[email protected]> | 2025-06-20 13:09:53 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2025-06-23 13:21:32 +0000 |
| commit | 95b6759a81833d0e8c7456430186c2f6d174764e (patch) | |
| tree | b640b6a61c2ac1bf2b3eb3a89ceb88e2d8328b06 /net/unix/af_unix.c | |
| parent | atm: clip: prevent NULL deref in clip_push() (diff) | |
| download | kernel-95b6759a81833d0e8c7456430186c2f6d174764e.tar.gz kernel-95b6759a81833d0e8c7456430186c2f6d174764e.zip | |
net: qed: reduce stack usage for TLV processing
clang gets a bit confused by the code in the qed_mfw_process_tlv_req and
ends up spilling registers to the stack hundreds of times. When sanitizers
are enabled, this can end up blowing the stack warning limit:
drivers/net/ethernet/qlogic/qed/qed_mng_tlv.c:1244:5: error: stack frame size (1824) exceeds limit (1280) in 'qed_mfw_process_tlv_req' [-Werror,-Wframe-larger-than]
Apparently the problem is the complexity of qed_mfw_update_tlvs()
after inlining, and marking the four main branches of that function
as noinline_for_stack makes this problem completely go away, the stack
usage goes down to 100 bytes.
Signed-off-by: Arnd Bergmann <[email protected]>
Reviewed-by: Alexander Lobakin <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions