sfc: fix crash when reading stats while NIC is resetting
authorEdward Cree <ecree.xilinx@gmail.com>
Fri, 23 Jun 2023 14:34:48 +0000 (15:34 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Jun 2023 08:28:27 +0000 (09:28 +0100)
commitd1b355438b8325a486f087e506d412c4e852f37b
tree38e47188f2d28db48204cfad0e41a1bfe2ff1af8
parentf1bc9fc4a06de0108e0dca2a9a7e99ba1fc632f9
sfc: fix crash when reading stats while NIC is resetting

efx_net_stats() (.ndo_get_stats64) can be called during an ethtool
 selftest, during which time nic_data->mc_stats is NULL as the NIC has
 been fini'd.  In this case do not attempt to fetch the latest stats
 from the hardware, else we will crash on a NULL dereference:
    BUG: kernel NULL pointer dereference, address: 0000000000000038
    RIP efx_nic_update_stats
    abridged calltrace:
    efx_ef10_update_stats_pf
    efx_net_stats
    dev_get_stats
    dev_seq_printf_stats
Skipping the read is safe, we will simply give out stale stats.
To ensure that the free in efx_ef10_fini_nic() does not race against
 efx_ef10_update_stats_pf(), which could cause a TOCTTOU bug, take the
 efx->stats_lock in fini_nic (it is already held across update_stats).

Fixes: d3142c193dca ("sfc: refactor EF10 stats handling")
Reviewed-by: Pieter Jansen van Vuuren <pieter.jansen-van-vuuren@amd.com>
Signed-off-by: Edward Cree <ecree.xilinx@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sfc/ef10.c