app: os_linux: Remove CONFIG_ADSP_IMR_CONTEXT_SAVE=n #10356
Conversation
Remove CONFIG_ADSP_IMR_CONTEXT_SAVE=n from app/os_linux_overlay.conf. This is a temporary remedy to overcome an issue we currently have in loading individual llext modules on second boot. The open-modules bundle apparently works Ok still. Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
jsarha
requested review from
dbaluta,
kv2019i,
lbetlej,
lgirdwood,
mmaka1 and
plbossart
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR removes the configuration setting that disables ADSP IMR context save in the Linux overlay configuration. This change addresses a boot issue with individual llext module loading by allowing firmware state retention.
- Removes
CONFIG_ADSP_IMR_CONTEXT_SAVE=nconfiguration setting
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
kv2019i
left a comment
kv2019i
left a comment
There was a problem hiding this comment.
Thanks @jsarha ! This is a bit odd to have an empty file, but this allows to enable the Linux overlay in CI instances, and then submit a revert of this PR and get CI feedback before merge.
FYI @cgturner1
We do need to figure out some way to identify the overlay use in FW version. Upto now, the combination of SOF version and hw target, has been unique so any bug reports could have been tracked to a specific source/build configuration. With the overlay, this link is broken and we can't anymore figure out from kernel/FW logs, how the FW was built. I'll probably file a ticket for this.
3 participants
Remove CONFIG_ADSP_IMR_CONTEXT_SAVE=n from app/os_linux_overlay.conf.
This is a temporary remedy to overcome an issue we currently have in loading individual llext modules on second boot. The open-modules bundle apparently works Ok still.