Skip to content

dax: add an instance manager#10688

Draft
checkupup wants to merge 1 commit intothesofproject:mainfrom
checkupup:main_dax_user_priority
Draft

dax: add an instance manager#10688
checkupup wants to merge 1 commit intothesofproject:mainfrom
checkupup:main_dax_user_priority

Conversation

@checkupup
Copy link
Copy Markdown
Contributor

@checkupup checkupup commented Apr 8, 2026

When multiple DAX components are running simultaneously, instance resources may be insufficient. In this case, DAX instance manager ensures that higher-priority DAX components are always granted instance resources. For DAX components that cannot obtain instance resources in a timely manner, dax_process will operate in pass-through mode.

@sofci
Copy link
Copy Markdown
Collaborator

sofci commented Apr 8, 2026

Can one of the admins verify this patch?

reply test this please to run this test once

@lgirdwood
Copy link
Copy Markdown
Member

test this please

Copy link
Copy Markdown
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

@checkupup are you able to add more inline comments describing the flow/mechanism and log when we have to fallback to pass through mode. How would priority be set by users, i.e. using topology tuples would mean same dax/sof binaries could be used but with different priorities depending on target.

@checkupup checkupup force-pushed the main_dax_user_priority branch from 5af4619 to a50b365 Compare April 14, 2026 03:19
@checkupup
Copy link
Copy Markdown
Contributor Author

@checkupup are you able to add more inline comments describing the flow/mechanism and log when we have to fallback to pass through mode.

Sure, see my latest commit.

When multiple DAX components are running simultaneously, instance
resources may be insufficient. In this case, DAX instance manager
ensures that higher-priority DAX components are always granted
instance resources. For DAX components that cannot obtain instance
resources in a timely manner, `dax_process` will operate in
pass-through mode.

Signed-off-by: Jun Lai <jun.lai@dolby.com>
@checkupup checkupup force-pushed the main_dax_user_priority branch from a50b365 to 2b0800b Compare April 15, 2026 07:12
@checkupup checkupup changed the title dax: add a user priority mechanism dax: add an instance manager Apr 15, 2026
@checkupup
Copy link
Copy Markdown
Contributor Author

To maintain code readability, I have extracted the entire instance resource handling logic into the file dax_instance_manager.c

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