Backport 27468 ([rescue,opentitanlib,test] Add functional test for GetOwnerPage)#29320
Open
pamaury wants to merge 4 commits intolowRISC:masterfrom
Open
Backport 27468 ([rescue,opentitanlib,test] Add functional test for GetOwnerPage)#29320pamaury wants to merge 4 commits intolowRISC:masterfrom
GetOwnerPage)#29320pamaury wants to merge 4 commits intolowRISC:masterfrom
Conversation
pamaury
requested review from
jwnrt and
moidx
and removed request for
a team
pamaury
force-pushed
the
backport_27468
branch
from
80f67c9 to
4498e72
Compare
The rescue mode validation logic was updated to immediately return bad mode if a prior error occurred and the selected service exclusively supports either upload or download operations in the DFU implementation. This change ensures that `opentitantool` receives an error when sending a disallowed command for uni-directional services. However, for services supporting both upload and download, the `opentitantool` might not receive an error for a disallowed command depending on the owner configuration (e.g., when only upload or download is permitted). Signed-off-by: Anthony Chen <antchen@google.com> (cherry picked from commit a140bc0)
This adds the `empty` function in BootSvc for creating the Empty boot service request and updates the Rescue trait. Signed-off-by: Anthony Chen <antchen@google.com> (cherry picked from commit 54ff8f5)
This introduces a new test to verify rescue command disabilities. It uses the owner block binary if provided; otherwise, it captures the owner page 0 Base64 data from the UART console. The test analyzes the owner block to identify disallowed rescue commands and then attempts to execute each of them. This ensures that the device correctly enforces the configured restrictions. The testing approach differs slightly between rescue protocols: - XMODEM: The test directly examines the result of the rescue operation to confirm if a command was disallowed by the device. - DFU: Due to the potential for the rescue operation result to be inconclusive, the test primarily relies on analyzing device logs to determine if a command was rejected. Only when informative device logs are absent will the test fall back to checking the rescue operation result. Signed-off-by: Anthony Chen <antchen@google.com> (cherry picked from commit 0fe7506)
This introduces the test to verify the functionality of the `GetOwnerPage` rescue command. The `get_boot_log_test` verifies the retrieved OwnerPage against the owner block binary if provided; otherwise it verifies against the owner page 0 base64 data captured from the UART console. Signed-off-by: Anthony Chen <antchen@google.com> (cherry picked from commit f96ed8e)
pamaury
force-pushed
the
backport_27468
branch
from
4498e72 to
2036319
Compare
timothytrippel
approved these changes
Feb 19, 2026
cfrantz
approved these changes
Feb 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport #27468, depends on #29319