Message ID | 20110926162552.GA4175@phenom.oracle.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
> -----Original Message----- > From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@oracle.com] > Sent: Monday, September 26, 2011 9:26 AM > To: Kirsher, Jeffrey T; davem@davemloft.net > Cc: Rose, Gregory V; Ian Campbell; davem@davemloft.net; Jesse Barnes; > netdev@vger.kernel.org; gospo@redhat.com; linux-pci@vger.kernel.org > Subject: Re: [net-next 1/8] pci: Add flag indicating device has been > assigned by KVM [and Xen] > > On Fri, Sep 23, 2011 at 09:41:39PM -0700, Jeff Kirsher wrote: > > On Fri, 2011-09-23 at 12:05 -0700, Konrad Rzeszutek Wilk wrote: > > > > > > Version 2 of this patch with the modified comment was just > posted to > > > > > netdev. Ian Campbell had mentioned that the Xen pciback driver > should be > > > > > modified to use this flag but I don't believe any patches for it > have been > > > > > created yet. > > > > > > > > > > http://marc.info/?i=20110729165446.GA6731@dumpdata.com > .. > Any plans to support bnx4 and igb with this feature? I'm updating the igb driver to support this feature right now. I would hesitate to get into other drivers and muck with them because the changes to support this feature are not trivial and also involve policy decisions that I'd prefer to leave up to the folks who maintain those drivers. Plus I have no other vendor hardware available to test the changes. > Any ETA? I should have the igb driver work done in the next week or so. - Greg -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Any plans to support bnx4 and igb with this feature? > > I'm updating the igb driver to support this feature right now. I would hesitate to get into other drivers and muck with them because the changes to support this feature are not trivial and also involve policy decisions that I'd prefer to leave up to the folks who maintain those drivers. Plus I have no other vendor hardware available to test the changes. > Sounds fair. Did you CC them? > > Any ETA? > > I should have the igb driver work done in the next week or so. Ok, please CC me on the code when you post it. > > - Greg -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> -----Original Message----- > From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@oracle.com] > Sent: Monday, September 26, 2011 9:43 AM > To: Rose, Gregory V > Cc: Kirsher, Jeffrey T; davem@davemloft.net; Ian Campbell; Jesse Barnes; > netdev@vger.kernel.org; gospo@redhat.com; linux-pci@vger.kernel.org > Subject: Re: [net-next 1/8] pci: Add flag indicating device has been > assigned by KVM [and Xen] > > > > Any plans to support bnx4 and igb with this feature? > > > > I'm updating the igb driver to support this feature right now. I would > hesitate to get into other drivers and muck with them because the changes > to support this feature are not trivial and also involve policy decisions > that I'd prefer to leave up to the folks who maintain those drivers. Plus > I have no other vendor hardware available to test the changes. > > > > Sounds fair. Did you CC them? I'll grep through the drivers and find who calls pci_disable_sriov() and send the respective driver maintainers a notification letting them know of the patches. > > > > Any ETA? > > > > I should have the igb driver work done in the next week or so. > > Ok, please CC me on the code when you post it. Will do. - Greg -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/xen/xen-pciback/xenbus.c b/drivers/xen/xen-pciback/xenbus.c index 978d2c6..18db31f 100644 --- a/drivers/xen/xen-pciback/xenbus.c +++ b/drivers/xen/xen-pciback/xenbus.c @@ -249,6 +249,7 @@ static int xen_pcibk_export_device(struct xen_pcibk_device *pdev, goto out; dev_dbg(&dev->dev, "registering for %d\n", pdev->xdev->otherend_id); + dev->dev_flags |= PCI_DEV_FLAGS_ASSIGNED; if (xen_register_device_domain_owner(dev, pdev->xdev->otherend_id) != 0) { dev_err(&dev->dev, "device has been assigned to another " \ @@ -288,6 +289,7 @@ static int xen_pcibk_remove_device(struct xen_pcibk_device *pdev, } dev_dbg(&dev->dev, "unregistering for %d\n", pdev->xdev->otherend_id); + dev->dev_flags &= ~PCI_DEV_FLAGS_ASSIGNED; xen_unregister_device_domain_owner(dev); xen_pcibk_release_pci_dev(pdev, dev);