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

genericx86-64-ext: Make VMD driver built-in #720

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mtoman
Copy link
Contributor

@mtoman mtoman commented Feb 16, 2024

This is a driver that adds a PCI bridge that provides access to (mostly storage) devices "located in a different PCI domain".

Without the driver/bridge, such devices are invisible to the kernel, which may trick the migrator into thinking there are no valid flashing targets and start the in-memory migration.

This patch makes the driver built-in, which means the devices will get initialized before the migrator checks for potential targets.

@mtoman
Copy link
Contributor Author

mtoman commented Feb 18, 2024

@resin-jenkins retest this please

1 similar comment
@mtoman
Copy link
Contributor Author

mtoman commented Feb 23, 2024

@resin-jenkins retest this please

This is a driver that adds a PCI bridge that provides access
to (mostly storage) devices "located in a different PCI domain".

Without the driver/bridge, such devices are invisible to the kernel,
which may trick the migrator into thinking there are no valid flashing
targets and start the in-memory migration.

This patch makes the driver built-in, which means the devices will
get initialized before the migrator checks for potential targets.

Changelog-entry: Initialize VMD devices before the migrator runs
Signed-off-by: Michal Toman <[email protected]>
@mtoman
Copy link
Contributor Author

mtoman commented Mar 8, 2024

lgtm

@mtoman
Copy link
Contributor Author

mtoman commented Mar 8, 2024

@resin-jenkins retest this please

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.

1 participant