[Intel-wired-lan] [PATCH iwl-net v3 3/3] ice: Cache struct ice_hw pointer for split register reads
Rinitha, SX
sx.rinitha at intel.com
Mon Jul 13 06:38:56 UTC 2026
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of Sergey Temerkhanov
> Sent: 01 June 2026 14:50
> To: intel-wired-lan at lists.osuosl.org
> Cc: netdev at vger.kernel.org
> Subject: [Intel-wired-lan] [PATCH iwl-net v3 3/3] ice: Cache struct ice_hw pointer for split register reads
>
> Cache the primary ice_hw pointer to ensure consistency between calls (both parts of a value will be read from the same NAC).
> ice_get_primary_hw() will never return NULL, but during the ctrl_pf cleanup there may be a case when one call will return the pointer to the ctrl_pf->hw and the subsequent one - to the
> pf->hw which generally are not the same.
> Struct ice_hw is embedded in the struct ice_pf so it is protected by the same critical section - no additional synchronization is needed.
>
> Fixes: e2193f9f9ec9 ("ice: enable timesync operation on 2xNAC E825 devices")
> Signed-off-by: Sergey Temerkhanov <sergey.temerkhanov at intel.com>
> Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov at intel.com>
> Reviewed-by: Arkadiusz Kubalewski <arkadiusz.kubalewski at intel.com>
> Tested-by: Frederick Lawler <fred at cloudflare.com>
> ---
> drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
Tested-by: Rinitha S <sx.rinitha at intel.com> (A Contingent worker at Intel)
More information about the Intel-wired-lan
mailing list