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

Revert "pm: Remove CURRENT_CPU macro" #69937

Conversation

kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Mar 7, 2024

This reverts commit b9d4b9d.

System resume fails occasionally on Intel ACE platform to assert with this patch in place. Revert the change until a proper fix is found.

Link: #69807

This reverts commit b9d4b9d.

System resume fails occasionally on Intel ACE platform to assert with
this patch in place. Revert the change until a proper fix is found.

Link: zephyrproject-rtos#69807
Signed-off-by: Kai Vehmanen <[email protected]>
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 7, 2024

Example results without the revert (not 100% reproduction rate but the DSP panic is hit still at a very high rate and most test cases are failing on Intel MTL):
https://sof-ci.01.org/sofpr/PR8913/build3216/devicetest/index.html

@kv2019i kv2019i requested review from lyakh and dcpleung March 8, 2024 12:44
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 11, 2024

So alternative to this revert is #68493 but it has still review comments open. Getting one of the solutions merged fairly quickly would be appreciated given this issue is hit at high rate in SOF public CI.

@kv2019i kv2019i marked this pull request as draft March 13, 2024 08:43
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 13, 2024

#68493 merged. Converted to a draft and will close if 68493 holds up.

@kv2019i kv2019i closed this Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants