-
Notifications
You must be signed in to change notification settings - Fork 72
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: Should update ISR related description about MSI-X #145
Comments
Thank you for your contribution. Please follow the process outlined in https://github.com/oasis-tcs/virtio-spec#providing-feedback; in particular, all proposed changes must be posted to the mailing list. Feel free to re-use this issue for requesting a vote later on (but please edit the description if you do so.) |
Thx, I have sent a patch: https://lists.oasis-open.org/archives/virtio-comment/202209/msg00001.html And I have a question. What do you mean "requesting a vote"? Thanks. |
See https://github.com/oasis-tcs/virtio-spec#use-of-github-issues -- basically, if you managed to get consensus for your proposed change, you need to request that the TC votes on it before it can be integrated. |
Got it, thanks. |
In commit 6a83c1d ("pci: set ISR bit on config change with MSI-X"),
device is required to set ISR bit on config change even when MSI-X enabled.
We should update other ISR related description for above commit. Here I have
a RFC change:
https://lists.oasis-open.org/archives/virtio-comment/202209/msg00001.html
And the latest change is v2:
https://lists.oasis-open.org/archives/virtio-comment/202209/msg00024.html
Thanks,
Keqian
The text was updated successfully, but these errors were encountered: