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? Thanks.
