From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A6D05C433FE for ; Tue, 18 Oct 2022 12:08:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230437AbiJRMIK (ORCPT ); Tue, 18 Oct 2022 08:08:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56656 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230433AbiJRMHb (ORCPT ); Tue, 18 Oct 2022 08:07:31 -0400 Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2088.outbound.protection.outlook.com [40.107.100.88]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DFBED23E93 for ; Tue, 18 Oct 2022 05:07:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LXzVj4qck3sLIScvUCOfpjbxhWVTtUa44m46aSrdenmfBM5jacd83dhMhhlr0aCVKAWcQfa/FtFd+9LO3rcfKWaWQR4UKpg7SO02TRY1dda5dvx7Ld64XFg8ZaR2jOWzM1r2EDKplQ9GJeCjD980in+s/gfcmR7cdc8KXsCz/MoudiE0qzNUJUbTYJjJWIwjlU0RigMzxu+B1082v9+a0hSHjM/gk01RVi6UXUXK7fBn8n8BGDv25LO29acFWvyLbsT5w13dHGY8kxHiR+8PPu6FxKZGpC5DIBpZ7PScb2xt0tPg49Jhsm1A1WSsMX1w6rL/jAqbEE1WgOS9q2f6Mw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=dtOv+Q0+/URrw8mkdChIMkWBywbxXqijseBtDgbLa6M=; b=J9MM8116kdC75L+tfXc6I0iBZjoNEKBGHXrvh91Pu6HP6w8VajddWMT9creNlvOsxo9N2kRMRaWGJQgf/AzaWdKj4WD2/r9Aftr7EJtY80yO4//piv3VVOcyqAk/IXr4rUMc99CGHGnFbDGbngDV99E9CtUI1R0tc7epr0r1oakz8DL7gX7rrYy6nErFDhmSBt3rqssO8pmUkxbFpPre3mP/q2LjOTIWI2zw5uKzXhxz6cRBEYdR5e3/1usG01FHEvcrwXsFdLhHyJbeHOutMxHjYTSvsNCsAEBQnozCfFT22Gsf5DniWTV35r0cJwqY5DE9NoeEPOnLVZE7rKzghw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dtOv+Q0+/URrw8mkdChIMkWBywbxXqijseBtDgbLa6M=; b=j2q4gDiZ5o8xWqpnLpL0qMt5anXBHJ0Uw/sOc14kHL0d0xgOLrVq6FSlQA34alx3CaOFr1BvrP6XWtNzdCyDGIuvC6MIX/jXTOC7F8OCmE/3XuwFuEquiUOa2MH9cwtUcxsWZJloNCoeBcDbH6JqyhN96Db5r24mmfEqPHpwVZqjYZvKG6qY/1G6ndD0YR92dsCgTabYF+avFqHISfrpTJOSUkiYLm5xKL19ifeHK8wjQaa/AYGnVquzC/eYVlv5LxTy25+p7buXpzWUz1nDKxPcZS+CUq5pTakxahvz5vXOtyEGdPGheXypeyrReEskv8n3tqn31/nkgoQJ9bmNUQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from CY5PR12MB6179.namprd12.prod.outlook.com (2603:10b6:930:24::22) by CY5PR12MB6406.namprd12.prod.outlook.com (2603:10b6:930:3d::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5723.29; Tue, 18 Oct 2022 12:06:17 +0000 Received: from CY5PR12MB6179.namprd12.prod.outlook.com ([fe80::4ff2:d93e:d200:227e]) by CY5PR12MB6179.namprd12.prod.outlook.com ([fe80::4ff2:d93e:d200:227e%7]) with mapi id 15.20.5723.033; Tue, 18 Oct 2022 12:06:17 +0000 From: Ido Schimmel To: netdev@vger.kernel.org, bridge@lists.linux-foundation.org Cc: davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com, edumazet@google.com, roopa@nvidia.com, razor@blackwall.org, mlxsw@nvidia.com, Ido Schimmel Subject: [RFC PATCH net-next 10/19] bridge: mcast: Split (*, G) and (S, G) addition into different functions Date: Tue, 18 Oct 2022 15:04:11 +0300 Message-Id: <20221018120420.561846-11-idosch@nvidia.com> X-Mailer: git-send-email 2.37.3 In-Reply-To: <20221018120420.561846-1-idosch@nvidia.com> References: <20221018120420.561846-1-idosch@nvidia.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: VI1PR0401CA0007.eurprd04.prod.outlook.com (2603:10a6:800:4a::17) To CY5PR12MB6179.namprd12.prod.outlook.com (2603:10b6:930:24::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CY5PR12MB6179:EE_|CY5PR12MB6406:EE_ X-MS-Office365-Filtering-Correlation-Id: 54717b16-c657-4acf-ae6c-08dab1012945 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Ii1R1uje3BRikd9SEDKwpKyHLq42FJ+PcEMI16jlqEGfkKzX23ldrIX7ATe3Mp3GL+Y9kisaPfql62j/GSGVoRRocftaOemkMkUu1FMcnEM8i51djgZ8UCwuY0PaUuvedUw3SjV7eTt2QkbWNPxCk8Luef3bRAxfrZFfiu73iSFZWy0angNeeyJwcJbyxxfGeqCBD/tkEYqSk0IPgAQ+YjveUodM0SVreIrXvir8ZLuipFiyvZmQJ+U/q+Nd4zXn0WSmLNzwjJAldVCk3uBrKBfIyvXYisYSZsIDlyaRDNuUoVofkkoJMUjK9xd716yK95yKH6zB1Wje3sIrQNq/sPFO+rJHk74WZ8c5PmdXroSREiGThK8b3nS4+yORNotTTpeYuGgJriJpFh1RAIqWGVbB8d9BEbQ2aK6oGf3sanyRwCnipzdaXgrOmXA/J8zVoxwUXJ2zLOilbJucHRkLx57VaE9og3eVg2CHWvmNVw1kX7l40BX4LzJJtDDIYKydC3eUi9sxS0r3eEdF5OIB01a6E19sP2e85Q9jcSKVyryAcKSgWbc7fSbDUNdMGoIHpL5CZW2HPbMMYRyf7duB1jEQbdtUDKnZ7G+NyKuRCZZGdEHYllTKBdPEPOmHFhnrA0Guku35mHDMzRC5r6gTwYNvsr2yJTx9/pDIGq9Xqq2wgcFlQwncFSNsObHqZj8D9ZTQY9si4VJAovCrpM3HBA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CY5PR12MB6179.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(376002)(346002)(396003)(366004)(136003)(39860400002)(451199015)(36756003)(86362001)(38100700002)(1076003)(186003)(5660300002)(107886003)(2616005)(6506007)(6666004)(26005)(6512007)(478600001)(83380400001)(2906002)(6486002)(316002)(66946007)(66556008)(8676002)(66476007)(41300700001)(8936002)(4326008);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?zPce+/ifoB2UNcb9jhdE+bL9NxbbFMij6HyC0AJA9ogHXfj4TR/a2eAHgnhB?= =?us-ascii?Q?Yt/YNOzY+Ii+YHWSwKSfAbAkhgzzBNkCsVk5IuBsF7zZ1oVPZleQUaINgJnj?= =?us-ascii?Q?+GDR3IQkw3vzCzKWHa4Ja9RBTF19iDSH0IhxmG4NEa+3QGYgQ4pg9SGaB+Eo?= =?us-ascii?Q?BbbY2fokuY+W9RMgEQVwGvYkKWMaqyaK005wNvQWK0O9WS057rK7TSOOB7Wn?= =?us-ascii?Q?v6wj9C/WZQTV9MEm7ZTWKiqkJQp5yfTTGT1mkTSbedE5gMrHyG5Cq52Afk1Y?= =?us-ascii?Q?rlbfv3wX7yPqRprs1PDlbasFNEJvrxAO+MZyYRR4gKoLoezbZREEC0O9CtvZ?= =?us-ascii?Q?MkN8am/eW6wm4Va/TY+Kfm1w9S+PiZi6w1UV+Iu/dQkYJn1468ETrOhL7jX/?= =?us-ascii?Q?ivTVTfe/FM1e33B6+PPTi+6i0Kj9EVRZ0H9ONkVyrW3aBgVOg/Amq20+kIA/?= =?us-ascii?Q?bNEUhymhyz7Hts9jVBhoEKVtVr7s7YyrLjf+Ax684I4GViKXyJ0GliHxr32m?= =?us-ascii?Q?GvJirIi1psrxcaqJFu1JD3bSuM+cdKcg3uek0boHHRQOBRbQCOuC0BR6b6Wb?= =?us-ascii?Q?bNq7Ms0DJ6pgd369mJiD/FZJfXW4Jp2X2fTFpB94xQ0zkZtTWlSi1RFdlOWn?= =?us-ascii?Q?NBAABsZcO/20eZ3u1FpgajQA7R3EoXJiTo0E9w7VtBrr60jgF6sGfTMZM/by?= =?us-ascii?Q?s66VJabvUymKaRDnyAO5i4nA8Nn3qe907YaP1ctHtKIEIyID5cCiZSYj/cz6?= =?us-ascii?Q?ydiGBaKMPccc7TzuF4yQax/7N3uPVj/7BvcdpCUFEf4it5wt+FYKmqBgnhOl?= =?us-ascii?Q?g2Fi7CMB08YMJ+HT+GNc4QzxoSOCFOss7ge4fLBdDQOM5rgz9vxGcrOzp/vl?= =?us-ascii?Q?4QikruObJWA7LQTZfN5cQu8p3JmZckik8ptMiMHHn+Gz2tfmnv8tWaVMwRap?= =?us-ascii?Q?jG1tCVQgF966/8bUNdSzR05nHzzJpMTQugWBUEGWR5IRIpDKStjrNBoK/n58?= =?us-ascii?Q?fcNZd5Gkdx5QdbXciau1PMClwCuoeL5A5QN9TZNnoLpJqq+Nw0sxyP0XFrQF?= =?us-ascii?Q?SDhNKZvAl35BLRjsoijYOhtJIgAxKC+JPL0mXN7mLyUtRhun8jIZy43VHtrV?= =?us-ascii?Q?bA1KNKlDPKpNmhNt8nz3Dru84kGY6Xi9dQtHXPLztc4KYqf3v5p88D6UL62U?= =?us-ascii?Q?4CYsHSSa37QLcszwcWA5W2VI/EseciOCyacw6R+K56tNrdSVMTnML9sCvzOh?= =?us-ascii?Q?Ud88TP0QQ+86CUrv7u29X926pepnTeb6YLCDNoDJM7Yxscgxj93D36iErzD+?= =?us-ascii?Q?D1LWgNF8Uh6UsJ2O67m+5svU5ngXpPW+3p6bfHHeMqfKPjEndTkglXgBLYjm?= =?us-ascii?Q?pFg3S8MiTMqELkz1NmIW9GPlLwoBcTkAvdgyzYc1AsUkJuozOZlYCE7Efmw7?= =?us-ascii?Q?bgBXPR6hfXIDYegFJbzhlNX2giLiIYkUqVVbnxy/SS5gHYVvLi7xLsRMliKU?= =?us-ascii?Q?CQs0vIatjjiAdwmwD4Vqwpq4EZ2xWqNrEBYMDPiGjP4Y02lBCkLt0X9KgXXo?= =?us-ascii?Q?lhXKkLz1X5KiQ+nIXaQgzXf6519vPF5Ru+aCFCIH?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 54717b16-c657-4acf-ae6c-08dab1012945 X-MS-Exchange-CrossTenant-AuthSource: CY5PR12MB6179.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2022 12:06:17.3581 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: s1cPbMUyCoae0Ny0qP4wBILSS6nd8tzKzmI2INPNGhpychn1lV36XuOvss/uoDVoZ9NU+O+k7IAT34r3mTBJoA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6406 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org When the bridge is using IGMP version 3 or MLD version 2, it handles the addition of (*, G) and (S, G) entries differently. When a new (S, G) port group entry is added, all the (*, G) EXCLUDE ports need to be added to the port group of the new entry. Similarly, when a new (*, G) EXCLUDE port group entry is added, the port needs to be added to the port group of all the matching (S, G) entries. Subsequent patches will create more differences between both entry types. Namely, filter mode and source list can only be specified for (*, G) entries. Given the current and future differences between both entry types, handle the addition of each entry type in a different function, thereby avoiding the creation of one complex function. Signed-off-by: Ido Schimmel --- net/bridge/br_mdb.c | 145 +++++++++++++++++++++++++++++--------------- 1 file changed, 96 insertions(+), 49 deletions(-) diff --git a/net/bridge/br_mdb.c b/net/bridge/br_mdb.c index dd56063430ed..a48eef866974 100644 --- a/net/bridge/br_mdb.c +++ b/net/bridge/br_mdb.c @@ -786,21 +786,107 @@ __br_mdb_choose_context(struct net_bridge *br, return brmctx; } +static int br_mdb_add_group_sg(struct br_mdb_config *cfg, + struct net_bridge_mdb_entry *mp, + struct net_bridge_mcast *brmctx, + unsigned char flags, + struct netlink_ext_ack *extack) +{ + struct net_bridge_port_group __rcu **pp; + struct net_bridge_port_group *p; + unsigned long now = jiffies; + + for (pp = &mp->ports; + (p = mlock_dereference(*pp, cfg->br)) != NULL; + pp = &p->next) { + if (p->key.port == cfg->p) { + NL_SET_ERR_MSG_MOD(extack, "(S, G) group is already joined by port"); + return -EEXIST; + } + if ((unsigned long)p->key.port < (unsigned long)cfg->p) + break; + } + + p = br_multicast_new_port_group(cfg->p, &cfg->group, *pp, flags, NULL, + MCAST_INCLUDE, RTPROT_STATIC); + if (unlikely(!p)) { + NL_SET_ERR_MSG_MOD(extack, "Couldn't allocate new (S, G) port group"); + return -ENOMEM; + } + rcu_assign_pointer(*pp, p); + if (!(flags & MDB_PG_FLAGS_PERMANENT)) + mod_timer(&p->timer, + now + brmctx->multicast_membership_interval); + br_mdb_notify(cfg->br->dev, mp, p, RTM_NEWMDB); + + /* All of (*, G) EXCLUDE ports need to be added to the new (S, G) for + * proper replication. + */ + if (br_multicast_should_handle_mode(brmctx, cfg->group.proto)) { + struct net_bridge_mdb_entry *star_mp; + struct br_ip star_group; + + star_group = p->key.addr; + memset(&star_group.src, 0, sizeof(star_group.src)); + star_mp = br_mdb_ip_get(cfg->br, &star_group); + if (star_mp) + br_multicast_sg_add_exclude_ports(star_mp, p); + } + + return 0; +} + +static int br_mdb_add_group_star_g(struct br_mdb_config *cfg, + struct net_bridge_mdb_entry *mp, + struct net_bridge_mcast *brmctx, + unsigned char flags, + struct netlink_ext_ack *extack) +{ + struct net_bridge_port_group __rcu **pp; + struct net_bridge_port_group *p; + unsigned long now = jiffies; + + for (pp = &mp->ports; + (p = mlock_dereference(*pp, cfg->br)) != NULL; + pp = &p->next) { + if (p->key.port == cfg->p) { + NL_SET_ERR_MSG_MOD(extack, "(*, G) group is already joined by port"); + return -EEXIST; + } + if ((unsigned long)p->key.port < (unsigned long)cfg->p) + break; + } + + p = br_multicast_new_port_group(cfg->p, &cfg->group, *pp, flags, NULL, + MCAST_EXCLUDE, RTPROT_STATIC); + if (unlikely(!p)) { + NL_SET_ERR_MSG_MOD(extack, "Couldn't allocate new (*, G) port group"); + return -ENOMEM; + } + rcu_assign_pointer(*pp, p); + if (!(flags & MDB_PG_FLAGS_PERMANENT)) + mod_timer(&p->timer, + now + brmctx->multicast_membership_interval); + br_mdb_notify(cfg->br->dev, mp, p, RTM_NEWMDB); + /* If we are adding a new EXCLUDE port group (*, G), it needs to be + * also added to all (S, G) entries for proper replication. + */ + if (br_multicast_should_handle_mode(brmctx, cfg->group.proto)) + br_multicast_star_g_handle_mode(p, MCAST_EXCLUDE); + + return 0; +} + static int br_mdb_add_group(struct br_mdb_config *cfg, struct netlink_ext_ack *extack) { - struct net_bridge_mdb_entry *mp, *star_mp; - struct net_bridge_port_group __rcu **pp; struct br_mdb_entry *entry = cfg->entry; struct net_bridge_port *port = cfg->p; + struct net_bridge_mdb_entry *mp; struct net_bridge *br = cfg->br; - struct net_bridge_port_group *p; struct net_bridge_mcast *brmctx; struct br_ip group = cfg->group; - unsigned long now = jiffies; unsigned char flags = 0; - struct br_ip star_group; - u8 filter_mode; brmctx = __br_mdb_choose_context(br, entry, extack); if (!brmctx) @@ -823,52 +909,13 @@ static int br_mdb_add_group(struct br_mdb_config *cfg, return 0; } - for (pp = &mp->ports; - (p = mlock_dereference(*pp, br)) != NULL; - pp = &p->next) { - if (p->key.port == port) { - NL_SET_ERR_MSG_MOD(extack, "Group is already joined by port"); - return -EEXIST; - } - if ((unsigned long)p->key.port < (unsigned long)port) - break; - } - - filter_mode = br_multicast_is_star_g(&group) ? MCAST_EXCLUDE : - MCAST_INCLUDE; - if (entry->state == MDB_PERMANENT) flags |= MDB_PG_FLAGS_PERMANENT; - p = br_multicast_new_port_group(port, &group, *pp, flags, NULL, - filter_mode, RTPROT_STATIC); - if (unlikely(!p)) { - NL_SET_ERR_MSG_MOD(extack, "Couldn't allocate new port group"); - return -ENOMEM; - } - rcu_assign_pointer(*pp, p); - if (entry->state == MDB_TEMPORARY) - mod_timer(&p->timer, - now + brmctx->multicast_membership_interval); - br_mdb_notify(br->dev, mp, p, RTM_NEWMDB); - /* if we are adding a new EXCLUDE port group (*,G) it needs to be also - * added to all S,G entries for proper replication, if we are adding - * a new INCLUDE port (S,G) then all of *,G EXCLUDE ports need to be - * added to it for proper replication - */ - if (br_multicast_should_handle_mode(brmctx, group.proto)) { - if (br_multicast_is_star_g(&group)) { - br_multicast_star_g_handle_mode(p, filter_mode); - } else { - star_group = p->key.addr; - memset(&star_group.src, 0, sizeof(star_group.src)); - star_mp = br_mdb_ip_get(br, &star_group); - if (star_mp) - br_multicast_sg_add_exclude_ports(star_mp, p); - } - } - - return 0; + if (br_multicast_is_star_g(&group)) + return br_mdb_add_group_star_g(cfg, mp, brmctx, flags, extack); + else + return br_mdb_add_group_sg(cfg, mp, brmctx, flags, extack); } static int __br_mdb_add(struct br_mdb_config *cfg, -- 2.37.3