Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] DSP panic with Zephyr upstream main #8908

Closed
kv2019i opened this issue Mar 6, 2024 · 4 comments
Closed

[BUG] DSP panic with Zephyr upstream main #8908

kv2019i opened this issue Mar 6, 2024 · 4 comments
Assignees
@kv2019i
Labels
bug Something isn't working as expected MTL Applies to Meteor Lake platform P1 Blocker bugs or important features Zephyr Issues only observed with Zephyr integrated

Comments

@kv2019i
Copy link
Collaborator

kv2019i commented Mar 6, 2024 •
edited by marc-hb
Loading

Describe the bug
This is a follow-up to #8818 and shadows Zephyr issue zephyrproject-rtos/zephyr#69807

Let's keep this open until we can remove the revert from SOF side.

To Reproduce
Build SOF with latest Zephyr main.

Reproduction Rate
100%

Expected behavior
Tst case passes

Impact
DSP panic, audio lost

Environment
See #8818 and https://sof-ci.01.org/sofpr/PR8901/build3107/devicetest/index.html?model=MTLP_RVP_NOCODEC&testcase=check-playback-10sec

Screenshots or console output
https://sof-ci.01.org/sofpr/PR8901/build3107/devicetest/index.html?model=MTLP_RVP_NOCODEC&testcase=check-playback-10sec

cc:
@kv2019i kv2019i added bug Something isn't working as expected P1 Blocker bugs or important features Zephyr Issues only observed with Zephyr integrated MTL Applies to Meteor Lake platform labels Mar 6, 2024
This was referenced Mar 6, 2024
Closed
Merged
Closed
@marc-hb
Copy link
Collaborator

marc-hb commented Mar 8, 2024

Observed with CAVS25 too:
https://sof-ci.01.org/sofpr/PR8781/build3249/devicetest/index.html?model=TGLU_RVP_NOCODEC-ipc4&testcase=check-playback-3times

Funny enough, in the same test run ACE did not crash:
https://sof-ci.01.org/sofpr/PR8781/build3248/devicetest/index.html

So it is not deterministic indeed.

This was referenced Mar 8, 2024
Merged
Closed
Merged
@kv2019i kv2019i self-assigned this Mar 11, 2024
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 11, 2024

Two fix candidates under work:

@kv2019i kv2019i mentioned this issue Mar 13, 2024
Merged
kv2019i added a commit to kv2019i/sof that referenced this issue Mar 13, 2024
@kv2019i
west.yml: update Zephyr to 35176b993d4e
40c3e0e 
Update Zephyr to newer baseline. Among other updates, this brings in
Zephyr commit "intel_adsp/ace: power: Lock interruption when power gate
fails" that fixes a frequent issue seen in SOF CI on Intel MTL
platforms.

Link: thesofproject#8908
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i kv2019i mentioned this issue Mar 13, 2024
Merged
kv2019i added a commit to kv2019i/sof that referenced this issue Mar 13, 2024
@kv2019i
west.yml: update Zephyr to 251f52cbceb8
eb5628b 
Update Zephyr to newer baseline. Among other updates, this brings in
Zephyr commit "intel_adsp/ace: power: Lock interruption when power gate
fails" that fixes a frequent issue seen in SOF CI on Intel MTL
platforms.

Patches affecting SOF:

251f52cbceb8 list_hardware.py: sort rglob(SOC_YML) HWMv2 results
f91065b7c983 nxp: adsp: enable usage of DCACHE API
03a79d638773 llext: elf.h: fix sign of the `r_addend` field of struct elf32_rela
7eb1d3d57d2d llext: elf.h: fix wrong 32 bits length of elf64_rela fields
4ad9f9063848 llext.c: fix logging of 64 bits pointers
07426a800c74 intel_adsp/ace: power: Lock interruption when power gate fails
c15f029a7108 init.h: restore designated initializers in SYS_INIT_NAMED()
223e81b96888 soc: intel_adsp/ace: use CONFIG_XTENSA_MORE_SPIN_RELAX_NOPS
0a6e90b3b833 xtensa: make assembly-called functions static
3fa291690f32 xtensa: use CONFIG_PRIVILEGED_STACK_SIZE for stack size

Link: thesofproject#8908
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i kv2019i mentioned this issue Mar 13, 2024
Closed
kv2019i added a commit that referenced this issue Mar 14, 2024
@kv2019i
west.yml: update Zephyr to 251f52cbceb8
494f424 
Update Zephyr to newer baseline. Among other updates, this brings in
Zephyr commit "intel_adsp/ace: power: Lock interruption when power gate
fails" that fixes a frequent issue seen in SOF CI on Intel MTL
platforms.

Patches affecting SOF:

251f52cbceb8 list_hardware.py: sort rglob(SOC_YML) HWMv2 results
f91065b7c983 nxp: adsp: enable usage of DCACHE API
03a79d638773 llext: elf.h: fix sign of the `r_addend` field of struct elf32_rela
7eb1d3d57d2d llext: elf.h: fix wrong 32 bits length of elf64_rela fields
4ad9f9063848 llext.c: fix logging of 64 bits pointers
07426a800c74 intel_adsp/ace: power: Lock interruption when power gate fails
c15f029a7108 init.h: restore designated initializers in SYS_INIT_NAMED()
223e81b96888 soc: intel_adsp/ace: use CONFIG_XTENSA_MORE_SPIN_RELAX_NOPS
0a6e90b3b833 xtensa: make assembly-called functions static
3fa291690f32 xtensa: use CONFIG_PRIVILEGED_STACK_SIZE for stack size

Link: #8908
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i kv2019i mentioned this issue Mar 14, 2024
Closed
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 14, 2024

Closed via #8938

@kv2019i kv2019i closed this as completed Mar 14, 2024
@marc-hb
Copy link
Collaborator

marc-hb commented Mar 15, 2024

The next panic is #8942

eddy1021 pushed a commit to eddy1021/sof that referenced this issue Jul 15, 2024
@kv2019i
west.yml: update Zephyr to 251f52cbceb8
ab15635 
Update Zephyr to newer baseline. Among other updates, this brings in
Zephyr commit "intel_adsp/ace: power: Lock interruption when power gate
fails" that fixes a frequent issue seen in SOF CI on Intel MTL
platforms.

Patches affecting SOF:

251f52cbceb8 list_hardware.py: sort rglob(SOC_YML) HWMv2 results
f91065b7c983 nxp: adsp: enable usage of DCACHE API
03a79d638773 llext: elf.h: fix sign of the `r_addend` field of struct elf32_rela
7eb1d3d57d2d llext: elf.h: fix wrong 32 bits length of elf64_rela fields
4ad9f9063848 llext.c: fix logging of 64 bits pointers
07426a800c74 intel_adsp/ace: power: Lock interruption when power gate fails
c15f029a7108 init.h: restore designated initializers in SYS_INIT_NAMED()
223e81b96888 soc: intel_adsp/ace: use CONFIG_XTENSA_MORE_SPIN_RELAX_NOPS
0a6e90b3b833 xtensa: make assembly-called functions static
3fa291690f32 xtensa: use CONFIG_PRIVILEGED_STACK_SIZE for stack size

Link: thesofproject#8908
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kv2019i kv2019i

Labels
bug Something isn't working as expected MTL Applies to Meteor Lake platform P1 Blocker bugs or important features Zephyr Issues only observed with Zephyr integrated
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kv2019i @marc-hb