On Tue, Oct 25, 2022 at 01:00:08PM +0300, Ido Schimmel wrote: > Merge plan > ========== > > We need to agree on a merge plan that allows us to start submitting > patches for inclusion and finally conclude this work. In my experience, > it is best to work in small batches. I therefore propose the following > plan: > > * Add MAB support in the bridge driver. This corresponds to patches > #1-#2.
Yes, exactly, let's keep the software implementation its own patch set, together with its selftest. > * Switchdev extensions for MAB offload together with mlxsw > support. This corresponds to patches #3-#16. I can reduce the number > of patches by splitting out the selftests to a separate submission. I don't think that further splitting of mlxsw will be necessary, the threshold is 15 patches, and the current 16-2 = 14. IMO patches 3-4 should go with mlxsw being the first user. > * mv88e6xxx support. I believe the blackhole stuff is an optimization, > so I suggest getting initial MAB offload support without that. Support > for blackhole entries together with offload can be added in a separate > submission. > > * Switchdev extensions for dynamic FDB entries together with mv88e6xxx > support. I can follow up with mlxsw support afterwards. The last 2 can also happen in parallel, if somebody else comes to it. Looks great, thanks for doing this!
