forked from torvalds/linux
-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SPEC2017: 538.imagick_r failure #23
Comments
shahab-vahedi
pushed a commit
that referenced
this issue
Nov 23, 2022
We enabled UBSAN in the ubuntu kernel, and the cs35l41 driver triggers a warning calltrace like below: cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: bitoffset= 8, word_offset=23, bit_sum mod 32=0, otp_map[i].size = 24 cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: bitoffset= 0, word_offset=24, bit_sum mod 32=24, otp_map[i].size = 0 ================================================================================ UBSAN: shift-out-of-bounds in linux-kernel-src/sound/soc/codecs/cs35l41-lib.c:836:8 shift exponent 64 is too large for 64-bit type 'long unsigned int' CPU: 10 PID: 595 Comm: systemd-udevd Not tainted 5.15.0-23-generic #23 Hardware name: LENOVO \x02MFG_IN_GO/\x02MFG_IN_GO, BIOS N3GET19W (1.00 ) 03/11/2022 Call Trace: <TASK> show_stack+0x52/0x58 dump_stack_lvl+0x4a/0x5f dump_stack+0x10/0x12 ubsan_epilogue+0x9/0x45 __ubsan_handle_shift_out_of_bounds.cold+0x61/0xef ? regmap_unlock_mutex+0xe/0x10 cs35l41_otp_unpack.cold+0x1c6/0x2b2 [snd_soc_cs35l41_lib] cs35l41_hda_probe+0x24f/0x33a [snd_hda_scodec_cs35l41] cs35l41_hda_i2c_probe+0x65/0x90 [snd_hda_scodec_cs35l41_i2c] When both bitoffset and otp_map[i].size are 0, the line 836 will result in GENMASK(-1, 0), this triggers the shift-out-of-bounds calltrace. Here add a checking, if both bitoffset and otp_map[i].size are 0, do not run GENMASK() and directly set otp_val to 0, this will not bring any function change on the driver but could avoid the calltrace. Signed-off-by: Hui Wang <hui.wang@canonical.com> Link: https://lore.kernel.org/r/20220324081839.62009-2-hui.wang@canonical.com Signed-off-by: Mark Brown <broonie@kernel.org>
xxkent
added a commit
that referenced
this issue
Aug 1, 2023
We have to change soft irq number to 22 since the previous number 21 now is occupied by CDMA. #23 is occupied by cluster PMU. 22 is the last unused by HW irq number.
xxkent
added a commit
that referenced
this issue
Aug 2, 2023
We have to change soft irq number to 22 since the previous number 21 now is occupied by CDMA. #23 is occupied by cluster PMU. 22 is the last unused by HW irq number.
xxkent
added a commit
that referenced
this issue
Aug 4, 2023
We have to change soft irq number to 22 since the previous number 21 now is occupied by CDMA. #23 is occupied by cluster PMU. 22 is the last unused by HW irq number.
xxkent
added a commit
that referenced
this issue
Oct 10, 2023
We have to change soft irq number to 22 since the previous number 21 now is occupied by CDMA. #23 is occupied by cluster PMU. 22 is the last unused by HW irq number.
xxkent
added a commit
that referenced
this issue
Oct 17, 2023
We have to change soft irq number to 22 since the previous number 21 now is occupied by CDMA. #23 is occupied by cluster PMU. 22 is the last unused by HW irq number.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
538.imagick_r test fails and calls stack trace.
How to reproduce this build:
https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/wiki/SPEC2017#how-to-run-prebuilt-spec
Config file used:
arc.cfg.txt
[Delte .txt extension before usage]
Test output: see below or in the file:
538.imagick_r_stack_trace.log
The text was updated successfully, but these errors were encountered: