Skip to content

zephyr: fix secondary core IPC completion - #4983

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
lyakh:zipc
Nov 12, 2021
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
lyakh:zipc

Conversation

@lyakh

@lyakh lyakh commented Nov 11, 2021

Copy link
Copy Markdown
Collaborator

Recently SOF IPC processing has been modified on multicore systems to let secondary cores reply to IPCs if the primary core has already left the IPC forwarding context. This hasn't been implemented for Zephyr which led to unanswered IPCs and IPC timeouts on the host. This patch lets secondary cores reply to IPCs under Zephyr.

@lyakh
lyakh requested a review from mrajwa as a code owner November 11, 2021 14:32
Recently SOF IPC processing has been modified on multicore systems
to let secondary cores reply to IPCs if the primary core has already
left the IPC forwarding context. This hasn't been implemented for
Zephyr which led to unanswered IPCs and IPC timeouts on the host.
This patch lets secondary cores reply to IPCs under Zephyr.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lgirdwood

Copy link
Copy Markdown
Member

@kkarask @wszypelt looks like CI got stuck. Can you restart. Thanks !

@lyakh

lyakh commented Nov 12, 2021

Copy link
Copy Markdown
Collaborator Author

@kkarask @wszypelt looks like CI got stuck. Can you restart. Thanks !

@lgirdwood apart from an addition of a missing header #include this PR only affects Zephyr and QB doesn't build Zephyr, so it cannot be affected

@wszypelt

wszypelt commented Nov 12, 2021 •

Copy link
Copy Markdown

@lgirdwood done :)

@lyakh

lyakh commented Nov 12, 2021

Copy link
Copy Markdown
Collaborator Author

Quickbuild seems to have a problem

@lgirdwood
lgirdwood merged commit 38ce524 into thesofproject:main Nov 12, 2021
@lyakh
lyakh deleted the zipc branch November 16, 2021 07:13
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