Skip to content

Main dax fixes - #11320

Draft
checkupup wants to merge 4 commits into
thesofproject:mainfrom
checkupup:main_dax_fixes
Draft

checkupup wants to merge 4 commits into
thesofproject:mainfrom
checkupup:main_dax_fixes

Conversation

@checkupup

Copy link
Copy Markdown
Contributor

A series of tiny fixes for DAX module:

  1. aadb750ea — dax: fix IPC3 configuration issues
    Recognizes the tuning-parameter blob configuration ID, rejects unsupported configuration IDs, and handles IPC3 zero-size SET_VALUE controls and volume-format conversion.

  2. b4b05d6ef — dax: isolate DAX parameter logging buffer
    Replaces a shared static formatting buffer with per-instance storage to prevent concurrent calls from overwriting one another’s output.

  3. 915233ae8 — dax: avoid DAX manager initialization race
    Initializes the instance manager from static data, removing the unguarded runtime initialization check and its obsolete initializer.

  4. a7015c3f7 — dax: remove invalid null-check
    Removes the dead null check in sof_dax_process().

Jun Lai added 4 commits October 19, 2026 18:10
Remove dead null-check in sof_dax_process().

Signed-off-by: Jun Lai <junlai@dolby.com>
Initialize the instance manager from static data instead of using an
unguarded runtime check. Remove the obsolete initializer and its call.

Signed-off-by: Jun Lai <junlai@dolby.com>
Replace the shared static parameter-formatting buffer with per-instance
storage. Pass that storage through get_params_str() to avoid concurrent
calls overwriting each other across module instances.

Signed-off-by: Jun Lai <junlai@dolby.com>
Recognize tuning-parameter blob configuration ID and reject
unsupported configuration IDs.

For IPC3, process zero-size SET_VALUE controls convert the
volume format.

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

sofci commented Oct 19, 2026

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

reply test this please to run this test once

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.

2 participants