Le 02/02/17 à 18:54, David Miller a écrit : > From: Florian Fainelli <f.faine...@gmail.com> > Date: Tue, 31 Jan 2017 18:46:43 -0800 > >> From: Mao Wenan <maowe...@huawei.com> >> >> There is currently no reference count being held on the PHY driver, >> which makes it possible to remove the PHY driver module while the PHY >> state machine is running and polling the PHY. This could cause crashes >> similar to this one to show up: > ... >> Keep references on the PHY driver module right before we are going to >> utilize it in phy_attach_direct(), and conversely when we don't use it >> anymore in phy_detach(). >> >> Signed-off-by: Mao Wenan <maowe...@huawei.com> >> [florian: rebase, rework commit message] >> Signed-off-by: Florian Fainelli <f.faine...@gmail.com> > > I think grabbing the module reference is the only easy fix we > can do for now. > > Hot plugging PHYs and notifications and all of that business is > net-next material. > > Florian, do you need to respin this with the workqueue or whatever > suggestion Russell made?
That seems to deserve a separate fix of its own it seems, because it becomes tangential to this particular problem. -- Florian
