Re: [PATCH net-next v17 00/14] Introduce PHY listing and link_topology tracking

[Christophe Leroy]

Hi Jakub, Russell

Le 17/07/2024 à 17:26, Jakub Kicinski a écrit :
> On Tue, 16 Jul 2024 10:16:26 +0200 Maxime Chevallier wrote:
> > > I lack the confidence to take this during the merge window, without
> > > Russell's acks. So Deferred, sorry :(
> >
> > Understood. Is there anything I can make next time to make that series
> > more digestable and easy to review ? I didn't want to split the netlink
> > part from the core part, as just the phy_link_topology alone doesn't
> > make much sense for now, but it that makes the lives of reviewers
> > easier I could submit these separately.
>
> TBH I can only review this from coding and netlink perspective, and
> it looks solid. Folk who actually know PHYs and SFPs may have more
> meaningful feedback :(

How can we progress on this ?

Russell, have you been able to have a look at that latest version of the 
series ? I know you reviewed earlier versions already but I understand 
Jakub is willing some feedback from you.

Jakub, as you say it looks solid. I can add to that that I have been 
using this series widely through the double Ethernet attachment on 
several boards and it works well, it is stable and more performant than 
the dirty home-made solution we had on v4.14.

So it would be great if the series could be merged for v6.12, and I 
guess the earliest it is merged into net-next the more time it spends in 
linux-next before the merge window. Any chance to get it merged anytime 
soon even without a formal feedback from Russell ? We are really looking 
forward to getting that series merged and step forward with all the work 
that depends on it and is awaiting.

Thanks
Christophe