diff options
author | Amadeusz Sławiński <[email protected]> | 2022-07-22 13:19:59 +0200 |
---|---|---|
committer | Mark Brown <[email protected]> | 2022-07-22 13:40:02 +0100 |
commit | 1e744351bcb9c4cee81300de5a6097100d835386 (patch) | |
tree | 2631ae689dcda62451ae051b216c6bd52f1dc8b0 /scripts/gdb/linux/utils.py | |
parent | b340128432a2b8849cc34f9653d7c43c83102bbd (diff) |
ASoC: Intel: avs: Use lookup table to create modules
As reported by Nathan, when building avs driver using clang with:
CONFIG_COMPILE_TEST=y
CONFIG_FORTIFY_SOURCE=y
CONFIG_KASAN=y
CONFIG_PCI=y
CONFIG_SOUND=y
CONFIG_SND=y
CONFIG_SND_SOC=y
CONFIG_SND_SOC_INTEL_AVS=y
there are reports of too big stack use, like:
sound/soc/intel/avs/path.c:815:18: error: stack frame size (2176) exceeds limit (2048) in 'avs_path_create' [-Werror,-Wframe-larger-than]
struct avs_path *avs_path_create(struct avs_dev *adev, u32 dma_id,
^
1 error generated.
This is apparently caused by inlining many calls to guid_equal which
inlines fortified memcpy, using 2 size_t variables.
Instead of hardcoding many calls to guid_equal, use lookup table with
one call, this improves stack usage.
Link: https://lore.kernel.org/alsa-devel/[email protected]/T/
Link: https://github.com/ClangBuiltLinux/linux/issues/1642
Signed-off-by: Amadeusz Sławiński <[email protected]>
Reported-by: Nathan Chancellor <[email protected]>
Build-tested-by: Nathan Chancellor <[email protected]>
Reviewed-by: Cezary Rojewski <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>
Diffstat (limited to 'scripts/gdb/linux/utils.py')
0 files changed, 0 insertions, 0 deletions