Thread (4 messages) 4 messages, 3 authors, 2014-09-02

Re: [PATCH net-next] bonding: add slave_changelink support and use it for queue_id

From: Jiri Pirko <jiri@resnulli.us>
Date: 2014-08-27 14:30:04

Wed, Aug 27, 2014 at 04:06:46PM CEST, nikolay@redhat.com wrote:
This patch adds support for slave_changelink to the bonding and uses it
to give the ability to change the queue_id of the enslaved devices via
netlink. It sets slave_maxtype and uses bond_changelink as a prototype for
bond_slave_changelink.
Example/test command after the iproute2 patch:
ip link set eth0 type bond_slave queue_id 10

CC: David S. Miller <davem@davemloft.net>
CC: Jay Vosburgh <redacted>
CC: Veaceslav Falico <redacted>
CC: Andy Gospodarek <andy@greyhouse.net>

Suggested-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: Nikolay Aleksandrov <redacted>
---
note: I intend to add a new option flag to indicate it's a slave option
     so a new netdev notifier can be called (NETDEV_CHANGESLAVEINFODATA).
	No need fo that atm.
     I believe the bonding is the first user of slave_changelink so I hope
     I haven't missed anything. Once this patch is accepted in some form
     I'll post the iproute2 one.

This patch is looking good to me.

Acked-by: Jiri Pirko <jiri@resnulli.us>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help