Skip to content

ace3.0: disable context saving#10128

Closed
lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:ctx
Closed

ace3.0: disable context saving#10128
lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:ctx

Conversation

@lyakh

@lyakh lyakh commented Jul 22, 2025

Copy link
Copy Markdown
Collaborator

We can now use the faster loadable module context restoring on ACE 3.0, disable the slower full context save.

We can now use the faster loadable module context restoring on ACE
3.0, disable the slower full context save.

Signed-off-by: Guennadi Liakhovetski <[email protected]>

@tmleman tmleman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The full context store/restore is verified in internal CI, so without removing the appropriate tests, the change will cause continuous failures.

@lgirdwood

lgirdwood commented Jul 23, 2025

Copy link
Copy Markdown
Member

@kv2019i can we use the new overlay for this when @cgturner1 is ready ? This way it wont impact the CI as reported by @tmleman

@kv2019i

kv2019i commented Jul 23, 2025

Copy link
Copy Markdown
Collaborator

@lyakh why this PR? we have the Linux overlay already just for this purpose -> #10084

@lyakh

lyakh commented Jul 24, 2025

Copy link
Copy Markdown
Collaborator Author

@lyakh why this PR? we have the Linux overlay already just for this purpose -> #10084

@kv2019i yes, but that overlay isn't used (yet). I think currently that overlay is considered the preferred way forward for this? I don't mind which way we go, just pushed an alternative in case we decide to go this way.

@lyakh

lyakh commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator Author

solved per #10084

@lyakh lyakh closed this Nov 17, 2025
@lyakh lyakh deleted the ctx branch November 17, 2025 11:26
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