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] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250508204059.msdda5kll4s7coti@skbuf>
Date: Thu, 8 May 2025 23:40:59 +0300
From: Vladimir Oltean <vladimir.oltean@....com>
To: Vadim Fedorenko <vadim.fedorenko@...ux.dev>
Cc: netdev@...r.kernel.org,
	Köry Maincent <kory.maincent@...tlin.com>,
	Kurt Kanzenbach <kurt@...utronix.de>, Andrew Lunn <andrew@...n.ch>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Woojung Huh <woojung.huh@...rochip.com>,
	UNGLinuxDriver@...rochip.com,
	Claudiu Manoil <claudiu.manoil@....com>,
	Alexandre Belloni <alexandre.belloni@...tlin.com>,
	Simon Horman <horms@...nel.org>,
	Richard Cochran <richardcochran@...il.com>,
	Russell King <linux@...linux.org.uk>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net-next] net: dsa: convert to ndo_hwtstamp_get() and
 ndo_hwtstamp_set()

On Thu, May 08, 2025 at 09:25:14PM +0100, Vadim Fedorenko wrote:
> The new interface also supports providing error explanation via extack,
> it would be great to add some error messages in case when setter fails.
> For example, HIRSCHMANN HellCreek switch doesn't support disabling
> of timestamps, it's not obvious from general -ERANGE error code, but can
> be explained by the text in extack message.

I wanted to keep the patches spartan and not lose track of the conversion
subtleties in embelishments like extack messages which can be added later
and do not require nearly as much attention to the flow before and after.
I'm afraid if I say "yes" here to the request to add extack to hellcreek
I'm opening the door to further requests to do that for other DSA drivers,
and sadly I do not have infinite time to fulfill them. Plus, I would
like to finalize the conversion tree-wide by the end of this development
cycle.

Even if I were to follow through with your request, I would do so in a
separate patch. I've self-reviewed this patch prior to posting it, and I
was already of the impression that it is pretty busy as it is.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ