Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport v3.6-branch] intel_adsp/ace: power: Lock interruption when power gate fails #70127

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

zephyrbot
Copy link
Collaborator

@zephyrbot zephyrbot commented Mar 12, 2024

Backport 07426a8 from #68493.

Fixes: #70697

In case the core is not power gated, waiti will restore intlevel. In
this case we lock interruption after it.

In the bug scenario, the host starts streaming and via SOF APIs, keeps a
lock to prevent Zephyr from entering PM_STATE_RUNTIME_IDLE. During the
test case, host removes this block and core0 is allowed to enter IDLE
state.

When core0 enters power gated state, interrrupts are left enabled (so
the core can be woken up when something happens). This leaves a race
where suitably timed interrupt will actually block entry to power gated
state and k_cpu_idle() in power_gate_entry() will return. This is rare,
but happens often enough that the relatively short test plan run on SOF
pull-requests will trigger this case.

Fixes #69807

Signed-off-by: Flavio Ceolin <[email protected]>
Signed-off-by: Anas Nashif <[email protected]>
(cherry picked from commit 07426a8)
@henrikbrixandersen
Copy link
Member

@ceolin This needs a bug report in order to get backported.

@henrikbrixandersen henrikbrixandersen added this to the v3.6.1 milestone Mar 15, 2024
@ceolin
Copy link
Member

ceolin commented Mar 25, 2024

@ceolin This needs a bug report in order to get backported.

done

@henrikbrixandersen henrikbrixandersen merged commit b6391ef into v3.6-branch Mar 29, 2024
20 checks passed
@nashif nashif deleted the backport-68493-to-v3.6-branch branch November 16, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport Backport PR and backport failure issues platform: Intel ADSP Intel Audio platforms
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

intel_adsp/ace: power: Lock interruption when power gate fails - Backport 3.6
5 participants