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

initial migration state not specified #779

Open
tmakatos opened this issue Sep 4, 2023 · 2 comments
Open

initial migration state not specified #779

tmakatos opened this issue Sep 4, 2023 · 2 comments
Assignees

Comments

@tmakatos
Copy link
Member

tmakatos commented Sep 4, 2023

VFIO uapi doesn't seem to specify the initial migration state of the device. Is it not required? Does the kernel code expect some speicfic state? In any case, we should check the code and clarify that in the vfio-user spec.

@w-henderson
Copy link
Contributor

w-henderson commented Sep 4, 2023

QEMU starts in RUNNING: https://elixir.bootlin.com/qemu/v8.0.4/source/hw/vfio/migration.c#L603

All I can find in the kernel is when the device resets, it gets set to RUNNING: https://elixir.bootlin.com/linux/v6.5.1/source/drivers/vfio/pci/mlx5/main.c#L1151

@tmakatos
Copy link
Member Author

tmakatos commented Sep 4, 2023

I can't find anything in the kernel either, maybe it doesn't matter, perhaps it's worth asking on lkml (make sure you CC the right people)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants