On 1/29/2020 11:57 AM, Gal Pressman wrote: > On 26/01/2020 15:21, Maor Gottlieb wrote: >> Add new ndo to get the xmit slave of master device. >> When slave selection method is based on hash, then the user can ask to >> get the xmit slave assume all the slaves can transmit by setting the >> LAG_FLAGS_HASH_ALL_SLAVES bit in the flags argument. >> >> Signed-off-by: Maor Gottlieb <maorg@xxxxxxxxxxxx> >> --- >> include/linux/netdevice.h | 3 +++ >> include/net/lag.h | 19 +++++++++++++++++++ >> 2 files changed, 22 insertions(+) >> >> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h >> index 11bdf6cb30bd..faba4aa094e5 100644 >> --- a/include/linux/netdevice.h >> +++ b/include/linux/netdevice.h >> @@ -1379,6 +1379,9 @@ struct net_device_ops { >> struct netlink_ext_ack *extack); >> int (*ndo_del_slave)(struct net_device *dev, >> struct net_device *slave_dev); >> + struct net_device* (*ndo_xmit_slave_get)(struct net_device *master_dev, >> + struct sk_buff *skb, >> + int lag); > Hey Maor, > Should lag be named flags? > Also, better to use unsigned type for it. Yeah, will change it.