netdev - Re: [PATCH net-next v2 1/7] net: pcs: xpcs: add specific vendor supoprt for Wangxun 10Gb NICs

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]

Date: Tue, 8 Aug 2023 10:21:16 +0100
From: "Russell King (Oracle)" <linux@...linux.org.uk>
To: Jiawen Wu <jiawenwu@...stnetic.com>
Cc: netdev@...r.kernel.org, davem@...emloft.net, edumazet@...gle.com,
	kuba@...nel.org, pabeni@...hat.com, andrew@...n.ch,
	hkallweit1@...il.com, Jose.Abreu@...opsys.com,
	mengyuanlou@...-swift.com
Subject: Re: [PATCH net-next v2 1/7] net: pcs: xpcs: add specific vendor
 supoprt for Wangxun 10Gb NICs

On Tue, Aug 08, 2023 at 10:17:02AM +0800, Jiawen Wu wrote:
> Since Wangxun 10Gb NICs require some special configuration on the IP of
> Synopsys Designware XPCS, introduce dev_flag for different vendors. The
> vendor identification of wangxun devices is added by comparing the name
> of mii bus.
> 
> And interrupt mode is used in Wangxun devices, so make it to be the first
> specific configuration.
> 
> Signed-off-by: Jiawen Wu <jiawenwu@...stnetic.com>

Does the XPCS in Wangxun devices have the device identifiers (registers
2 and 3) and the package identifiers (registers 14 and 15) implemented,
and would they be set to an implementation specific value that would
allow their integration into Wangxun devices to be detected?

If the answer to that is yes, it would be preferable to use that
rather than adding a bitarray of flags to indicate various "quirks".

Thanks.

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