From: Florian Fainelli <f.faine...@gmail.com> Date: Thu, 2 Feb 2017 19:47:43 -0800
> 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. Ok, patch applied then. Thanks!
