Messages in this thread | | | From | (Eric W. Biederman) | Date | Tue, 24 Feb 2009 20:22:44 -0800 | Subject | Re: [PATCH] pcie_portdriver: FIX: pcie_port_device_remove (take 2) |
| |
Jesse Barnes <jbarnes@virtuousgeek.org> writes:
> On Friday, February 20, 2009 8:16:07 pm Eric W. Biederman wrote: >> pcie_port_device_remove currently calls the remove method of port >> drivers twice. Ouch! >> >> We are calling device_for_each_child multiple times for no apparent >> reason. >> >> So make it simple. Place put_device and device_unregister into >> remove_iter, and throw out the rest. Only call device_for_each_child >> once. >> >> The code is simpler and actually works! >> >> Changelog: >> v2 rebase against the linux-next tree so I don't conflict with Rafael's >> irq work, and remove the irq handling cleanups as Rafael's patch already >> made them. > > Thanks for rebasing; it's queued up in my linux-next branch now. If we really > need this in the stable kernel it'll be a bit painful, since some of these > structures have changed around a bit...
I don't think it is necessary for stable. The pciehp still needs more work, and if anyone of the other port drivers was being hotplugged I expect someone would have noticed the bug and fixed it before now.
Eric
| |