On 2/2/24 16:33, Stewart Hildebrand wrote: > --- > Changes in v13: > - hold off adding Roger's R-b tag even though it was provided on v12.2 > - use a wrapper construct to ease readability of odd-looking ASSERTs > - new placement of ASSERT in __pci_enable_msix(), __pci_enable_msi(), > and pci_enable_msi(). Rearrange/add pdev NULL check. > - expand commit description with details about using either > pcidevs_lock() or d->pci_lock > > Changes in v12.2: > - drop Roger's R-b > - drop both locks on error paths in vpci_msix_arch_print() > - add another ASSERT in vpci_msix_arch_print(), to enforce the > expectation both locks are held before calling vpci_msix_arch_print() > - move pdev_done label in vpci_dump_msi() > - update comments in vpci_dump_msi() to say locks (plural)
Here's a patchew link to show just the diff-of-diff from v12.2 (where Roger had given a R-b) to v13. https://patchew.org/Xen/[email protected]/diff/[email protected]/
