[PATCH net v2 0/2] Fix 'ethtool --show-eee' during initial stage
Russell King (Oracle)
linux at armlinux.org.uk
Sat Nov 16 09:44:16 PST 2024
On Sat, Nov 16, 2024 at 06:41:13PM +0100, Heiner Kallweit wrote:
> On 16.11.2024 16:34, Russell King (Oracle) wrote:
> > Hmm, don't we want to do this under phydev->lock, because network
> > drivers and phylib may be reading from phydev->eee_cfg? If we
> > update it outside the lock, and then revert, there's a chance that
> > the phylib state machine / network driver may see the changes
> > which then get reverted on failure, potentially leading to
> > inconsistent state.
>
> Good point, then the patch would look like this.
> BTW: Saw that Jakub applied your patch already.
Yes indeed, so I hope Jakub will apply your follow-up patch soon!
This LGTM.
Reviewed-by: Russell King (Oracle) <rmk+kernel at armlinux.org.uk>
Thanks!
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
More information about the linux-arm-kernel
mailing list