From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1602503972; x=1634039972; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=ZQpmQnv2bppFu4iHMRCySxuz/8he+l1cLdbBjy/BBIY=; b=JR165EMWKj7uVed2iCNotZgOzWTpraiYREYkboksK6GgfwyfZoYQlZNm qnHR3E5CU4AJmp/6v0ET5p2Egdznrwo0gmumHHP8CBn57o+keM3h5f1DV zOpwtztCUq79X1q3qff86xhpIPxfllWDXcmW0d/ZxCo+COEat4MEx/l4i 0MrkMHRla69t5RQJv2pT/Nz0PgEZMcKNuF2yQKJyQoH8Zd3ETjmBI9Kcx MAGHMpJChkXeR5s/eupeaPmQANxZnR637WX05MbQ/clgoRGHQBOShMfwY 8meZFKvwYH1A3Fh0TRaEU01nRgePcO9KuinTNzezCsqVLnsppkykeY0lJ w==; Date: Mon, 12 Oct 2020 11:57:44 +0000 From: "henrik.bjoernlund@microchip.com" Message-ID: <20201012115744.56mjeuoe7nlonjxt@soft-test08> References: <20201009143530.2438738-1-henrik.bjoernlund@microchip.com> <20201009143530.2438738-2-henrik.bjoernlund@microchip.com> <3fc314a2074001f7b39bada2c50529eb2aefd077.camel@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <3fc314a2074001f7b39bada2c50529eb2aefd077.camel@nvidia.com> Subject: Re: [Bridge] [PATCH net-next v4 01/10] net: bridge: extend the process of special frames List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Nikolay Aleksandrov Cc: "netdev@vger.kernel.org" , "bridge@lists.linux-foundation.org" , "linux-kernel@vger.kernel.org" , "UNGLinuxDriver@microchip.com" , "idosch@mellanox.com" , "jiri@mellanox.com" , Roopa Prabhu , "kuba@kernel.org" , "horatiu.vultur@microchip.com" , "davem@davemloft.net" Thanks for the review. Comments below. The 10/12/2020 09:12, Nikolay Aleksandrov wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On Fri, 2020-10-09 at 14:35 +0000, Henrik Bjoernlund wrote: > > This patch extends the processing of frames in the bridge. Currently MRP > > frames needs special processing and the current implementation doesn't > > allow a nice way to process different frame types. Therefore try to > > improve this by adding a list that contains frame types that need > > special processing. This list is iterated for each input frame and if > > there is a match based on frame type then these functions will be called > > and decide what to do with the frame. It can process the frame then the > > bridge doesn't need to do anything or don't process so then the bridge > > will do normal forwarding. > > > > Signed-off-by: Henrik Bjoernlund > > Reviewed-by: Horatiu Vultur > > --- > > net/bridge/br_device.c | 1 + > > net/bridge/br_input.c | 33 ++++++++++++++++++++++++++++++++- > > net/bridge/br_mrp.c | 19 +++++++++++++++---- > > net/bridge/br_private.h | 18 ++++++++++++------ > > 4 files changed, 60 insertions(+), 11 deletions(-) > > > [snip] > > diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h > > index 345118e35c42..3e62ce2ef8a5 100644 > > --- a/net/bridge/br_private.h > > +++ b/net/bridge/br_private.h > > @@ -480,6 +480,8 @@ struct net_bridge { > > #endif > > struct hlist_head fdb_list; > > > > + struct hlist_head frame_type_list; > > Since there will be a v5, I'd suggest to move this struct member in the first > cache line as it will be always used in the bridge fast-path for all cases. > In order to make room for it there you can move port_list after fdb_hash_tbl and > add this in its place, port_list is currently used only when flooding and soon > I'll even change that. > > Thanks, > Nik > I will do change as requested. > > + > > #if IS_ENABLED(CONFIG_BRIDGE_MRP) > > struct list_head mrp_list; > > #endif > > @@ -755,6 +757,16 @@ int nbp_backup_change(struct net_bridge_port *p, struct net_device *backup_dev); > > int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb); > > rx_handler_func_t *br_get_rx_handler(const struct net_device *dev); > > > > +struct br_frame_type { > > + __be16 type; > > + int (*frame_handler)(struct net_bridge_port *port, > > + struct sk_buff *skb); > > + struct hlist_node list; > > +}; > > + > > +void br_add_frame(struct net_bridge *br, struct br_frame_type *ft); > > +void br_del_frame(struct net_bridge *br, struct br_frame_type *ft); > > + > > static inline bool br_rx_handler_check_rcu(const struct net_device *dev) > > { > > return rcu_dereference(dev->rx_handler) == br_get_rx_handler(dev); > > @@ -1417,7 +1429,6 @@ extern int (*br_fdb_test_addr_hook)(struct net_device *dev, unsigned char *addr) > > #if IS_ENABLED(CONFIG_BRIDGE_MRP) > > int br_mrp_parse(struct net_bridge *br, struct net_bridge_port *p, > > struct nlattr *attr, int cmd, struct netlink_ext_ack *extack); > > -int br_mrp_process(struct net_bridge_port *p, struct sk_buff *skb); > > bool br_mrp_enabled(struct net_bridge *br); > > void br_mrp_port_del(struct net_bridge *br, struct net_bridge_port *p); > > int br_mrp_fill_info(struct sk_buff *skb, struct net_bridge *br); > > @@ -1429,11 +1440,6 @@ static inline int br_mrp_parse(struct net_bridge *br, struct net_bridge_port *p, > > return -EOPNOTSUPP; > > } > > > > -static inline int br_mrp_process(struct net_bridge_port *p, struct sk_buff *skb) > > -{ > > - return 0; > > -} > > - > > static inline bool br_mrp_enabled(struct net_bridge *br) > > { > > return false; > -- /Henrik