Messages in this thread | | | From | Bjorn Helgaas <> | Date | Fri, 6 Apr 2012 10:07:03 -0600 | Subject | Re: [PATCH -v3 40/47] PCI: Add pci bus removal through /sys/.../pci_bus/.../remove |
| |
On Fri, Apr 6, 2012 at 10:01 AM, Yinghai Lu <yhlu.kernel@gmail.com> wrote: > On Fri, Apr 6, 2012 at 8:50 AM, Jiang Liu <liuj97@gmail.com> wrote: >> Hi Yinghai, >> I found many other drivers assume that a pci bus won't disappear if >> the corresponding PCI bridge device still exists. The sysfs interface proposed >> here breaks that assumption and may cause many access-after-free issues. >> So what's the purpose of this interface? Should we remove this interface or >> enhance other drivers to avoid invalid memory access issues?
Can you point out some of the specifics about drivers making this assumption? I'm not thrilled about the idea of removing a pci_bus while the upstream bridge pci_dev still exists either.
> ok, will make it only show up on root bus.
OK. I'm still interested in the specifics because I don't like the way the pci_bus is exposed, even inside the kernel. The bus itself is not an active entity, and we can't really do anything with it except by touching a device connected to it.
Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |