Skip to content

Cherry-pick to master: [sw,keymgr] trap if keymgr is idle after starting op #28129#28477

Merged
timothytrippel merged 1 commit intolowRISC:masterfrom
sasdf:mgT35b51071
Dec 3, 2025
Merged

Cherry-pick to master: [sw,keymgr] trap if keymgr is idle after starting op #28129#28477
timothytrippel merged 1 commit intolowRISC:masterfrom
sasdf:mgT35b51071

Conversation

@sasdf
Copy link
Contributor

@sasdf sasdf commented Oct 14, 2025

This is manual cherrypick of:


Original commit message:

The keymgr_wait_until_done() function should be only called after an operation has been started by the caller.

This commit removes the IDLE status check from the switch-case in keymgr_wait_until_done() to enforce this invariant. If the keymgr is found to be IDLE the execution will now proceed to the default error-handling case and trigger a trap.

@sasdf sasdf marked this pull request as ready for review October 14, 2025 14:38
@sasdf sasdf requested a review from a team as a code owner October 14, 2025 14:38
@timothytrippel
Copy link
Contributor

Rebase?

The `keymgr_wait_until_done()` function should be only called after
an operation has been started by the caller.

This commit removes the `IDLE` status check from the switch-case in
`keymgr_wait_until_done()` to enforce this invariant. If the keymgr
is found to be `IDLE` the execution will now proceed to the default
error-handling case and trigger a trap.

Change-Id: I35b5107174796e9a52eff7ab33bdc2260453a73b
Signed-off-by: Yi-Hsuan Deng <[email protected]>
(cherry picked from commit abac206)
@timothytrippel timothytrippel added this pull request to the merge queue Dec 3, 2025
Merged via the queue into lowRISC:master with commit 987b322 Dec 3, 2025
47 checks passed
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