netdev - RE: [PATCH v1 net-next 3/3] net: ena: Add PHC documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <448a2aca058d4aeabb39a01e1b56d51e@amazon.com>
Date: Tue, 22 Oct 2024 14:39:13 +0000
From: "Arinzon, David" <darinzon@...zon.com>
To: David Woodhouse <dwmw2@...radead.org>, David Miller <davem@...emloft.net>,
	Jakub Kicinski <kuba@...nel.org>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>
CC: Eric Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>,
	"Machulsky, Zorik" <zorik@...zon.com>, "Matushevsky, Alexander"
	<matua@...zon.com>, "Bshara, Saeed" <saeedb@...zon.com>, "Wilson, Matt"
	<msw@...zon.com>, "Liguori, Anthony" <aliguori@...zon.com>, "Bshara, Nafea"
	<nafea@...zon.com>, "Schmeilin, Evgeny" <evgenys@...zon.com>, "Belgazal,
 Netanel" <netanel@...zon.com>, "Saidi, Ali" <alisaidi@...zon.com>,
	"Herrenschmidt, Benjamin" <benh@...zon.com>, "Kiyanovski, Arthur"
	<akiyano@...zon.com>, "Dagan, Noam" <ndagan@...zon.com>, "Bernstein, Amit"
	<amitbern@...zon.com>, "Agroskin, Shay" <shayagr@...zon.com>, "Abboud, Osama"
	<osamaabb@...zon.com>, "Ostrovsky, Evgeny" <evostrov@...zon.com>, "Tabachnik,
 Ofir" <ofirt@...zon.com>, "Machnikowski, Maciek" <maciek@...hnikowski.net>
Subject: RE: [PATCH v1 net-next 3/3] net: ena: Add PHC documentation

> > > > +  testptp -d /dev/ptp$(ethtool -T <interface> | awk '/PTP
> > > > Hardware
> > > Clock:/ {print $NF}') -k 1
> > > > +
> > >
> > > Does udev create a stable symlink for this, like it does for e.g.
> > > /dev/ptp_kvm ?
> > >
> >
> > Yes, you can add a specific rule for the ena ptp device
> >
> > SUBSYSTEM=="ptp", ATTR{clock_name}=="ena-ptp-*", SYMLINK += "ena-
> ptp"
> 
> I don't see it here yet though:
> 
> https://github.com/systemd/systemd/blob/main/rules.d/50-udev-
> default.rules.in#L33

We will add a pull request to systemd which adds the above suggested rule.