>>> Roger Pau Monne <roger....@citrix.com> 06/30/17 5:02 PM >>> >--- a/xen/drivers/vpci/header.c >+++ b/xen/drivers/vpci/header.c >@@ -459,7 +459,7 @@ static int vpci_init_bars(struct pci_dev *pdev) >return 0; >} > >-REGISTER_VPCI_INIT(vpci_init_bars); >+REGISTER_VPCI_INIT(vpci_init_bars, VPCI_PRIORITY_LOW); Why "LOW"? I'd expect the BARs to possibly have further dependents, so their init should be somewhere in the middle.
>--- a/xen/drivers/vpci/msi.c >+++ b/xen/drivers/vpci/msi.c >@@ -290,7 +290,7 @@ static int vpci_init_msi(struct pci_dev *pdev) >return ret; >} > >-REGISTER_VPCI_INIT(vpci_init_msi); >+REGISTER_VPCI_INIT(vpci_init_msi, VPCI_PRIORITY_LOW); Whereas these indeed are unlikely to have further dependents. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
