Re: [PATCH net-next 3/3] net: xgene: avoid bogus maybe-uninitialized warning

From: David Miller
Date: Thu Dec 08 2016 - 21:31:23 EST


From: Arnd Bergmann <arnd@xxxxxxxx>
Date: Thu, 8 Dec 2016 22:57:05 +0100

> In some configurations, gcc cannot trace the state of variables
> across a spin_unlock() barrier, leading to a warning about
> correct code:
>
> xgene_enet_main.c: In function 'xgene_enet_start_xmit':
> ../../../phy/mdio-xgene.h:112:14: error: 'mss_index' may be used uninitialized in this function [-Werror=maybe-uninitialized]
>
> Here we can trivially move the assignment before that spin_unlock,
> which reliably avoids the warning.
>
> Fixes: e3978673f514 ("drivers: net: xgene: Fix MSS programming")
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>

Applied.