diff options
author | David Ahern <dsahern@kernel.org> | 2020-06-09 04:54:43 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-06-10 22:18:40 +0200 |
commit | ce9ac056d9cd15630dfca352ff6d3051ba3ba8f6 (patch) | |
tree | ac8f0d1e56f0cede157282bbc982974e36e0c9e3 /drivers | |
parent | net: flow_offload: remove indirect flow_block declarations leftover (diff) | |
download | linux-ce9ac056d9cd15630dfca352ff6d3051ba3ba8f6.tar.xz linux-ce9ac056d9cd15630dfca352ff6d3051ba3ba8f6.zip |
nexthop: Fix fdb labeling for groups
fdb nexthops are marked with a flag. For standalone nexthops, a flag was
added to the nh_info struct. For groups that flag was added to struct
nexthop when it should have been added to the group information. Fix
by removing the flag from the nexthop struct and adding a flag to nh_group
that mirrors nh_info and is really only a caching of the individual types.
Add a helper, nexthop_is_fdb, for use by the vxlan code and fixup the
internal code to use the flag from either nh_info or nh_group.
v2
- propagate fdb_nh in remove_nh_grp_entry
Fixes: 38428d68719c ("nexthop: support for fdb ecmp nexthops")
Cc: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: David Ahern <dsahern@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/vxlan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c index 47424b2da643..8a39e8047f14 100644 --- a/drivers/net/vxlan.c +++ b/drivers/net/vxlan.c @@ -876,7 +876,7 @@ static int vxlan_fdb_nh_update(struct vxlan_dev *vxlan, struct vxlan_fdb *fdb, nh = NULL; goto err_inval; } - if (!nh->is_fdb_nh) { + if (!nexthop_is_fdb(nh)) { NL_SET_ERR_MSG(extack, "Nexthop is not a fdb nexthop"); goto err_inval; } |