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

[TEST] Update to a recent Zephyr hash, containing minimum fixes #8901

Closed
wants to merge 2 commits into from

Conversation

lyakh
Copy link
Collaborator

@lyakh lyakh commented Mar 5, 2024

A subset of #8732

dcpleung and others added 2 commits March 5, 2024 10:44
This changes the secondary core power up routine to use the newly
introduced k_smp_cpu_start() and k_smp_cpu_resume(). This removes
the need to mirror part of the SMP start up code from Zephyr, and
no longer need to call into Zephyr private kernel code.

West update includes :

eefaeee061c8 kernel: smp: introduce k_smp_cpu_resume
042cb6ac4e00 soc: intel_adsp: enable DfTTS-based time stamping
             on ACE platforms
6a0b1da158a4 soc: intel_adsp: call framework callback function for restore
e7217925c93e ace: use a 'switch' statement in pm_state_set()
c99a604bbf2c ace: remove superfluous variable initialisation
a0ac2faf9bde intel_adsp: ace: enable power domain
4204ca9bcb3f ace: fix DSP panic during startup  (fixes c3a6274bf5e4)
d4b0273ab0c4 cmake: sparse.template: add COMMAND_ERROR_IS_FATAL
ca12fd13c6d3 xtensa: intel_adsp: fix a cache handling error
0ee1e28a2f5f xtensa: polish doxygen and add to missing doc
035c8d8ceb4b xtensa: remove sys_define_gpr_with_alias()
a64eec6aaeec xtensa: remove XTENSA_ERR_NORET

Signed-off-by: Daniel Leung <[email protected]>
Signed-off-by: Rander Wang <[email protected]>
[[email protected]: update Zephyr hversion]
Signed-off-by: Guennadi Liakhovetski <[email protected]>
z_soc_uncached_ptr() / z_soc_cached_ptr() have been removed from
Zephyr and replaced with sys_cache_uncached_ptr_get() and
sys_cache_cached_ptr_get() respectively.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
@kv2019i
Copy link
Collaborator

kv2019i commented Mar 5, 2024

Hmm, @lyakh didn't work out, we got a panic:
https://sof-ci.01.org/sofpr/PR8901/build3107/devicetest/index.html?model=MTLP_RVP_NOCODEC&testcase=check-playback-10sec

ASSERTION FAIL [!z_smp_cpu_mobile()] @ ZEPHYR_BASE/subsys/pm/pm.c:133

Hmm... this was supposed to be fixed already, but alas we are still seeing it. See discussion in zephyrproject-rtos/zephyr#68234 and zephyrproject-rtos/zephyr#67863

kv2019i added a commit to thesofproject/zephyr that referenced this pull request Mar 5, 2024
This reverts commit b9d4b9d.

Needed to avoid assert in SOF CI test run as seen
on thesofproject/sof#8901

Signed-off-by: Kai Vehmanen <[email protected]>
@lyakh
Copy link
Collaborator Author

lyakh commented Mar 5, 2024

dropping this in favour of #8903

@lyakh lyakh closed this Mar 5, 2024
@lyakh lyakh deleted the zephyr-main branch March 5, 2024 14:34
serhiy-katsyuba-intel pushed a commit to serhiy-katsyuba-intel/zephyr that referenced this pull request Mar 6, 2024
This reverts commit b9d4b9d.

Needed to avoid assert in SOF CI test run as seen
on thesofproject/sof#8901

Signed-off-by: Kai Vehmanen <[email protected]>
kv2019i added a commit to kv2019i/zephyr that referenced this pull request Mar 7, 2024
This reverts commit b9d4b9d.

Needed to avoid assert in SOF CI test run as seen
on thesofproject/sof#8901

Signed-off-by: Kai Vehmanen <[email protected]>
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.

3 participants