> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of
> Pavan Kumar Linga
> Sent: Friday, October 25, 2024 11:39 AM
> To: intel-wired-lan@xxxxxxxxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx; horms@xxxxxxxxxx; Linga, Pavan Kumar
> <pavan.kumar.linga@xxxxxxxxx>; stable@xxxxxxxxxxxxxxx; Singh, Tarun K
> <tarun.k.singh@xxxxxxxxx>
> Subject: [Intel-wired-lan] [PATCH iwl-net v2 1/2] idpf: avoid vport access in
> idpf_get_link_ksettings
>
> When the device control plane is removed or the platform
> running device control plane is rebooted, a reset is detected
> on the driver. On driver reset, it releases the resources and
> waits for the reset to complete. If the reset fails, it takes
> the error path and releases the vport lock. At this time if the
> monitoring tools tries to access link settings, it call traces
> for accessing released vport pointer.
>
> To avoid it, move link_speed_mbps to netdev_priv structure
> which removes the dependency on vport pointer and the vport lock
> in idpf_get_link_ksettings. Also use netif_carrier_ok()
> to check the link status and adjust the offsetof to use link_up
> instead of link_speed_mbps.
>
> Fixes: 02cbfba1add5 ("idpf: add ethtool callbacks")
> Cc: stable@xxxxxxxxxxxxxxx # 6.7+
> Reviewed-by: Tarun K Singh <tarun.k.singh@xxxxxxxxx>
> Signed-off-by: Pavan Kumar Linga <pavan.kumar.linga@xxxxxxxxx>
> ---
> drivers/net/ethernet/intel/idpf/idpf.h | 4 ++--
> drivers/net/ethernet/intel/idpf/idpf_ethtool.c | 11 +++--------
> drivers/net/ethernet/intel/idpf/idpf_lib.c | 4 ++--
> drivers/net/ethernet/intel/idpf/idpf_virtchnl.c | 2 +-
> 4 files changed, 8 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/idpf/idpf.h
> b/drivers/net/ethernet/intel/idpf/idpf.h
> index 2c31ad87587a..66544faab710 100644
> --- a/drivers/net/ethernet/intel/idpf/idpf.h
> +++ b/drivers/net/ethernet/intel/idpf/idpf.h
Tested-by: Krishneil Singh <krishneil.k.singh@xxxxxxxxx>
