Skip to content

dax: fix race condition on the tuning buffer - #11056

Open
checkupup wants to merge 2 commits into
thesofproject:mainfrom
checkupup:main_dax_fixes
Open

dax: fix race condition on the tuning buffer#11056
checkupup wants to merge 2 commits into
thesofproject:mainfrom
checkupup:main_dax_fixes

Conversation

@checkupup

Copy link
Copy Markdown
Contributor

No description provided.

It is necessary cause p_dax is used in dax.c now.

Signed-off-by: Jun Lai <jun.lai@dolby.com>
@sofci

sofci commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

reply test this please to run this test once

Comment thread src/audio/module_adapter/module/dolby/dax.c Outdated
@checkupup
checkupup force-pushed the main_dax_fixes branch 2 times, most recently from 0749fcc to 95f5201 Compare August 5, 2026 06:29
@checkupup
checkupup marked this pull request as ready for review August 6, 2026 06:35
Comment thread src/audio/module_adapter/module/dolby/dax_mock.c
Comment thread src/audio/module_adapter/module/dolby/dax.c Outdated
Comment thread src/audio/module_adapter/module/dolby/dax.c Outdated
Comment thread src/audio/module_adapter/module/dolby/dax.c
While the tuning buffer is being updated, it may also
be being used simultaneously in the process thread.

Signed-off-by: Jun Lai <jun.lai@dolby.com>
@lgirdwood

Copy link
Copy Markdown
Member

@lrudyX good to merge, I'm pretty sure not tested by CI.

@lrudyX

lrudyX commented Aug 11, 2026

Copy link
Copy Markdown

@lrudyX good to merge, I'm pretty sure not tested by CI.

No, It was not build and run with any test. (Scheduler ignores this PR until "test this please" command) The CI pipeline is stopped at the moment due to cmake upgrade.

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.

5 participants