On Fri, Oct 21, 2016 at 06:58:10PM +0200, Lukas Wunner wrote: > On Tue, Sep 27, 2016 at 04:23:32PM -0400, Keith Busch wrote: > I note that you've removed the change to pci_device_is_present() > completely from v3 of this patch, however only a portion of the > change was problematic. This portion would have been okay: > > > + if (pdev->is_removed) > + return false; > > return pci_bus_read_dev_vendor_id(pdev->bus, pdev->devfn, &v, 0); > > > Up to you if you want to include this in the next iteration or not. I was thinking it didn't matter since patch 3/5 has config access return -ENODEV on a removed pci_dev, but pci_bus_read_dev_vendor_id doesn't use the pci_dev. I'll add it back in. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
