Nikolay Aleksandrov
2017-Apr-14 10:49 UTC
[Bridge] [PATCH net-next] net: bridge: notify on hw fdb takeover
Recently we added support for SW fdbs to take over HW ones, but that results in changing a user-visible fdb flag thus we need to send a notification, also it's consistent with how HW takes over SW entries. Signed-off-by: Nikolay Aleksandrov <nikolay at cumulusnetworks.com> --- net/bridge/br_fdb.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/bridge/br_fdb.c b/net/bridge/br_fdb.c index 5a40a87c4f4f..de7988b0349e 100644 --- a/net/bridge/br_fdb.c +++ b/net/bridge/br_fdb.c @@ -595,8 +595,10 @@ void br_fdb_update(struct net_bridge *br, struct net_bridge_port *source, if (unlikely(added_by_user)) fdb->added_by_user = 1; /* Take over HW learned entry */ - if (unlikely(fdb->added_by_external_learn)) + if (unlikely(fdb->added_by_external_learn)) { fdb->added_by_external_learn = 0; + fdb_modified = true; + } if (unlikely(fdb_modified)) fdb_notify(br, fdb, RTM_NEWNEIGH); } -- 2.1.4
David Miller
2017-Apr-17 17:45 UTC
[Bridge] [PATCH net-next] net: bridge: notify on hw fdb takeover
From: Nikolay Aleksandrov <nikolay at cumulusnetworks.com> Date: Fri, 14 Apr 2017 13:49:34 +0300> Recently we added support for SW fdbs to take over HW ones, but that > results in changing a user-visible fdb flag thus we need to send a > notification, also it's consistent with how HW takes over SW entries. > > Signed-off-by: Nikolay Aleksandrov <nikolay at cumulusnetworks.com>Applied.