netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Kory Maincent <kory.maincent@bootlin.com>
Cc: Andrew Lunn <andrew@lunn.ch>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>,
	netdev@vger.kernel.org, Paolo Abeni <pabeni@redhat.com>,
	Simon Horman <horms@kernel.org>
Subject: [Query] ethtool cumbersome timestamping options
Date: Tue, 9 Sep 2025 16:40:55 +0100	[thread overview]
Message-ID: <aMBKh2sqDwkRY04y@shell.armlinux.org.uk> (raw)

While spending some time with PTP stuff, specifically my Marvell PTP
library, and getting mv88e6xxx converted to it, etc, I was trying
out the timestamping related ethtool options:

        ethtool [ FLAGS ] -T|--show-time-stamping DEVNAME       Show time stamping capabilities
                [ index N qualifier precise|approx ]
        ethtool [ FLAGS ] --get-hwtimestamp-cfg DEVNAME Get selected hardware time stamping
        ethtool [ FLAGS ] --set-hwtimestamp-cfg DEVNAME Select hardware time stamping
                [ index N qualifier precise|approx ]

and I'm finding them particularly cumbersome and irritating to use.

Typing:

  ethtool -T eth0 index 0 qualifier precise

or

  ethtool -T eth0 index 1 qualifier precise

is quite annoying, especially when the man page states:

           qualifier precise|approx
                  Qualifier of the ptp hardware clock. Mainly "precise" the de‐
                  fault one is for IEEE 1588 quality and "approx" is  for  NICs
                  DMA point.

Note "the default one". That implies if it isn't given, this is what
will be used if it isn't specified, but this isn't so, you have to
type the whole "qualifier precise" thing out each and every time.
So, it isn't a default at all.

Either the man page needs to be fixed, or ethtool needs to actually
default to the value stated in the man page.

Alternatively, in this age of AI, I'd suggest changing the -- options
for timestamping to be:

ethtool --please-would-you-be-so-kind-as-to-change-the-timestamping-device-to
...

and

ethtool --please-show-me-the-current-timestamping-device-information

and similar, because with AI giving the commands to be executed, it
no longer matters how verbose command options are today. :D (I
suspect there will be some who will have a humour failure with that
suggestion...)

Thanks.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!

             reply	other threads:[~2025-09-09 15:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-09 15:40 Russell King (Oracle) [this message]
2025-09-22 23:19 ` [Query] ethtool cumbersome timestamping options Kory Maincent

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=aMBKh2sqDwkRY04y@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=kory.maincent@bootlin.com \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /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;
as well as URLs for NNTP newsgroup(s).