Commit e5a55a89 authored by John Fastabend's avatar John Fastabend Committed by David S. Miller

net: create generic bridge ops

The PF_BRIDGE:RTM_{GET|SET}LINK nlmsg family and type are
currently embedded in the ./net/bridge module. This prohibits
them from being used by other bridging devices. One example
of this being hardware that has embedded bridging components.

In order to use these nlmsg types more generically this patch
adds two net_device_ops hooks. One to set link bridge attributes
and another to dump the current bride attributes.

	ndo_bridge_setlink()
	ndo_bridge_getlink()

CC: Lennert Buytenhek <buytenh@wantstofly.org>
CC: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarJohn Fastabend <john.r.fastabend@intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a932657f
...@@ -887,6 +887,10 @@ struct netdev_fcoe_hbainfo { ...@@ -887,6 +887,10 @@ struct netdev_fcoe_hbainfo {
* struct net_device *dev, int idx) * struct net_device *dev, int idx)
* Used to add FDB entries to dump requests. Implementers should add * Used to add FDB entries to dump requests. Implementers should add
* entries to skb and update idx with the number of entries. * entries to skb and update idx with the number of entries.
*
* int (*ndo_bridge_setlink)(struct net_device *dev, struct nlmsghdr *nlh)
* int (*ndo_bridge_getlink)(struct sk_buff *skb, u32 pid, u32 seq,
* struct net_device *dev)
*/ */
struct net_device_ops { struct net_device_ops {
int (*ndo_init)(struct net_device *dev); int (*ndo_init)(struct net_device *dev);
...@@ -998,6 +1002,12 @@ struct net_device_ops { ...@@ -998,6 +1002,12 @@ struct net_device_ops {
struct netlink_callback *cb, struct netlink_callback *cb,
struct net_device *dev, struct net_device *dev,
int idx); int idx);
int (*ndo_bridge_setlink)(struct net_device *dev,
struct nlmsghdr *nlh);
int (*ndo_bridge_getlink)(struct sk_buff *skb,
u32 pid, u32 seq,
struct net_device *dev);
}; };
/* /*
......
...@@ -313,6 +313,8 @@ static const struct net_device_ops br_netdev_ops = { ...@@ -313,6 +313,8 @@ static const struct net_device_ops br_netdev_ops = {
.ndo_fdb_add = br_fdb_add, .ndo_fdb_add = br_fdb_add,
.ndo_fdb_del = br_fdb_delete, .ndo_fdb_del = br_fdb_delete,
.ndo_fdb_dump = br_fdb_dump, .ndo_fdb_dump = br_fdb_dump,
.ndo_bridge_getlink = br_getlink,
.ndo_bridge_setlink = br_setlink,
}; };
static void br_dev_free(struct net_device *dev) static void br_dev_free(struct net_device *dev)
......
...@@ -111,54 +111,33 @@ void br_ifinfo_notify(int event, struct net_bridge_port *port) ...@@ -111,54 +111,33 @@ void br_ifinfo_notify(int event, struct net_bridge_port *port)
/* /*
* Dump information about all ports, in response to GETLINK * Dump information about all ports, in response to GETLINK
*/ */
static int br_dump_ifinfo(struct sk_buff *skb, struct netlink_callback *cb) int br_getlink(struct sk_buff *skb, u32 pid, u32 seq,
struct net_device *dev)
{ {
struct net *net = sock_net(skb->sk); int err = 0;
struct net_device *dev;
int idx;
idx = 0;
rcu_read_lock();
for_each_netdev_rcu(net, dev) {
struct net_bridge_port *port = br_port_get_rcu(dev); struct net_bridge_port *port = br_port_get_rcu(dev);
/* not a bridge port */ /* not a bridge port */
if (!port || idx < cb->args[0]) if (!port)
goto skip; goto out;
if (br_fill_ifinfo(skb, port,
NETLINK_CB(cb->skb).portid,
cb->nlh->nlmsg_seq, RTM_NEWLINK,
NLM_F_MULTI) < 0)
break;
skip:
++idx;
}
rcu_read_unlock();
cb->args[0] = idx;
return skb->len; err = br_fill_ifinfo(skb, port, pid, seq, RTM_NEWLINK, NLM_F_MULTI);
out:
return err;
} }
/* /*
* Change state of port (ie from forwarding to blocking etc) * Change state of port (ie from forwarding to blocking etc)
* Used by spanning tree in user space. * Used by spanning tree in user space.
*/ */
static int br_rtm_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) int br_setlink(struct net_device *dev, struct nlmsghdr *nlh)
{ {
struct net *net = sock_net(skb->sk);
struct ifinfomsg *ifm; struct ifinfomsg *ifm;
struct nlattr *protinfo; struct nlattr *protinfo;
struct net_device *dev;
struct net_bridge_port *p; struct net_bridge_port *p;
u8 new_state; u8 new_state;
if (nlmsg_len(nlh) < sizeof(*ifm))
return -EINVAL;
ifm = nlmsg_data(nlh); ifm = nlmsg_data(nlh);
if (ifm->ifi_family != AF_BRIDGE)
return -EPFNOSUPPORT;
protinfo = nlmsg_find_attr(nlh, sizeof(*ifm), IFLA_PROTINFO); protinfo = nlmsg_find_attr(nlh, sizeof(*ifm), IFLA_PROTINFO);
if (!protinfo || nla_len(protinfo) < sizeof(u8)) if (!protinfo || nla_len(protinfo) < sizeof(u8))
...@@ -168,10 +147,6 @@ static int br_rtm_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) ...@@ -168,10 +147,6 @@ static int br_rtm_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg)
if (new_state > BR_STATE_BLOCKING) if (new_state > BR_STATE_BLOCKING)
return -EINVAL; return -EINVAL;
dev = __dev_get_by_index(net, ifm->ifi_index);
if (!dev)
return -ENODEV;
p = br_port_get_rtnl(dev); p = br_port_get_rtnl(dev);
if (!p) if (!p)
return -EINVAL; return -EINVAL;
...@@ -218,29 +193,7 @@ struct rtnl_link_ops br_link_ops __read_mostly = { ...@@ -218,29 +193,7 @@ struct rtnl_link_ops br_link_ops __read_mostly = {
int __init br_netlink_init(void) int __init br_netlink_init(void)
{ {
int err; return rtnl_link_register(&br_link_ops);
err = rtnl_link_register(&br_link_ops);
if (err < 0)
goto err1;
err = __rtnl_register(PF_BRIDGE, RTM_GETLINK, NULL,
br_dump_ifinfo, NULL);
if (err)
goto err2;
err = __rtnl_register(PF_BRIDGE, RTM_SETLINK,
br_rtm_setlink, NULL, NULL);
if (err)
goto err3;
return 0;
err3:
rtnl_unregister_all(PF_BRIDGE);
err2:
rtnl_link_unregister(&br_link_ops);
err1:
return err;
} }
void __exit br_netlink_fini(void) void __exit br_netlink_fini(void)
......
...@@ -553,6 +553,9 @@ extern struct rtnl_link_ops br_link_ops; ...@@ -553,6 +553,9 @@ extern struct rtnl_link_ops br_link_ops;
extern int br_netlink_init(void); extern int br_netlink_init(void);
extern void br_netlink_fini(void); extern void br_netlink_fini(void);
extern void br_ifinfo_notify(int event, struct net_bridge_port *port); extern void br_ifinfo_notify(int event, struct net_bridge_port *port);
extern int br_setlink(struct net_device *dev, struct nlmsghdr *nlmsg);
extern int br_getlink(struct sk_buff *skb, u32 pid, u32 seq,
struct net_device *dev);
#ifdef CONFIG_SYSFS #ifdef CONFIG_SYSFS
/* br_sysfs_if.c */ /* br_sysfs_if.c */
......
...@@ -2252,6 +2252,83 @@ static int rtnl_fdb_dump(struct sk_buff *skb, struct netlink_callback *cb) ...@@ -2252,6 +2252,83 @@ static int rtnl_fdb_dump(struct sk_buff *skb, struct netlink_callback *cb)
return skb->len; return skb->len;
} }
static int rtnl_bridge_getlink(struct sk_buff *skb, struct netlink_callback *cb)
{
struct net *net = sock_net(skb->sk);
struct net_device *dev;
int idx = 0;
u32 portid = NETLINK_CB(cb->skb).portid;
u32 seq = cb->nlh->nlmsg_seq;
rcu_read_lock();
for_each_netdev_rcu(net, dev) {
const struct net_device_ops *ops = dev->netdev_ops;
struct net_device *master = dev->master;
if (idx < cb->args[0])
continue;
if (master && master->netdev_ops->ndo_bridge_getlink) {
const struct net_device_ops *bops = master->netdev_ops;
int err = bops->ndo_bridge_getlink(skb, portid,
seq, dev);
if (err < 0)
break;
else
idx++;
}
if (ops->ndo_bridge_getlink) {
int err = ops->ndo_bridge_getlink(skb, portid,
seq, dev);
if (err < 0)
break;
else
idx++;
}
}
rcu_read_unlock();
cb->args[0] = idx;
return skb->len;
}
static int rtnl_bridge_setlink(struct sk_buff *skb, struct nlmsghdr *nlh,
void *arg)
{
struct net *net = sock_net(skb->sk);
struct ifinfomsg *ifm;
struct net_device *dev;
int err = -EINVAL;
if (nlmsg_len(nlh) < sizeof(*ifm))
return -EINVAL;
ifm = nlmsg_data(nlh);
if (ifm->ifi_family != AF_BRIDGE)
return -EPFNOSUPPORT;
dev = __dev_get_by_index(net, ifm->ifi_index);
if (!dev) {
pr_info("PF_BRIDGE: RTM_SETLINK with unknown ifindex\n");
return -ENODEV;
}
if (dev->master && dev->master->netdev_ops->ndo_bridge_setlink) {
err = dev->master->netdev_ops->ndo_bridge_setlink(dev, nlh);
if (err)
goto out;
}
if (dev->netdev_ops->ndo_bridge_setlink)
err = dev->netdev_ops->ndo_bridge_setlink(dev, nlh);
out:
return err;
}
/* Protected by RTNL sempahore. */ /* Protected by RTNL sempahore. */
static struct rtattr **rta_buf; static struct rtattr **rta_buf;
static int rtattr_max; static int rtattr_max;
...@@ -2433,5 +2510,8 @@ void __init rtnetlink_init(void) ...@@ -2433,5 +2510,8 @@ void __init rtnetlink_init(void)
rtnl_register(PF_BRIDGE, RTM_NEWNEIGH, rtnl_fdb_add, NULL, NULL); rtnl_register(PF_BRIDGE, RTM_NEWNEIGH, rtnl_fdb_add, NULL, NULL);
rtnl_register(PF_BRIDGE, RTM_DELNEIGH, rtnl_fdb_del, NULL, NULL); rtnl_register(PF_BRIDGE, RTM_DELNEIGH, rtnl_fdb_del, NULL, NULL);
rtnl_register(PF_BRIDGE, RTM_GETNEIGH, NULL, rtnl_fdb_dump, NULL); rtnl_register(PF_BRIDGE, RTM_GETNEIGH, NULL, rtnl_fdb_dump, NULL);
rtnl_register(PF_BRIDGE, RTM_GETLINK, NULL, rtnl_bridge_getlink, NULL);
rtnl_register(PF_BRIDGE, RTM_SETLINK, rtnl_bridge_setlink, NULL, NULL);
} }
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