Re: Microchip net DSA with ptp4l getting tx_timeout failed msg using 6.3.12 kernel and KSZ9567 switch

[Christian Eggers <ceggers@arri.de>]

[-- Attachment #1: Type: text/plain, Size: 1961 bytes --]

Hi Brian,

I just return from my holidays...

Am Dienstag, 22. August 2023, 23:49:33 CEST schrieben Sie:
> Getting this tx_timestamp_timeout error over and over when I try to run ptp4l:
> 
> ptp4l[1366.143]: selected best master clock 001747.fffe.70151b
> ptp4l[1366.143]: updating UTC offset to 37
> ptp4l[1366.143]: port 1: LISTENING to UNCALIBRATED on RS_SLAVE
> ptp4l[1366.860]: port 1: delay timeout
> ptp4l[1376.871]: timed out while polling for tx timestamp
> ptp4l[1376.871]: increasing tx_timestamp_timeout may correct this
> issue, but it is likely caused by a driver bug
> ptp4l[1376.871]: port 1: send delay request failed
> 
> I was using 5.10.69 with Christians patches before they were mainlined
> and had everything working with the help of Christian, Vladimir and
> others.
> 
> Now I need to update kernel so tried 6.3.12 which contains Christians
> upstream patches and I also back ported v8 of the upstreamed patches
> to 6.1.38 and I'm getting the same results with that kernel too.
> 

I am also in the process of upgrading to 6.1.38 (but not really tested).
I cherry-picked all necessary patches from the latest master (see attached
archive). Maybe you would like to compare this with your patch series.

> [...]
>
> I tried increasing tx_timestamp and it doesn't appear to matter.  I
> feel like I had this problem before when first starting to work with
> 5.10.69 but can't remember if another patch resolved it.  With 5.10.69
> I've got quite a few more patches than just the 13 that were mainlined
> in 6.3.  Looking through old emails I want to say it might have been
> resolved with net-dsa-ksz9477-avoid-PTP-races-with-the-data-path-l.patch
> that Vladimir gave me but looking at the code it doesn't appear
> mainline has that one.

How is the IRQ line of you switch attached? I remember there was a problem
with the IRQ type (edge vs. level), but I think this has already been
applied to 6.1.38 (via -stable).

regards,
Christian

[-- Attachment #2: 2023-08-23_ptp.tar.gz --]
[-- Type: application/x-compressed-tar, Size: 29954 bytes --]