From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH net] net: phy: Fix lack of reference count on PHY driver Date: Wed, 1 Feb 2017 19:10:24 +0000 Message-ID: <20170201191024.GQ27312@n2100.armlinux.org.uk> References: <20170201102208.GM27312@n2100.armlinux.org.uk> <20170201105121.GA8191@n2100.armlinux.org.uk> <264c73a7-8294-23f0-88e9-1f27d558cbf8@gmail.com> <20170201.135938.186383946169055237.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: f.fainelli@gmail.com, netdev@vger.kernel.org, andrew@lunn.ch, maowenan@huawei.com To: David Miller Return-path: Received: from pandora.armlinux.org.uk ([78.32.30.218]:50824 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751093AbdBATKf (ORCPT ); Wed, 1 Feb 2017 14:10:35 -0500 Content-Disposition: inline In-Reply-To: <20170201.135938.186383946169055237.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Wed, Feb 01, 2017 at 01:59:38PM -0500, David Miller wrote: > From: Florian Fainelli > Date: Wed, 1 Feb 2017 10:55:46 -0800 > > > You are right, but there is still a fundamental problem IMHO in that you > > should not be able to rmmod a PHY driver as long as a network device is > > attached to the PHY, and if the PHY driver is attached from several > > different network devices, they should all have a way to prevent a PHY > > driver rmmod, each of them incrementing the driver refcount, which is > > what the patche from Maowan does here. > > It briefly occurred to me that we might want to be able to disconnect > PHYs to allow an unload using notifiers, the same way that when you > take a netdevice down we emit notifiers so that all of the references > to the netdevice can release themselves. > > I have no idea how well that would work, or whether it is valuable or > not. But it is another way to handle this. > > But that is a longer-term thing even if we want to go that way, and > thus grabbing the proper refs is the right things to do for now. It's something I'm effectively already doing as part of my phylink project for SFP support, since you can hot-unplug a copper SFP module, and the PHY on the copper SFP module will be gone. phylink, however, sits between the network driver and phylib, which isn't ideal. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.