Repository navigation
fix(pcie): Save and restore selected config state - #619
Merged
chetan-rathore merged 2 commits intoOct 8, 2026
Merged
chetan-rathore merged 2 commits into
chetan-rathore merged 2 commits into
Conversation
- Replace the full 4KB config space save/restore in PCIe FLR tests with targeted save/restore of standard header and selected PCIe capability state. - Save and restore BARs, command register bits, PCIe control registers, and supported capability state including AER, DPC, ACS, ATS, PASID, SR-IOV, LTR, PTM, and TPH. This avoids blindly writing back the entire config space after FLR while still restoring state needed for continued test execution. - Also share the saved-state definitions through acs_pcie.h and reuse the common helper flow in both p035 and p063. Signed-off-by: Sujana M <sujana.murali@arm.com> Change-Id: Ice00c233bf96b43741d7d1cf0a248095bcc1966a
- Replace full 4KB config-space save/restore with targeted save/restore of standard header and selected PCIe capability state. - Save and restore BARs, command register bits, PCIe control registers and supported capability state including AER, DPC, ACS, ATS, PASID, SR-IOV, LTR, PTM, and TPH. This avoids blindly writing back the entire config space while still restoring state needed for continued test execution. - Use the common selected config-state flow in FLR tests p035 and p063. Also use it in DPC exerciser tests by saving all BDFs under the affected Root Port before Secondary Bus Reset and restoring them after DPC recovery. This covers endpoints behind switches without restoring the full 4KB config space. Signed-off-by: Sujana M <sujana.murali@arm.com> Change-Id: I508e717db6ade169118c2e71cf80335fb9aed189
chetan-rathore
merged commit Oct 8, 2026
f6869ad
into
ARM-software:release/sbsa/v8.0.3
29 checks passed
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
2 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.
Replace full 4KB config-space save/restore with targeted save/restore
of standard header and selected PCIe capability state.
Save and restore BARs, command register bits, PCIe control registers
and supported capability state including AER, DPC, ACS, ATS, PASID,
SR-IOV, LTR, PTM, and TPH. This avoids blindly writing back the entire
config space while still restoring state needed for continued test
execution.
Use the common selected config-state flow in FLR tests p035 and p063.
Also use it in DPC exerciser tests by saving all BDFs under the
affected Root Port before Secondary Bus Reset and restoring them after
DPC recovery. This covers endpoints behind switches without restoring
the full 4KB config space.