netdev - Re: [net-next PATCH v3 11/15] eth: fbnic: Add link detection

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, 2 Jul 2024 22:37:20 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Alexander Duyck <alexander.duyck@...il.com>
Cc: "Russell King (Oracle)" <linux@...linux.org.uk>, netdev@...r.kernel.org,
	Alexander Duyck <alexanderduyck@...com>, kuba@...nel.org,
	davem@...emloft.net, pabeni@...hat.com, edumazet@...gle.com,
	kernel-team@...a.com
Subject: Re: [net-next PATCH v3 11/15] eth: fbnic: Add link detection

> > As for multiple PCS for one connection, is this common, or special to
> > your hardware?
> 
> I would think it is common. Basically once you get over 10G you start
> seeing all these XXXXXbase[CDKLS]R[248] speeds advertised and usually
> the 2/4/8 represents the number of lanes being used. I would think
> most hardware probably has a PCS block per lane as they can be
> configured separately and in our case anyway you can use just the one
> lane mode and then you only need to setup 1 lane, or you can use the 2
> lane mode and you need to setup 2.
> 
> Some of our logic is merged like I mentioned though so maybe it would
> make more sense to just merge the lanes. Anyway I guess I can start
> working on that code for the next patch set. I will look at what I
> need to do to extend the logic. For now I might be able to get by with
> just dropping support for 50R1 since that isn't currently being used
> as a default.

So maybe a dumb question. How does negotiation work? Just one performs
negotiation? They all do, and if you get different results you declare
the link broken? First one to complete wins? Or even, you can
configure each lane to use different negotiation parameters...

    Andrew