> > Does this help? Maybe we should walk before running and focus on > > getting non-batch ops working and then revisit? > > I agree. I understand the need for a prepare phase, but it looks like it > exists for specific combinations, i.e. stacked and bonded devices. > > For basic Ethernet switch chips (even DSA), it is *for the moment* a bit > too unnecessarily complex.
I think we are going to need it though. I have bonding on my TODO list. That will put DSA into a stacked system. > What I will suggest next, is to explicitly skip the prepare phase in DSA > (with a good comment as you already suggested), and fix switchdev to > allow drivers to return -EOPNOTSUPP from its commit phase. The switches have a limited number of bonds, called trunks in Marvells terminology. So we will need the prepare phase to say: Sorry, im out of trunks, do it in software. And different chips have different numbers of trunks, so it will need to go down into the chip driver, the DSA layer probably cannot decide. Andrew -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html