On Wed, 2015-04-08 at 23:26 -0500, Bjorn Helgaas wrote: > I'm not planning to review this until after the merge window opens, > but I took a quick glance, and I agree with Ben. I don't want to add > a new IORESOURCE_ flag. I think a pci_resource_compatible() helper is > a great idea. So the new resource flag was handy here still regardless of the implementation choice because otherwise, we have to do the whole tree walk to check for "PCI Express only path". *But*, this is a property of the device as a whole, not of the resource, so we could instead have a pci_dev flag established at probe time that indicates that the path to a given device is PCIe only. The cool thing is that can be very easily done by simply propagating from the parent. If the parent has it and the device is PCIe, then set it, the only break would happen on PCIe to PCI bridges, which is exactly what we are after. That way, you avoid the special resource flag alltogether. > I am absolutely not in favor of "minimally intrusive" as a goal. > "Minimally intrusive" sounds good but it is often used to justify > clever hacks which end up being an anti-maintainer strategy in the > long term. "Maximum readability" is what I'm looking for. Yup, I agree. The code is too hard to understand already which makes it bug prone in the long run. Cheers, Ben. -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html