diff options
author | Nathan Chancellor <nathan@kernel.org> | 2021-12-10 14:36:27 -0700 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2022-01-04 17:29:31 -0800 |
commit | be185c2988b48db65348d94168c793bdbc8d23c3 (patch) | |
tree | 8642d0d5e1d76a9b4b831b7a4ce0d33139ad9a87 /kernel/jump_label.c | |
parent | 53989fad1286e652ea3655ae3367ba698da8d2ff (diff) | |
download | linux-be185c2988b48db65348d94168c793bdbc8d23c3.tar.bz2 |
cxl/core: Remove cxld_const_init in cxl_decoder_alloc()
Commit 48667f676189 ("cxl/core: Split decoder setup into alloc + add")
aimed to fix a large stack frame warning but from v5 to v6, it
introduced a new instance of the warning due to allocating
cxld_const_init on the stack, which was done due to the use of const on
the nr_target member of the cxl_decoder struct. With ARCH=arm
allmodconfig minus CONFIG_KASAN:
GCC 11.2.0:
drivers/cxl/core/bus.c: In function ‘cxl_decoder_alloc’:
drivers/cxl/core/bus.c:523:1: error: the frame size of 1032 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]
523 | }
| ^
cc1: all warnings being treated as errors
Clang 12.0.1:
drivers/cxl/core/bus.c:486:21: error: stack frame size of 1056 bytes in function 'cxl_decoder_alloc' [-Werror,-Wframe-larger-than=]
struct cxl_decoder *cxl_decoder_alloc(struct cxl_port *port, int nr_targets)
^
1 error generated.
Revert that part of the change, which makes the stack frame of
cxl_decoder_alloc() much more reasonable.
Fixes: 48667f676189 ("cxl/core: Split decoder setup into alloc + add")
Link: https://github.com/ClangBuiltLinux/linux/issues/1539
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Link: https://lore.kernel.org/r/20211210213627.2477370-1-nathan@kernel.org
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'kernel/jump_label.c')
0 files changed, 0 insertions, 0 deletions