| Message ID | 20220831101631.13585-1-ihuguet@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | sfc: add support for PTP over IPv6 and 802.3 | expand |
On Wed, Aug 31, 2022 at 12:16:28PM +0200, Íñigo Huguet wrote: > Most recent cards (8000 series and newer) had enough hardware support > for this, but it was not enabled in the driver. The transmission of PTP > packets over these protocols was already added in commit bd4a2697e5e2 > ("sfc: use hardware tx timestamps for more than PTP"), but receiving > them was already unsupported so synchronization didn't happen. > > These patches add support for timestamping received packets over > IPv6/UPD and IEEE802.3. > > v2: fixed weird indentation in efx_ptp_init_filter > v3: fixed bug caused by usage of htons in PTP_EVENT_PORT definition. > It was used in more places, where htons was used too, so using it > 2 times leave it again in host order. I didn't detected it in my > tests because it only affected if timestamping through the MC, but > the model I used do it through the MAC. Detected by kernel test > robot <lkp@intel.com> > v4: removed `inline` specifiers from 2 local functions > > Íñigo Huguet (3): > sfc: allow more flexible way of adding filters for PTP > sfc: support PTP over IPv6/UDP > sfc: support PTP over Ethernet > > drivers/net/ethernet/sfc/filter.h | 22 +++++ > drivers/net/ethernet/sfc/ptp.c | 131 ++++++++++++++++++++---------- > 2 files changed, 111 insertions(+), 42 deletions(-) For the series: Acked-by: Richard Cochran <richardcochran@gmail.com>
Most recent cards (8000 series and newer) had enough hardware support for this, but it was not enabled in the driver. The transmission of PTP packets over these protocols was already added in commit bd4a2697e5e2 ("sfc: use hardware tx timestamps for more than PTP"), but receiving them was already unsupported so synchronization didn't happen. These patches add support for timestamping received packets over IPv6/UPD and IEEE802.3. v2: fixed weird indentation in efx_ptp_init_filter v3: fixed bug caused by usage of htons in PTP_EVENT_PORT definition. It was used in more places, where htons was used too, so using it 2 times leave it again in host order. I didn't detected it in my tests because it only affected if timestamping through the MC, but the model I used do it through the MAC. Detected by kernel test robot <lkp@intel.com> v4: removed `inline` specifiers from 2 local functions Íñigo Huguet (3): sfc: allow more flexible way of adding filters for PTP sfc: support PTP over IPv6/UDP sfc: support PTP over Ethernet drivers/net/ethernet/sfc/filter.h | 22 +++++ drivers/net/ethernet/sfc/ptp.c | 131 ++++++++++++++++++++---------- 2 files changed, 111 insertions(+), 42 deletions(-)