* [PATCH net-next] selftests/net: timestamping: add ptp v2 support
@ 2020-10-29 19:09 Grygorii Strashko
2020-10-31 18:40 ` Jakub Kicinski
0 siblings, 1 reply; 4+ messages in thread
From: Grygorii Strashko @ 2020-10-29 19:09 UTC (permalink / raw)
To: David S. Miller, netdev, Jakub Kicinski, Shuah Khan
Cc: Sekhar Nori, linux-kernel, linux-omap, linux-kselftest,
Grygorii Strashko
The timestamping tool is supporting now only PTPv1 (IEEE-1588 2002) while
modern HW often supports also/only PTPv2.
Hence timestamping tool is still useful for sanity testing of PTP drivers
HW timestamping capabilities it's reasonable to upstate it to support
PTPv2. This patch adds corresponding support which can be enabled by using
new parameter "PTPV2".
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
tools/testing/selftests/net/timestamping.c | 47 ++++++++++++++++------
1 file changed, 34 insertions(+), 13 deletions(-)
diff --git a/tools/testing/selftests/net/timestamping.c b/tools/testing/selftests/net/timestamping.c
index f4bb4fef0f39..21091be70688 100644
--- a/tools/testing/selftests/net/timestamping.c
+++ b/tools/testing/selftests/net/timestamping.c
@@ -59,7 +59,8 @@ static void usage(const char *error)
" SOF_TIMESTAMPING_SOFTWARE - request reporting of software time stamps\n"
" SOF_TIMESTAMPING_RAW_HARDWARE - request reporting of raw HW time stamps\n"
" SIOCGSTAMP - check last socket time stamp\n"
- " SIOCGSTAMPNS - more accurate socket time stamp\n");
+ " SIOCGSTAMPNS - more accurate socket time stamp\n"
+ " PTPV2 - use PTPv2 messages\n");
exit(1);
}
@@ -115,13 +116,28 @@ static const unsigned char sync[] = {
0x00, 0x00, 0x00, 0x00
};
-static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len)
+static const unsigned char sync_v2[] = {
+ 0x00, 0x02, 0x00, 0x2C,
+ 0x00, 0x00, 0x02, 0x00,
+ 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0xFF,
+ 0xFE, 0x00, 0x00, 0x00,
+ 0x00, 0x01, 0x00, 0x01,
+ 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0x00,
+ 0x00, 0x00, 0x00, 0x00,
+};
+
+static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len, int ptpv2)
{
+ size_t sync_len = ptpv2 ? sizeof(sync_v2) : sizeof(sync);
+ const void *sync_p = ptpv2 ? sync_v2 : sync;
struct timeval now;
int res;
- res = sendto(sock, sync, sizeof(sync), 0,
- addr, addr_len);
+ res = sendto(sock, sync_p, sync_len, 0, addr, addr_len);
gettimeofday(&now, 0);
if (res < 0)
printf("%s: %s\n", "send", strerror(errno));
@@ -134,9 +150,11 @@ static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len)
static void printpacket(struct msghdr *msg, int res,
char *data,
int sock, int recvmsg_flags,
- int siocgstamp, int siocgstampns)
+ int siocgstamp, int siocgstampns, int ptpv2)
{
struct sockaddr_in *from_addr = (struct sockaddr_in *)msg->msg_name;
+ size_t sync_len = ptpv2 ? sizeof(sync_v2) : sizeof(sync);
+ const void *sync_p = ptpv2 ? sync_v2 : sync;
struct cmsghdr *cmsg;
struct timeval tv;
struct timespec ts;
@@ -210,10 +228,9 @@ static void printpacket(struct msghdr *msg, int res,
"probably SO_EE_ORIGIN_TIMESTAMPING"
#endif
);
- if (res < sizeof(sync))
+ if (res < sync_len)
printf(" => truncated data?!");
- else if (!memcmp(sync, data + res - sizeof(sync),
- sizeof(sync)))
+ else if (!memcmp(sync_p, data + res - sync_len, sync_len))
printf(" => GOT OUR DATA BACK (HURRAY!)");
break;
}
@@ -257,7 +274,7 @@ static void printpacket(struct msghdr *msg, int res,
}
static void recvpacket(int sock, int recvmsg_flags,
- int siocgstamp, int siocgstampns)
+ int siocgstamp, int siocgstampns, int ptpv2)
{
char data[256];
struct msghdr msg;
@@ -288,7 +305,7 @@ static void recvpacket(int sock, int recvmsg_flags,
} else {
printpacket(&msg, res, data,
sock, recvmsg_flags,
- siocgstamp, siocgstampns);
+ siocgstamp, siocgstampns, ptpv2);
}
}
@@ -300,6 +317,7 @@ int main(int argc, char **argv)
int siocgstamp = 0;
int siocgstampns = 0;
int ip_multicast_loop = 0;
+ int ptpv2 = 0;
char *interface;
int i;
int enabled = 1;
@@ -335,6 +353,8 @@ int main(int argc, char **argv)
siocgstampns = 1;
else if (!strcasecmp(argv[i], "IP_MULTICAST_LOOP"))
ip_multicast_loop = 1;
+ else if (!strcasecmp(argv[i], "PTPV2"))
+ ptpv2 = 1;
else if (!strcasecmp(argv[i], "SOF_TIMESTAMPING_TX_HARDWARE"))
so_timestamping_flags |= SOF_TIMESTAMPING_TX_HARDWARE;
else if (!strcasecmp(argv[i], "SOF_TIMESTAMPING_TX_SOFTWARE"))
@@ -369,6 +389,7 @@ int main(int argc, char **argv)
HWTSTAMP_TX_ON : HWTSTAMP_TX_OFF;
hwconfig.rx_filter =
(so_timestamping_flags & SOF_TIMESTAMPING_RX_HARDWARE) ?
+ ptpv2 ? HWTSTAMP_FILTER_PTP_V2_L4_SYNC :
HWTSTAMP_FILTER_PTP_V1_L4_SYNC : HWTSTAMP_FILTER_NONE;
hwconfig_requested = hwconfig;
if (ioctl(sock, SIOCSHWTSTAMP, &hwtstamp) < 0) {
@@ -496,16 +517,16 @@ int main(int argc, char **argv)
printf("has error\n");
recvpacket(sock, 0,
siocgstamp,
- siocgstampns);
+ siocgstampns, ptpv2);
recvpacket(sock, MSG_ERRQUEUE,
siocgstamp,
- siocgstampns);
+ siocgstampns, ptpv2);
}
} else {
/* write one packet */
sendpacket(sock,
(struct sockaddr *)&addr,
- sizeof(addr));
+ sizeof(addr), ptpv2);
next.tv_sec += 5;
continue;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] selftests/net: timestamping: add ptp v2 support
2020-10-29 19:09 [PATCH net-next] selftests/net: timestamping: add ptp v2 support Grygorii Strashko
@ 2020-10-31 18:40 ` Jakub Kicinski
2020-11-01 2:01 ` Richard Cochran
0 siblings, 1 reply; 4+ messages in thread
From: Jakub Kicinski @ 2020-10-31 18:40 UTC (permalink / raw)
To: Grygorii Strashko
Cc: David S. Miller, netdev, Shuah Khan, Sekhar Nori, linux-kernel,
linux-omap, linux-kselftest, Richard Cochran
On Thu, 29 Oct 2020 21:09:31 +0200 Grygorii Strashko wrote:
> The timestamping tool is supporting now only PTPv1 (IEEE-1588 2002) while
> modern HW often supports also/only PTPv2.
>
> Hence timestamping tool is still useful for sanity testing of PTP drivers
> HW timestamping capabilities it's reasonable to upstate it to support
> PTPv2. This patch adds corresponding support which can be enabled by using
> new parameter "PTPV2".
>
> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
CC: Richard
> diff --git a/tools/testing/selftests/net/timestamping.c b/tools/testing/selftests/net/timestamping.c
> index f4bb4fef0f39..21091be70688 100644
> --- a/tools/testing/selftests/net/timestamping.c
> +++ b/tools/testing/selftests/net/timestamping.c
> @@ -59,7 +59,8 @@ static void usage(const char *error)
> " SOF_TIMESTAMPING_SOFTWARE - request reporting of software time stamps\n"
> " SOF_TIMESTAMPING_RAW_HARDWARE - request reporting of raw HW time stamps\n"
> " SIOCGSTAMP - check last socket time stamp\n"
> - " SIOCGSTAMPNS - more accurate socket time stamp\n");
> + " SIOCGSTAMPNS - more accurate socket time stamp\n"
> + " PTPV2 - use PTPv2 messages\n");
> exit(1);
> }
>
> @@ -115,13 +116,28 @@ static const unsigned char sync[] = {
> 0x00, 0x00, 0x00, 0x00
> };
>
> -static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len)
> +static const unsigned char sync_v2[] = {
> + 0x00, 0x02, 0x00, 0x2C,
> + 0x00, 0x00, 0x02, 0x00,
> + 0x00, 0x00, 0x00, 0x00,
> + 0x00, 0x00, 0x00, 0x00,
> + 0x00, 0x00, 0x00, 0x00,
> + 0x00, 0x00, 0x00, 0xFF,
> + 0xFE, 0x00, 0x00, 0x00,
> + 0x00, 0x01, 0x00, 0x01,
> + 0x00, 0x00, 0x00, 0x00,
> + 0x00, 0x00, 0x00, 0x00,
> + 0x00, 0x00, 0x00, 0x00,
> +};
> +
> +static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len, int ptpv2)
> {
> + size_t sync_len = ptpv2 ? sizeof(sync_v2) : sizeof(sync);
> + const void *sync_p = ptpv2 ? sync_v2 : sync;
> struct timeval now;
> int res;
>
> - res = sendto(sock, sync, sizeof(sync), 0,
> - addr, addr_len);
> + res = sendto(sock, sync_p, sync_len, 0, addr, addr_len);
> gettimeofday(&now, 0);
> if (res < 0)
> printf("%s: %s\n", "send", strerror(errno));
> @@ -134,9 +150,11 @@ static void sendpacket(int sock, struct sockaddr *addr, socklen_t addr_len)
> static void printpacket(struct msghdr *msg, int res,
> char *data,
> int sock, int recvmsg_flags,
> - int siocgstamp, int siocgstampns)
> + int siocgstamp, int siocgstampns, int ptpv2)
> {
> struct sockaddr_in *from_addr = (struct sockaddr_in *)msg->msg_name;
> + size_t sync_len = ptpv2 ? sizeof(sync_v2) : sizeof(sync);
> + const void *sync_p = ptpv2 ? sync_v2 : sync;
> struct cmsghdr *cmsg;
> struct timeval tv;
> struct timespec ts;
> @@ -210,10 +228,9 @@ static void printpacket(struct msghdr *msg, int res,
> "probably SO_EE_ORIGIN_TIMESTAMPING"
> #endif
> );
> - if (res < sizeof(sync))
> + if (res < sync_len)
> printf(" => truncated data?!");
> - else if (!memcmp(sync, data + res - sizeof(sync),
> - sizeof(sync)))
> + else if (!memcmp(sync_p, data + res - sync_len, sync_len))
> printf(" => GOT OUR DATA BACK (HURRAY!)");
> break;
> }
> @@ -257,7 +274,7 @@ static void printpacket(struct msghdr *msg, int res,
> }
>
> static void recvpacket(int sock, int recvmsg_flags,
> - int siocgstamp, int siocgstampns)
> + int siocgstamp, int siocgstampns, int ptpv2)
> {
> char data[256];
> struct msghdr msg;
> @@ -288,7 +305,7 @@ static void recvpacket(int sock, int recvmsg_flags,
> } else {
> printpacket(&msg, res, data,
> sock, recvmsg_flags,
> - siocgstamp, siocgstampns);
> + siocgstamp, siocgstampns, ptpv2);
> }
> }
>
> @@ -300,6 +317,7 @@ int main(int argc, char **argv)
> int siocgstamp = 0;
> int siocgstampns = 0;
> int ip_multicast_loop = 0;
> + int ptpv2 = 0;
> char *interface;
> int i;
> int enabled = 1;
> @@ -335,6 +353,8 @@ int main(int argc, char **argv)
> siocgstampns = 1;
> else if (!strcasecmp(argv[i], "IP_MULTICAST_LOOP"))
> ip_multicast_loop = 1;
> + else if (!strcasecmp(argv[i], "PTPV2"))
> + ptpv2 = 1;
> else if (!strcasecmp(argv[i], "SOF_TIMESTAMPING_TX_HARDWARE"))
> so_timestamping_flags |= SOF_TIMESTAMPING_TX_HARDWARE;
> else if (!strcasecmp(argv[i], "SOF_TIMESTAMPING_TX_SOFTWARE"))
> @@ -369,6 +389,7 @@ int main(int argc, char **argv)
> HWTSTAMP_TX_ON : HWTSTAMP_TX_OFF;
> hwconfig.rx_filter =
> (so_timestamping_flags & SOF_TIMESTAMPING_RX_HARDWARE) ?
> + ptpv2 ? HWTSTAMP_FILTER_PTP_V2_L4_SYNC :
> HWTSTAMP_FILTER_PTP_V1_L4_SYNC : HWTSTAMP_FILTER_NONE;
> hwconfig_requested = hwconfig;
> if (ioctl(sock, SIOCSHWTSTAMP, &hwtstamp) < 0) {
> @@ -496,16 +517,16 @@ int main(int argc, char **argv)
> printf("has error\n");
> recvpacket(sock, 0,
> siocgstamp,
> - siocgstampns);
> + siocgstampns, ptpv2);
> recvpacket(sock, MSG_ERRQUEUE,
> siocgstamp,
> - siocgstampns);
> + siocgstampns, ptpv2);
> }
> } else {
> /* write one packet */
> sendpacket(sock,
> (struct sockaddr *)&addr,
> - sizeof(addr));
> + sizeof(addr), ptpv2);
> next.tv_sec += 5;
> continue;
> }
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] selftests/net: timestamping: add ptp v2 support
2020-10-31 18:40 ` Jakub Kicinski
@ 2020-11-01 2:01 ` Richard Cochran
2020-11-02 22:23 ` Jakub Kicinski
0 siblings, 1 reply; 4+ messages in thread
From: Richard Cochran @ 2020-11-01 2:01 UTC (permalink / raw)
To: Jakub Kicinski
Cc: Grygorii Strashko, David S. Miller, netdev, Shuah Khan,
Sekhar Nori, linux-kernel, linux-omap, linux-kselftest
On Sat, Oct 31, 2020 at 11:40:40AM -0700, Jakub Kicinski wrote:
> On Thu, 29 Oct 2020 21:09:31 +0200 Grygorii Strashko wrote:
> > The timestamping tool is supporting now only PTPv1 (IEEE-1588 2002) while
> > modern HW often supports also/only PTPv2.
> >
> > Hence timestamping tool is still useful for sanity testing of PTP drivers
> > HW timestamping capabilities it's reasonable to upstate it to support
> > PTPv2. This patch adds corresponding support which can be enabled by using
> > new parameter "PTPV2".
> >
> > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
>
> CC: Richard
Acked-by: Richard Cochran <richardcochran@gmail.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] selftests/net: timestamping: add ptp v2 support
2020-11-01 2:01 ` Richard Cochran
@ 2020-11-02 22:23 ` Jakub Kicinski
0 siblings, 0 replies; 4+ messages in thread
From: Jakub Kicinski @ 2020-11-02 22:23 UTC (permalink / raw)
To: Richard Cochran
Cc: Grygorii Strashko, David S. Miller, netdev, Shuah Khan,
Sekhar Nori, linux-kernel, linux-omap, linux-kselftest
On Sat, 31 Oct 2020 19:01:41 -0700 Richard Cochran wrote:
> On Sat, Oct 31, 2020 at 11:40:40AM -0700, Jakub Kicinski wrote:
> > On Thu, 29 Oct 2020 21:09:31 +0200 Grygorii Strashko wrote:
> > > The timestamping tool is supporting now only PTPv1 (IEEE-1588 2002) while
> > > modern HW often supports also/only PTPv2.
> > >
> > > Hence timestamping tool is still useful for sanity testing of PTP drivers
> > > HW timestamping capabilities it's reasonable to upstate it to support
> > > PTPv2. This patch adds corresponding support which can be enabled by using
> > > new parameter "PTPV2".
> > >
> > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
> >
> > CC: Richard
>
> Acked-by: Richard Cochran <richardcochran@gmail.com>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-11-02 22:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-29 19:09 [PATCH net-next] selftests/net: timestamping: add ptp v2 support Grygorii Strashko
2020-10-31 18:40 ` Jakub Kicinski
2020-11-01 2:01 ` Richard Cochran
2020-11-02 22:23 ` Jakub Kicinski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).