Re: [PATCH v2 net-next 0/2] net: dsa: mv88e6xxx: LAG fixes

On Fri, Jan 15, 2021 at 15:46, Jakub Kicinski [off-list ref] wrote:

On Sat, 16 Jan 2021 01:24:35 +0200 Vladimir Oltean wrote:

quoted

On Fri, Jan 15, 2021 at 03:02:46PM -0800, Jakub Kicinski wrote:

quoted

On Fri, 15 Jan 2021 13:52:57 +0100 Tobias Waldekranz wrote:  

quoted

The kernel test robot kindly pointed out that Global 2 support in
mv88e6xxx is optional.

This also made me realize that we should verify that the hardware
actually supports LAG offloading before trying to configure it.

v1 -> v2:
- Do not allocate LAG ID mappings on unsupported hardware (Vladimir).
- Simplify _has_lag predicate (Vladimir).  

If I'm reading the discussion on v1 right there will be a v3,
LMK if I got it wrong.  

I don't think a v3 was supposed to be coming, what made you think that?

I thought you concluded that the entire CONFIG_NET_DSA_MV88E6XXX_GLOBAL2
should go, you said:

quoted

So, roughly, you save 10%/13k. That hardly justifies the complexity IMO.

Well, based on what Andrew said, my guess is that that is where we will
end up.

But since at the moment net-next does not build for configs without
Global2-support, I would really appreciate it if this series could be
applied to solve that immediate problem.