-
Notifications
You must be signed in to change notification settings - Fork 322
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
intel_adsp_ace15_mtpm.conf: temporarily disable CONFIG_MODULES #9313
Conversation
Temporarily disable CONFIG_MODULES to get MTL working again. Avoids crash thesofproject#9308 Signed-off-by: Marc Herbert <[email protected]>
MTL https://sof-ci.01.org/sofpr/PR9313/build6572/devicetest/index.html This one-line CONFIG_ change moves us from "NOTHING works on MTL" back to "everything works on MTL". This is NOT an actual fix #9308 but this restores project velocity while not even reverting a single line of actual code. This does not even lose much LLEXT coverage because LNL is still being tested with LLEXT! I don't think there can be any least intrusive way to get MTL CI back up and running and things moving again. EDIT: kudos to @fredoh9 for leading me to this minimal solution: #9308 (comment) The 2 pause failures in LNL https://sof-ci.01.org/sofpr/PR9313/build6571/devicetest/index.html are known (thesofproject/linux#5109 and #9123) and unrelated. This PR makes absolutely zero LNL change. No cavs failure in https://sof-ci.01.org/sofpr/PR9313/build6573/devicetest/index.html, just a model missing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable. Weird that the resume crash is MTL-specific though
Wow, good finding for minimum change. 👍 |
Yes: this seems like a very complex issue. All the relevant PRs passed in isolation, see details in #9308. I think this shows that bug is complex and unlikely to be root-caused and fixed in a short time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporarily disable CONFIG_MODULES to get MTL working again.
Avoids crash #9308