From patchwork Mon Feb 21 13:33:25 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Niels Dossche X-Patchwork-Id: 12753633 X-Patchwork-Delegate: kuba@kernel.org 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 A92CFC433EF for ; Mon, 21 Feb 2022 13:33:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1359109AbiBUNeE (ORCPT ); Mon, 21 Feb 2022 08:34:04 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:48498 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1359136AbiBUNd5 (ORCPT ); Mon, 21 Feb 2022 08:33:57 -0500 Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-eopbgr140100.outbound.protection.outlook.com [40.107.14.100]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1BAEE2250B; Mon, 21 Feb 2022 05:33:29 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=c1a4KmTBLr4pYKhPLVWD68kRSHGJl/dIhPehFI878wqyE/EtO4sznI1G1Z/xvdPZDg8tWNJL17HdXEk/BTZd40qhTFNNm8P4WjwsVEnFVGdjm5XjOIQzECwg5g+gnFBin76ypeg59LMzrAv4WBaLK+3fAelBRVsGSExKkG9vVJ2qauwOXm7JE19CTaFisMZ4P7e9hhdI0OMROTy1LoL/c0xYNxz4KK3ls/tPWWaLZZTZv+mo5vO14f7vI5IxA/IiEug40d6/pLn2SYbKuieCqCXuYIAx2dqL/YcI/Hw8wzy2LgjGZqZlofi/tlxUr+e3G7e8ZlHRNgdEOjvVc7bKtQ== 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=ZXXC4Ms1YPF9rtbP/xo39+/D4ucpbMqmhYCjSs8cCzs=; b=DC/sFBDGHHTNhv7IuN5VnOjl51LF59QVzVjrGNiqpgqFQHKJ0A4J/5lK2k0yzdSGWeIPgGNPfqeoyHfjU0mAK0ONbxTPqwOaQhd9vuVPTFB1daLi4JfnC88pE5EvjvS2nskA9TlMf4svkgfbpG5ZcP8QPtb1x96K+2QxD544cjHjvRofQZfPplBKv1rljs6EiKt02z8/KfwSWF4OskiFNrwPU60MYjK8K4ldV0cVvDNtsHHpZjuRNyiVwgTyKyTjuN+ts4dMkyp0vuL7wG8ywR19rA8qy4ZJa90+rgOaC84Y115JHsfzoHjYZE/22BowU4Nd+SV/zgIAcXc6iQzvCQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ugent.be; dmarc=pass action=none header.from=ugent.be; dkim=pass header.d=ugent.be; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ugent.be; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ZXXC4Ms1YPF9rtbP/xo39+/D4ucpbMqmhYCjSs8cCzs=; b=Vffnyv/k+/bO53tc6Eejgos7dag3vIbNnx96/hEj/05ldyx6X5QLqinNUOi3PpLhlg4hiqg8B4jg7s5IuWQmtDhVOnAgEt0wiKYHyWg+Fau1RE6E8jV9/kCIbCCRs/kMzKg1V/JVXQMz9zPInW3+ET+QhlJAMY1XtK9G7gxVvaU= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=ugent.be; Received: from AM0PR09MB2324.eurprd09.prod.outlook.com (2603:10a6:208:d9::26) by VE1PR09MB3136.eurprd09.prod.outlook.com (2603:10a6:802:a2::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4995.17; Mon, 21 Feb 2022 13:33:26 +0000 Received: from AM0PR09MB2324.eurprd09.prod.outlook.com ([fe80::fc49:e396:8dd8:5cb9]) by AM0PR09MB2324.eurprd09.prod.outlook.com ([fe80::fc49:e396:8dd8:5cb9%5]) with mapi id 15.20.4995.027; Mon, 21 Feb 2022 13:33:26 +0000 Message-ID: <18b57f7b-6aa6-e87f-e187-feead42fc90a@ugent.be> Date: Mon, 21 Feb 2022 14:33:25 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.1 Content-Language: en-US To: Jiri Pirko Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org From: Niels Dossche Subject: [PATCH] devlink: use devlink lock on DEVLINK_CMD_PORT_SPLIT X-ClientProxiedBy: PR0P264CA0175.FRAP264.PROD.OUTLOOK.COM (2603:10a6:100:1c::19) To AM0PR09MB2324.eurprd09.prod.outlook.com (2603:10a6:208:d9::26) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f9a8c3fc-1b2b-4c83-998a-08d9f53ebd8e X-MS-TrafficTypeDiagnostic: VE1PR09MB3136:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: x74WuRFcF1gH4IccNcxOTp/rOJQ5CD6Eq1woegCbxGcJHRBSpU4GWbKjvpZsl4DpW2JHRvEGsxlZ7WLJA7aX8DUBipbpjzvLCbl14iwXm92atCY2boVq032EaNgf5BDWDmtNu3mozsBtnArNPyJZu8lPvxdmaLRKLklbSBLffXPu6ZioGpsQNfzuAmBj3GWrq/At9cIhnkB1j/Jt/58H+3811MRULxuFRrYvBJmV/IFrZZPHXsyk0HAHQ8yHUW9QJh1H1R55KrJSVsrjdiVqB749HbHsy9+XvnmKuWKyg9iAN8kInYo7bmCNZXjzqp9N338xQxKxtbsqf3YchWz3QCJpEjSBtX6cUaJgY0WPz6+f5WEjikE/Xc3+sawEL3mcpcqlno2a9jnveygfOTr8dsl4xIWyE/LHm8DngCu08N11qauQwKoPAg+Zevxg8b1vy2JYFrHc/mI3OczNXCXyyu4Whfw5EXdZZ+A+KA7TtqT6M6R1G3Mp9AgxFC4u2N9xfJwVklVT1x4IgqrLnLHnh+Irz35qqpaYUkfiCTC9UVTEhuwZWnQXBZC9R5FwWO0AZfMLk+h0jVYfaXh5TcI/uK+NEuWyC96v2sEoi0fcx2ds6RwxOFwOtJmkmiKsfpJA9mfOEJPvl5hga1DQ7u9/IlnR7ke36HJLmoySFIfucv2skkLhJsVa0Ap7HSB/4purZpk0s8rHDnePg7+TbBvSCJT7EZaJIMb1ZOULnEUbrPM= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR09MB2324.eurprd09.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(4636009)(366004)(36756003)(6506007)(508600001)(6486002)(6512007)(31686004)(2616005)(2906002)(38100700002)(66946007)(5660300002)(186003)(6916009)(8936002)(786003)(316002)(44832011)(31696002)(86362001)(83380400001)(66556008)(66476007)(4326008)(8676002)(43740500002)(45980500001);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?KOKoaBvFerMreViTirRi/vuxlLNe?= =?utf-8?q?s8DYeRUkt6U/1fpmg8O0O7q1HpG7DbXo1/GqzpETmt9r3vShLYuzZB8aPKSmtiTNU?= =?utf-8?q?SBguuiA7lfX5CjqaVyMUIzkwZaGqcWuXpCZrGCUBKbAEAyxsdpxC3e99k51lcRqIV?= =?utf-8?q?GekU7yx+4T/drgSxJryFhziqwJ3PnSBJT1qQ4PUx2CIusa2iIY2AuuTnMv82k+RGH?= =?utf-8?q?a8DVLKkJJAI+ClYkMMtL7dMJxge24/Rm/gD4Vr4k9YMZ9qp9GhH7QdhmrTIzTvKHq?= =?utf-8?q?WCyTW4OuTJ4pej7WQiKk0i718jEBds3wbv7EPwkFQvJj+0tUK1g4V3q9UfRdHwJtU?= =?utf-8?q?w8Ng9u6mreMj+xim4FZ9/STNgeUqbBsm5XU21QDxVnt9QU5geac4h1E7mfTEQ4pmx?= =?utf-8?q?F2WyMC6X9cFQJjcA2kAPuUVbjoMA7AYZFz/Fhc1+IREWhCnC0t6/bt74ky7yon8sw?= =?utf-8?q?PrBYwaqhrbIbfzIjmHVY1DK8BsDo8nn+OKdj1Y0TLv1dIPX9/O+ANdNUH1cMsZMhe?= =?utf-8?q?SE5VMUxz9QchXyAwsykQGEM4DMHNylr2OhHlCIrZOOk3lOhj7bQB9BEOWyfG5Me68?= =?utf-8?q?LS2oHagmsPaSy9qXHWndUxrfWoVMUncl7MYQ9jrJ2RR/u1tDZM/XC0+oWYU5hDiqx?= =?utf-8?q?XG5/ik9GcWjiexO3A0n68oasYLdkaE9uZbP4BSCIJ06dPzp+6wI/zXWJ1KyqZQBqX?= =?utf-8?q?w4OxB7dKu0WmgNm2HjHokXdhqXVmKpOUkBlnxw8nfBSYC70MDxesLZs1U1xuQdmB7?= =?utf-8?q?Dp9KlCicCxxIEfWSxve6oBt6uoKG7vu1qsBbEgU86Zg4A1M28Go1wxwu3jxwMilnk?= =?utf-8?q?rY0wzMRwBDr/XqFubhMPj9Yhjt0LU2lJtQn1B7PJfeOMl8P6NL/dkQXD0uq9MJv+R?= =?utf-8?q?OczrArPxiPg2zrJqks9ZgnzuVFXjkYS3hLAITQyLc5jjBUwAf6vSnSBRlmr8/rlyS?= =?utf-8?q?CZ82/5oK4j9lSZXZW7333NdH2kFzRiQgA14WPzMo49h6UtyDcHTgIUpTw5S478Auh?= =?utf-8?q?l7s9Caywyks1JJ/tJUpRfG5o/uMG29uZv3v+AJISUziYPHSFcP1YNOwUD8us8XP4G?= =?utf-8?q?RKmp54G/iF4WBV/eWUOo8ITJyRXUb5WV/2DTzVm4lkGkTEatl3bzSSKYMzuND8Kl1?= =?utf-8?q?A4jne8wSAL+6SgvCSAqmitpKr+Gk6MAvEQ3NeMfc6yd+WGB1u3K/3aLIODAdPUZbV?= =?utf-8?q?4gNDSUnP8FdFMX6IZP0Prx1OQpoTay9csNwUqvnGLq4uPOSB2rKz5c6ePrZPpoh7G?= =?utf-8?q?G/3JjhFKmtOVAslU0BhPCnhurPrgcpoiwuE/N7aXxGI9tY8GIDjNqUL/WJ7u561LP?= =?utf-8?q?ZWuMfNWL7lHihMdhy3yor/7yJBCs74tmQYcjO7GvneqmLZhPR3vRGdaHDaTNz8eEl?= =?utf-8?q?ukaz2UX5+bTWFDJG9rKeKGbkcuTtSKpan31jMoAzCYJ6LwMAY76ad3Yyg81DAI+ys?= =?utf-8?q?eXPNvMPmFaJNdd+jT/xhfYG8rzmMJplDoIygHzc6GxoP9XS1IhPqjUWft0WNyHz9q?= =?utf-8?q?UiKvtYhSaQ3fUpTo+U2dl9+PfUuH/rVW9AyA7T0Im0a290OcKtMENlbtWKgKF0NRf?= =?utf-8?q?sTax57qz+YEyw0u/WPKvV8l/Ewy2vyCu7say/eDxO/eyXPBas89drI=3D?= X-OriginatorOrg: ugent.be X-MS-Exchange-CrossTenant-Network-Message-Id: f9a8c3fc-1b2b-4c83-998a-08d9f53ebd8e X-MS-Exchange-CrossTenant-AuthSource: AM0PR09MB2324.eurprd09.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Feb 2022 13:33:26.8214 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d7811cde-ecef-496c-8f91-a1786241b99c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: wdZVDTjcaFnNDNtt4CdkyUqS45pT5QBQup71OaOqqfde+k63D0eeQNwHXCuIRzPkacKsVBiTQJhDiFT5SnApCw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR09MB3136 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org devlink_nl_cmd_port_split_doit is executed without taking the devlink instance lock. This function calls to devlink_port_get_from_info, which calls devlink_port_get_from_attrs, which calls devlink_port_get_by_index, which accesses devlink->port_list without the instance lock taken, while in other places devlink->port_list access always happens with the instance lock taken. The documentation in the struct also say that the devlink lock protects the port_list. The flag for no locking was added after refactoring the code to no longer use a global lock. Fixes: 2406e7e546b2 ("devlink: Add per devlink instance lock") Signed-off-by: Niels Dossche --- net/core/devlink.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/core/devlink.c b/net/core/devlink.c index fcd9f6d85cf1..563becaa03a6 100644 --- a/net/core/devlink.c +++ b/net/core/devlink.c @@ -8645,7 +8645,6 @@ static const struct genl_small_ops devlink_nl_ops[] = { .validate = GENL_DONT_VALIDATE_STRICT | GENL_DONT_VALIDATE_DUMP, .doit = devlink_nl_cmd_port_split_doit, .flags = GENL_ADMIN_PERM, - .internal_flags = DEVLINK_NL_FLAG_NO_LOCK, }, { .cmd = DEVLINK_CMD_PORT_UNSPLIT,