From: Vladimir Oltean <olteanv@gmail.com>
To: Lukasz Majewski <lukma@denx.de>
Cc: Oleksij Rempel <o.rempel@pengutronix.de>,
Richard Cochran <richardcochran@gmail.com>,
Vadim Fedorenko <vadim.fedorenko@linux.dev>,
netdev@vger.kernel.org,
Arun Ramadoss <arun.ramadoss@microchip.com>,
Tristram.Ha@microchip.com, Christian Eggers <ceggers@arri.de>
Subject: Re: [PTP][KSZ9477][p2p1step] Questions for PTP support on KSZ9477 device
Date: Sat, 28 Jun 2025 00:58:04 +0300 [thread overview]
Message-ID: <20250627215804.mcqsav2x6gbngkib@skbuf> (raw)
In-Reply-To: <20250626233325.559e48a6@wsk>
On Thu, Jun 26, 2025 at 11:33:25PM +0200, Lukasz Majewski wrote:
> The second problem which I've found after some debugging:
> - One device is selected as grandmaster clock. Another one tries to
> synchronize (for the simpler setup I've used two the same boards with
> identical kernel and KSZ9477 setup).
>
> - tshark from host on which we do have grandmaster running:
> IEEEI&MS_00:00:00 PTPv2 58 Sync Message
> LLDP_Multicast PTPv2 68 Peer_Delay_Req Message
> IEEEI&MS_00:00:00 PTPv2 58 Sync Message
> LLDP_Multicast PTPv2 68 Peer_Delay_Req Message
>
> So the SYNC is send, then the "slave" responds correctly with
> Peer_Delay_Req_Message.
Peer delay measurement is an independent process, not a response to Sync
messages.
> But then the "grandmaster" is NOT replying with PER_DELAY_RESPONSE.
>
> After some digging into the code it turned out that
> dsa_skb_defer_rx_timestamp() (from net/dsa/tag.c) calls
> ptp_classify_raw(skb), which is a bpf program.
>
> Instead of returning 0x42 I do receive "PTP_CLASS_NONE" and the frame is
> dropped.
>
> That is why grandmaster cannot send reply and finish the PTP clock
> adjustment process.
>
> The CONFIG_NET_PTP_CLASSIFY=y.
>
> Any hints on how to proceed? If this would help - I'm using linux
> kernel with PREEMPT_RT applied to it.
Which frame is classified as PTP_CLASS_NONE? The peer delay request?
That doesn't sound convincing, can you place a call to skb_dump() and
show the contents of the PTP packets that don't pass this BPF filter?
Notably, the filter matches for event messages and doesn't match for
general messages, maybe that confused your debugging process in some way.
next prev parent reply other threads:[~2025-06-27 21:58 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-16 15:25 [PTP][KSZ9477][p2p1step] Questions for PTP support on KSZ9477 device Lukasz Majewski
2025-06-17 5:25 ` Oleksij Rempel
2025-06-17 9:53 ` Lukasz Majewski
2025-06-17 16:10 ` Vadim Fedorenko
2025-06-18 5:07 ` Richard Cochran
2025-06-18 6:27 ` Oleksij Rempel
2025-06-26 21:33 ` Lukasz Majewski
2025-06-27 21:58 ` Vladimir Oltean [this message]
2025-06-29 9:28 ` Lukasz Majewski
2025-06-30 4:36 ` Oleksij Rempel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250627215804.mcqsav2x6gbngkib@skbuf \
--to=olteanv@gmail.com \
--cc=Tristram.Ha@microchip.com \
--cc=arun.ramadoss@microchip.com \
--cc=ceggers@arri.de \
--cc=lukma@denx.de \
--cc=netdev@vger.kernel.org \
--cc=o.rempel@pengutronix.de \
--cc=richardcochran@gmail.com \
--cc=vadim.fedorenko@linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox