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.

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

west.yml: update Zephyr to 251f52cbceb8 (March 13) #8938

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Mar 13, 2024

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: #8908

@iuliana-prodan
Copy link
Contributor

@kv2019i we have a pending PR in zephyr we want to include in SOF - zephyrproject-rtos/zephyr#70032
If we are not in a rush, can we wait for that PR to be merge? Hope it will be today. :)
And update the west.yml sha to the new commit id.

PS: I know we can do a new update to zephyr, but maybe we can catch this one, since the NXP PR has been approved.

@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

@iuliana-prodan Ack. Let's check how the CI results go. If we need to make updates in any case, I'll pick that PR up as well, otherwise let's do a quick follow-up in another PR. I want to do pretty frequent updates now, so we can pick any possible problems early and not let them pile up.

@iuliana-prodan
Copy link
Contributor

@iuliana-prodan Ack. Let's check how the CI results go. If we need to make updates in any case, I'll pick that PR up as well, otherwise let's do a quick follow-up in another PR.

Ok, sure. Thanks!

I want to do pretty frequent updates now, so we can pick any possible problems early and not let them pile up.

Totally agree.

@dbaluta
Copy link
Collaborator

dbaluta commented Mar 13, 2024

@kv2019i PR merged. Can you please add this hash into your w

SOF - zephyrproject-rtos/zephyr#70032

Merged now. @kv2019i can you please help include this hash too?

@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

@wszypelt I'll repush this with, but it seems the previous test failed to a test code checkout error (internal id 13712043). Let me see if the new build fares better.

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
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

No DSP panic seen in Intel MTL test, so the fix seems to hold up. https://sof-ci.01.org/sofpr/PR8938/build3358/devicetest/index.html Doing a force-push but wanted to capture a link to the test results here as a comment.

@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

MTL test still looks good https://sof-ci.01.org/sofpr/PR8938/build3370/devicetest/index.html There's one known pause-resume fail which should be addressed by other fixes (in review in Zephyr upstream).

@kv2019i kv2019i requested a review from lyakh March 13, 2024 14:44
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

Hmm, we have still DSP panic on https://sof-ci.01.org/sofpr/PR8938/build3371/devicetest/index.html on cAVS2.5. This is looks like a different scenario even if the assert is the same as other platforms.
UPDATE: filed this as #8942 (not blocking this merge, seen before this pull request so not a regression

@kv2019i kv2019i changed the title west.yml: update Zephyr to 35176b993d4e west.yml: update Zephyr to 251f52cbceb8 (March 13) Mar 13, 2024
@kv2019i kv2019i merged commit 494f424 into thesofproject:main Mar 14, 2024
40 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants