Commit 33eaf2a6 authored by David Ahern's avatar David Ahern Committed by David S. Miller

net: Add extack to ndo_add_slave

Pass extack to do_set_master and down to ndo_add_slave
Signed-off-by: default avatarDavid Ahern <dsahern@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 51d0c047
...@@ -1328,7 +1328,8 @@ void bond_lower_state_changed(struct slave *slave) ...@@ -1328,7 +1328,8 @@ void bond_lower_state_changed(struct slave *slave)
} }
/* enslave device <slave> to bond device <master> */ /* enslave device <slave> to bond device <master> */
int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev) int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev,
struct netlink_ext_ack *extack)
{ {
struct bonding *bond = netdev_priv(bond_dev); struct bonding *bond = netdev_priv(bond_dev);
const struct net_device_ops *slave_ops = slave_dev->netdev_ops; const struct net_device_ops *slave_ops = slave_dev->netdev_ops;
...@@ -3492,7 +3493,7 @@ static int bond_do_ioctl(struct net_device *bond_dev, struct ifreq *ifr, int cmd ...@@ -3492,7 +3493,7 @@ static int bond_do_ioctl(struct net_device *bond_dev, struct ifreq *ifr, int cmd
switch (cmd) { switch (cmd) {
case BOND_ENSLAVE_OLD: case BOND_ENSLAVE_OLD:
case SIOCBONDENSLAVE: case SIOCBONDENSLAVE:
res = bond_enslave(bond_dev, slave_dev); res = bond_enslave(bond_dev, slave_dev, NULL);
break; break;
case BOND_RELEASE_OLD: case BOND_RELEASE_OLD:
case SIOCBONDRELEASE: case SIOCBONDRELEASE:
......
...@@ -1383,7 +1383,7 @@ static int bond_option_slaves_set(struct bonding *bond, ...@@ -1383,7 +1383,7 @@ static int bond_option_slaves_set(struct bonding *bond,
switch (command[0]) { switch (command[0]) {
case '+': case '+':
netdev_dbg(bond->dev, "Adding slave %s\n", dev->name); netdev_dbg(bond->dev, "Adding slave %s\n", dev->name);
ret = bond_enslave(bond->dev, dev); ret = bond_enslave(bond->dev, dev, NULL);
break; break;
case '-': case '-':
......
...@@ -1914,7 +1914,8 @@ static int team_netpoll_setup(struct net_device *dev, ...@@ -1914,7 +1914,8 @@ static int team_netpoll_setup(struct net_device *dev,
} }
#endif #endif
static int team_add_slave(struct net_device *dev, struct net_device *port_dev) static int team_add_slave(struct net_device *dev, struct net_device *port_dev,
struct netlink_ext_ack *extack)
{ {
struct team *team = netdev_priv(dev); struct team *team = netdev_priv(dev);
int err; int err;
......
...@@ -788,7 +788,8 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev) ...@@ -788,7 +788,8 @@ static int do_vrf_add_slave(struct net_device *dev, struct net_device *port_dev)
return ret; return ret;
} }
static int vrf_add_slave(struct net_device *dev, struct net_device *port_dev) static int vrf_add_slave(struct net_device *dev, struct net_device *port_dev,
struct netlink_ext_ack *extack)
{ {
if (netif_is_l3_master(port_dev) || netif_is_l3_slave(port_dev)) if (netif_is_l3_master(port_dev) || netif_is_l3_slave(port_dev))
return -EINVAL; return -EINVAL;
......
...@@ -1246,7 +1246,8 @@ struct net_device_ops { ...@@ -1246,7 +1246,8 @@ struct net_device_ops {
u32 flow_id); u32 flow_id);
#endif #endif
int (*ndo_add_slave)(struct net_device *dev, int (*ndo_add_slave)(struct net_device *dev,
struct net_device *slave_dev); struct net_device *slave_dev,
struct netlink_ext_ack *extack);
int (*ndo_del_slave)(struct net_device *dev, int (*ndo_del_slave)(struct net_device *dev,
struct net_device *slave_dev); struct net_device *slave_dev);
netdev_features_t (*ndo_fix_features)(struct net_device *dev, netdev_features_t (*ndo_fix_features)(struct net_device *dev,
......
...@@ -596,7 +596,8 @@ void bond_destroy_sysfs(struct bond_net *net); ...@@ -596,7 +596,8 @@ void bond_destroy_sysfs(struct bond_net *net);
void bond_prepare_sysfs_group(struct bonding *bond); void bond_prepare_sysfs_group(struct bonding *bond);
int bond_sysfs_slave_add(struct slave *slave); int bond_sysfs_slave_add(struct slave *slave);
void bond_sysfs_slave_del(struct slave *slave); void bond_sysfs_slave_del(struct slave *slave);
int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev); int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev,
struct netlink_ext_ack *extack);
int bond_release(struct net_device *bond_dev, struct net_device *slave_dev); int bond_release(struct net_device *bond_dev, struct net_device *slave_dev);
u32 bond_xmit_hash(struct bonding *bond, struct sk_buff *skb); u32 bond_xmit_hash(struct bonding *bond, struct sk_buff *skb);
int bond_set_carrier(struct bonding *bond); int bond_set_carrier(struct bonding *bond);
......
...@@ -867,7 +867,8 @@ static int batadv_softif_init_late(struct net_device *dev) ...@@ -867,7 +867,8 @@ static int batadv_softif_init_late(struct net_device *dev)
* Return: 0 if successful or error otherwise. * Return: 0 if successful or error otherwise.
*/ */
static int batadv_softif_slave_add(struct net_device *dev, static int batadv_softif_slave_add(struct net_device *dev,
struct net_device *slave_dev) struct net_device *slave_dev,
struct netlink_ext_ack *extack)
{ {
struct batadv_hard_iface *hard_iface; struct batadv_hard_iface *hard_iface;
struct net *net = dev_net(dev); struct net *net = dev_net(dev);
......
...@@ -320,7 +320,8 @@ void br_netpoll_disable(struct net_bridge_port *p) ...@@ -320,7 +320,8 @@ void br_netpoll_disable(struct net_bridge_port *p)
#endif #endif
static int br_add_slave(struct net_device *dev, struct net_device *slave_dev) static int br_add_slave(struct net_device *dev, struct net_device *slave_dev,
struct netlink_ext_ack *extack)
{ {
struct net_bridge *br = netdev_priv(dev); struct net_bridge *br = netdev_priv(dev);
......
...@@ -1957,7 +1957,8 @@ static int do_setvfinfo(struct net_device *dev, struct nlattr **tb) ...@@ -1957,7 +1957,8 @@ static int do_setvfinfo(struct net_device *dev, struct nlattr **tb)
return err; return err;
} }
static int do_set_master(struct net_device *dev, int ifindex) static int do_set_master(struct net_device *dev, int ifindex,
struct netlink_ext_ack *extack)
{ {
struct net_device *upper_dev = netdev_master_upper_dev_get(dev); struct net_device *upper_dev = netdev_master_upper_dev_get(dev);
const struct net_device_ops *ops; const struct net_device_ops *ops;
...@@ -1982,7 +1983,7 @@ static int do_set_master(struct net_device *dev, int ifindex) ...@@ -1982,7 +1983,7 @@ static int do_set_master(struct net_device *dev, int ifindex)
return -EINVAL; return -EINVAL;
ops = upper_dev->netdev_ops; ops = upper_dev->netdev_ops;
if (ops->ndo_add_slave) { if (ops->ndo_add_slave) {
err = ops->ndo_add_slave(upper_dev, dev); err = ops->ndo_add_slave(upper_dev, dev, extack);
if (err) if (err)
return err; return err;
} else { } else {
...@@ -2115,7 +2116,7 @@ static int do_setlink(const struct sk_buff *skb, ...@@ -2115,7 +2116,7 @@ static int do_setlink(const struct sk_buff *skb,
} }
if (tb[IFLA_MASTER]) { if (tb[IFLA_MASTER]) {
err = do_set_master(dev, nla_get_u32(tb[IFLA_MASTER])); err = do_set_master(dev, nla_get_u32(tb[IFLA_MASTER]), extack);
if (err) if (err)
goto errout; goto errout;
status |= DO_SETLINK_MODIFIED; status |= DO_SETLINK_MODIFIED;
...@@ -2753,7 +2754,8 @@ static int rtnl_newlink(struct sk_buff *skb, struct nlmsghdr *nlh, ...@@ -2753,7 +2754,8 @@ static int rtnl_newlink(struct sk_buff *skb, struct nlmsghdr *nlh,
goto out_unregister; goto out_unregister;
} }
if (tb[IFLA_MASTER]) { if (tb[IFLA_MASTER]) {
err = do_set_master(dev, nla_get_u32(tb[IFLA_MASTER])); err = do_set_master(dev, nla_get_u32(tb[IFLA_MASTER]),
extack);
if (err) if (err)
goto out_unregister; goto out_unregister;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment