On Mon, Jun 8, 2020 at 7:54 PM David Ahern <dsah...@kernel.org> wrote: > > 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 <ro...@cumulusnetworks.com> > Signed-off-by: David Ahern <dsah...@kernel.org>
nice cleanups. Thanks David. looks like the patches were just applied, FWIW, Acked-by: Roopa Prabhu <ro...@cumulusnetworks.com>