Re: [PATCH net-next 3/4] net: dsa: setup master ethtool after dsa_ptr
From: Florian Fainelli
Date: Tue Sep 19 2017 - 13:50:03 EST
On 09/19/2017 08:56 AM, Vivien Didelot wrote:
> DSA overrides the master's ethtool ops so that we can inject its CPU
> port's statistics. Because of that, we need to setup the ethtool ops
> after the master's dsa_ptr pointer has been assigned, not before.
Yes, good point, technically this is a bugfix, but since we have changed
this quite often and the race is tiny, I am not positive we could a)
trigger this in real life, and b) provide a proper Fixes tag.
>
> This patch setups the ethtool ops after dsa_ptr is assigned, and
> restores them before it gets cleared.
>
> Signed-off-by: Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
--
Florian