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

DRAM: fast_get() cache-handling improvements #9817

Merged
merged 3 commits into from
Feb 11, 2025
Merged

Conversation

lyakh
Copy link
Collaborator

@lyakh lyakh commented Feb 7, 2025

Use uncached aliases for potentially shared objects, synchronise cache properly when sharing cached access

lyakh added 3 commits February 7, 2025 10:16
When multiple users access the same DRAM data via the fast_get() API,
they can run on different cores. The data is immutable, but when
initially pointing to it, cache has to be invalidated.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
fast_get entry list is global, it can be accessed from any core,
allocate it uncached.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
Invert a condition to eliminate a single goto.

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

lyakh commented Feb 7, 2025

SOFCI TEST

@lyakh
Copy link
Collaborator Author

lyakh commented Feb 11, 2025

CI: 1 alsabat failure on HDA MTL https://sof-ci.01.org/sofpr/PR9817/build10578/devicetest/index.html?model=MTLP_RVP_HDA&testcase=check-alsabat-headset-capture-997, TGL nocodec not tested, but the LNL failure puzzles me... https://sof-ci.01.org/sofpr/PR9817/build10577/devicetest/index.html but it absolutely cannot be caused by this PR. This PR changes behaviour of fast_get() and fast_put() functions only and these functions so far are only used by the SRC component, which is only used during audio streaming, so there's no way it can lead to module loading failing. I'll re-run jenkins

@lyakh
Copy link
Collaborator Author

lyakh commented Feb 11, 2025

SOFCI TEST

@lyakh
Copy link
Collaborator Author

lyakh commented Feb 11, 2025

CI: now ACE (MTL), cAVE and LNL are all green (except for a single untested system in each), and PTL is just not fully configured yet in CI

@kv2019i kv2019i merged commit b7f3871 into thesofproject:main Feb 11, 2025
44 of 49 checks passed
@lyakh lyakh deleted the dram branch February 11, 2025 11:58
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