From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net-next v3 0/2] net: bridge: add per-port unknown multicast flood control Date: Wed, 31 Aug 2016 08:59:18 -0700 Message-ID: <20160831085918.686148a6@xeon-e3> References: <1472650612-2123-1-git-send-email-nikolay@cumulusnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, roopa@cumulusnetworks.com, bridge@lists.linux-foundation.org, davem@davemloft.net To: Nikolay Aleksandrov Return-path: In-Reply-To: <1472650612-2123-1-git-send-email-nikolay@cumulusnetworks.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: bridge-bounces@lists.linux-foundation.org Errors-To: bridge-bounces@lists.linux-foundation.org List-Id: netdev.vger.kernel.org On Wed, 31 Aug 2016 15:36:50 +0200 Nikolay Aleksandrov wrote: > Hi, > The first patch prepares the forwarding path by having the exact packet > type passed down so we can later filter based on it and the per-port > unknown mcast flood flag introduced in the second patch. It is similar to > how the per-port unknown unicast flood flag works. > Nice side-effects of patch 01 are the slight reduction of tests in the > fast-path and a few minor checkpatch fixes. > > v3: don't change br_auto_mask as that will change user-visible behaviour > v2: make pkt_type an enum as per Stephen's comment > > Thanks, > Nik > > > Nikolay Aleksandrov (2): > net: bridge: change unicast boolean to exact pkt_type > net: bridge: add per-port multicast flood flag > > include/linux/if_bridge.h | 1 + > include/uapi/linux/if_link.h | 1 + > net/bridge/br_device.c | 8 ++++---- > net/bridge/br_forward.c | 7 +++++-- > net/bridge/br_if.c | 2 +- > net/bridge/br_input.c | 40 +++++++++++++++++++++++++--------------- > net/bridge/br_netlink.c | 12 +++++++++--- > net/bridge/br_private.h | 7 ++++++- > net/bridge/br_sysfs_if.c | 1 + > 9 files changed, 53 insertions(+), 26 deletions(-) > Looks good thanks, and most importantly shouldn't break existing users. Acked-by: Stephen Hemminger