Commit 50cb8769 authored by David Ahern's avatar David Ahern Committed by David S. Miller

vxlan: Remove access to nexthop group struct

vxlan driver should be using helpers to access nexthop struct
internals. Remove open check if whether nexthop is multipath in
favor of the existing nexthop_is_multipath helper. Add a new
helper, nexthop_has_v4, to cover the need to check has_v4 in
a group.

Fixes: 1274e1cc ("vxlan: ecmp support for mac fdb entries")
Cc: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: default avatarDavid Ahern <dsahern@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ce9ac056
...@@ -857,7 +857,6 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb, ...@@ -857,7 +857,6 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb,
u32 nhid, struct netlink_ext_ack *extack) u32 nhid, struct netlink_ext_ack *extack)
{ {
struct nexthop *old_nh = rtnl_dereference(fdb->nh); struct nexthop *old_nh = rtnl_dereference(fdb->nh);
struct nh_group *nhg;
struct nexthop *nh; struct nexthop *nh;
int err = -EINVAL; int err = -EINVAL;
...@@ -881,8 +880,7 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb, ...@@ -881,8 +880,7 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb,
goto err_inval; goto err_inval;
} }
nhg = rtnl_dereference(nh->nh_grp); if (!nexthop_is_multipath(nh)) {
if (!nh->is_group || !nhg->mpath) {
NL_SET_ERR_MSG(extack, "Nexthop is not a multipath group"); NL_SET_ERR_MSG(extack, "Nexthop is not a multipath group");
goto err_inval; goto err_inval;
} }
...@@ -890,14 +888,14 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb, ...@@ -890,14 +888,14 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb,
/* check nexthop group family */ /* check nexthop group family */
switch (vxlan->default_dst.remote_ip.sa.sa_family) { switch (vxlan->default_dst.remote_ip.sa.sa_family) {
case AF_INET: case AF_INET:
if (!nhg->has_v4) { if (!nexthop_has_v4(nh)) {
err = -EAFNOSUPPORT; err = -EAFNOSUPPORT;
NL_SET_ERR_MSG(extack, "Nexthop group family not supported"); NL_SET_ERR_MSG(extack, "Nexthop group family not supported");
goto err_inval; goto err_inval;
} }
break; break;
case AF_INET6: case AF_INET6:
if (nhg->has_v4) { if (nexthop_has_v4(nh)) {
err = -EAFNOSUPPORT; err = -EAFNOSUPPORT;
NL_SET_ERR_MSG(extack, "Nexthop group family not supported"); NL_SET_ERR_MSG(extack, "Nexthop group family not supported");
goto err_inval; goto err_inval;
......
...@@ -151,6 +151,17 @@ static inline bool nexthop_is_fdb(const struct nexthop *nh) ...@@ -151,6 +151,17 @@ static inline bool nexthop_is_fdb(const struct nexthop *nh)
} }
} }
static inline bool nexthop_has_v4(const struct nexthop *nh)
{
if (nh->is_group) {
struct nh_group *nh_grp;
nh_grp = rcu_dereference_rtnl(nh->nh_grp);
return nh_grp->has_v4;
}
return false;
}
static inline bool nexthop_is_multipath(const struct nexthop *nh) static inline bool nexthop_is_multipath(const struct nexthop *nh)
{ {
if (nh->is_group) { if (nh->is_group) {
......
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