netdev - Re: [PATCH net-next 0/2] net: phy: sfp: Add single-byte SMBus SFP access

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Z7x4oxR5_KtyvSYg@shell.armlinux.org.uk>
Date: Mon, 24 Feb 2025 13:48:19 +0000
From: "Russell King (Oracle)" <linux@...linux.org.uk>
To: Andrew Lunn <andrew@...n.ch>
Cc: Bjørn Mork <bjorn@...k.no>,
	Maxime Chevallier <maxime.chevallier@...tlin.com>,
	davem@...emloft.net, Jakub Kicinski <kuba@...nel.org>,
	Eric Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>,
	Heiner Kallweit <hkallweit1@...il.com>, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org, thomas.petazzoni@...tlin.com,
	Florian Fainelli <f.fainelli@...il.com>,
	Köry Maincent <kory.maincent@...tlin.com>,
	Simon Horman <horms@...nel.org>,
	Romain Gantois <romain.gantois@...tlin.com>,
	Antoine Tenart <atenart@...nel.org>,
	Marek Behún <kabel@...nel.org>
Subject: Re: [PATCH net-next 0/2] net: phy: sfp: Add single-byte SMBus SFP
 access

On Mon, Feb 24, 2025 at 02:31:42PM +0100, Andrew Lunn wrote:
> > What do you think will be the effect of such a warning?  Who is the
> > target audience?
> 
> It will act as a disclaimer. The kernel is doing its best with broken
> hardware, but don't blame the kernel when it does not work
> correctly....

Indeed.

> > You can obviously add it, and I don't really care.  But I believe the
> > result will be an endless stream of end users worrying about this scary
> > warning and wanting to know what they can do about it.  What will be
> > your answer?
> 
> I agree that the wording needs to be though about. Maybe something
> like:
> 
> This hardware is broken by design, and there is nothing the kernel, or
> the community can do about it. The kernel will try its best, but some
> standard SFP features are disabled, and the features which are
> implemented may not work correctly because of the design errors. Use
> with caution, and don't blame the kernel when it all goes horribly
> wrong.

I was hoping for something shorter, but I think it needs to be expansive
so that users can fully understand. Another idea based on your
suggestion above:

"Please note:
This hardware is broken by design. There is nothing that the kernel or
community can do to fix it. The kernel will try best efforts, but some
features are disabled, other features may be unreliable or sporadically
fail. Use with caution. Please verify any problems on hardware that
supports multi-byte I2C transactions."

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