[PATCH net v2 0/2] Fix 'ethtool --show-eee' during initial stage
Heiner Kallweit
hkallweit1 at gmail.com
Sat Nov 16 10:06:02 PST 2024
On 16.11.2024 18:44, Russell King (Oracle) wrote:
> 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.
>
OK, then I'll submit the followup.
> Reviewed-by: Russell King (Oracle) <rmk+kernel at armlinux.org.uk>
>
> Thanks!
>
More information about the linux-arm-kernel
mailing list