[Intel-wired-lan] [PATCH v2 iwl-next 0/6] ice: fix timestamping in reset process

[Pucha, HimasekharX Reddy]

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of Karol Kolacinski
> Sent: Monday, December 11, 2023 8:32 PM
> To: intel-wired-lan at lists.osuosl.org
> Cc: netdev at vger.kernel.org; Kolacinski, Karol <karol.kolacinski at intel.com>; Nguyen, Anthony L <anthony.l.nguyen at intel.com>; Brandeburg, Jesse <jesse.brandeburg at intel.com>
> Subject: [Intel-wired-lan] [PATCH v2 iwl-next 0/6] ice: fix timestamping in reset process
>
> PTP reset process has multiple places where timestamping can end up in
> an incorrect state.
>
> This series introduces a proper state machine for PTP and refactors
> a large part of the code to ensure that timestamping does not break.
>
> Jacob Keller (5):
>   ice: pass reset type to PTP reset functions
>   ice: rename verify_cached to has_ready_bitmap
>   ice: rename ice_ptp_tx_cfg_intr
>   ice: factor out ice_ptp_rebuild_owner()
>   ice: stop destroying and reinitalizing Tx tracker during reset
>
>  Karol Kolacinski (1):
>   ice: introduce PTP state machine
>
> V1 -> V2: rebased the series and dropped already merged patches
>
>  drivers/net/ethernet/intel/ice/ice.h         |   1 -
>  drivers/net/ethernet/intel/ice/ice_ethtool.c |   2 +-
>  drivers/net/ethernet/intel/ice/ice_main.c    |   4 +-
>  drivers/net/ethernet/intel/ice/ice_ptp.c     | 226 +++++++++++--------
>  drivers/net/ethernet/intel/ice/ice_ptp.h     |  34 ++-
>  5 files changed, 163 insertions(+), 104 deletions(-)
>
>
> base-commit: 9615a96563f03aef04320cb9b4c33f7bdabac5af
> -- 
> 2.40.1

With ice: fix timestamping in reset process patch series, PTP time sync not happening. Observing missing timestamp on transmitted sync continuously at master side. Without patch series PTP working fine.

Observing below logs when running ptp4l as a master:

#./ptp4l -i eth8 -f configs/default.cfg -m

ptp4l[13259.050]: selected /dev/ptp7 as PTP clock
ptp4l[13259.065]: port 1: INITIALIZING to LISTENING on INIT_COMPLETE
ptp4l[13259.065]: port 0: INITIALIZING to LISTENING on INIT_COMPLETE
ptp4l[13265.651]: port 1: LISTENING to MASTER on ANNOUNCE_RECEIPT_TIMEOUT_EXPIRES
ptp4l[13265.651]: selected local clock 6805ca.fffe.787776 as best master
ptp4l[13265.651]: port 1: assuming the grand master role
ptp4l[13266.652]: missing timestamp on transmitted sync
ptp4l[13266.652]: port 1: MASTER to FAULTY on FAULT_DETECTED (FT_UNSPECIFIED)
ptp4l[13282.654]: port 1: FAULTY to LISTENING on INIT_COMPLETE
ptp4l[13289.267]: port 1: LISTENING to MASTER on ANNOUNCE_RECEIPT_TIMEOUT_EXPIRES
ptp4l[13289.267]: selected local clock 6805ca.fffe.787776 as best master
ptp4l[13289.267]: port 1: assuming the grand master role
ptp4l[13290.268]: missing timestamp on transmitted sync
ptp4l[13290.268]: port 1: MASTER to FAULTY on FAULT_DETECTED (FT_UNSPECIFIED)
ptp4l[13306.269]: port 1: FAULTY to LISTENING on INIT_COMPLETE
ptp4l[13313.261]: port 1: LISTENING to MASTER on ANNOUNCE_RECEIPT_TIMEOUT_EXPIRES
ptp4l[13313.261]: selected local clock 6805ca.fffe.787776 as best master
ptp4l[13313.261]: port 1: assuming the grand master role
ptp4l[13314.262]: missing timestamp on transmitted sync
ptp4l[13314.262]: port 1: MASTER to FAULTY on FAULT_DETECTED (FT_UNSPECIFIED)