Skip to content

ipc4: set_dx: ignoring logger status notifications - #8029

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
tmleman:topic/upstream/pr/issue/7969/fix/v1/ipc_list_check
Aug 14, 2023
Merged

kv2019i merged 1 commit into
thesofproject:mainfrom
tmleman:topic/upstream/pr/issue/7969/fix/v1/ipc_list_check

Conversation

@tmleman

@tmleman tmleman commented Aug 11, 2023

Copy link
Copy Markdown
Contributor

Before HOST can request FW to enter D3 state, has to stop all pipelines. If it's done properly all possible source of IPC messages from FW to HOST are disabled.

The only exception from this is logger. If logs are enabled FW will continue to produce new traces and eventually trigger threshold notification.

This notification is not critical so we can skip it during D3 entry. The message will remain in the internal list and will be sent when the DSP wakes up.

Before HOST can request FW to enter D3 state, has to stop all pipelines.
If it's done properly all possible source of IPC messages from FW to
HOST are disabled.

The only exception from this is logger. If logs are enabled FW will
continue to produce new traces and eventually trigger threshold
notification.

This notification is not critical so we can skip it during D3 entry. The
message will remain in the internal list and will be sent when the DSP
wakes up.

Signed-off-by: Tomasz Leman <tomasz.m.leman@intel.com>

@kv2019i kv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @tmleman !

@mengdonglin
mengdonglin requested a review from RanderWang August 14, 2023 03:21
@kv2019i

kv2019i commented Aug 14, 2023

Copy link
Copy Markdown
Collaborator

We seem to have a another build issue with Zephyr main that needs to be followed-up -> not related to this PR though, so can ignore the Zephyr main fails in CI. We hit the known #7759 (under debug, unrelated) and alsabat fail on the main-ace plan, but this looks to be a DUT issue. Proceeding with merge.

@kv2019i
kv2019i merged commit 49199a6 into thesofproject:main Aug 14, 2023
@tmleman
tmleman deleted the topic/upstream/pr/issue/7969/fix/v1/ipc_list_check branch December 17, 2025 12:14
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