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

pci_device: Add bridge reset when a device is unplugged #3143

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

michael-adler
Copy link
Member

Hot unplug/plug alone does not trigger a reset. Add warm reset as a side effect of unplugging a card after it has been removed.

Signed-off-by: Michael Adler <michael.adler@intel.com>
@michael-adler michael-adler self-assigned this Dec 16, 2024
@michael-adler michael-adler requested a review from a team as a code owner December 16, 2024 20:47
@michael-adler michael-adler merged commit c97c3f2 into OFS:master Dec 16, 2024
17 checks passed
@michael-adler michael-adler deleted the mcadler/unplug_reset branch December 16, 2024 22:11
michael-adler added a commit to michael-adler/opae-sdk that referenced this pull request Dec 19, 2024
- Wait for data link active after triggering a bridge's secondary device reset.
- Move reset to a separate command that unloads drivers, then devices, triggers
  warm reset on FPGAs, and finally rescans and clears errors.

Fixes: c97c3f2 ("pci_device: Add bridge reset when a device is unplugged (OFS#3143)")

Signed-off-by: Michael Adler <michael.adler@intel.com>
michael-adler added a commit that referenced this pull request Dec 19, 2024
- Wait for data link active after triggering a bridge's secondary device reset.
- Move reset to a separate command that unloads drivers, then devices, triggers
  warm reset on FPGAs, and finally rescans and clears errors.

Fixes: c97c3f2 ("pci_device: Add bridge reset when a device is unplugged (#3143)")

Signed-off-by: Michael Adler <michael.adler@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants