From patchwork Wed Jun 15 17:42:18 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: biggest dreamer <835703180@qq.com> X-Patchwork-Id: 112793 X-Patchwork-Delegate: qi.z.zhang@intel.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B8D45A00BE; Wed, 15 Jun 2022 19:43:00 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6951C427EB; Wed, 15 Jun 2022 19:43:00 +0200 (CEST) Received: from out203-205-251-80.mail.qq.com (out203-205-251-80.mail.qq.com [203.205.251.80]) by mails.dpdk.org (Postfix) with ESMTP id 264EA410D0 for ; Wed, 15 Jun 2022 19:42:57 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qq.com; s=s201512; t=1655314975; bh=5TTRO8TkDwIoXaQ90ZjAsFxUy/YlazVA5NZWuXZcsZs=; h=From:To:Cc:Subject:Date; b=oK5FFz1Jwi7pRCxVkMoUPvIcpIYoyXQ+6Yxvu61aKVTmM5SbEl46uOaimlM2CvFAm panNTV8LE690kWXEBSLcqYi8J2lZ/ocyz3PMM8mPa9LJx+DvcN41PzNGbhFHGL8seC a7xIimzbkJVOj5O/oUpAtnP6S35o1kC7e4R38fq8= Received: from localhost.localdomain ([202.192.80.113]) by newxmesmtplogicsvrsza7.qq.com (NewEsmtp) with SMTP id AB136E23; Thu, 16 Jun 2022 01:42:49 +0800 X-QQ-mid: xmsmtpt1655314969tyvbmo29k Message-ID: X-QQ-XMAILINFO: M0PjjqbLT90wpQu8YkRphHzeq/SkISynkSCV0Z8HDqgofGid39bT0UgJ/Ki3RB XRYiz4Hknes0Bdyk6rlbQnpFy6kZzIA0EZlPBubg6YHoaTHlQaOqSvLWT3TzJBMDmaju7Qf/nivO ME+eYgrgtsuHTcRWneipluP9D1NahketX/t4CpjYB1Io9vh8aQLlBbzhjsuZLIZdKPL9abQvnBW0 PqILua8kt0eHpCMq0ozILLGDTyk7PL6ZHKz5srIb82UgrGCqrLe2OBQ5HNI4p5nfSlE+upggpjv9 DwRr2n8OVK28o1+y8MClfrFI3tlvj+LlYq/Tu9bmlNSBlBo4O6sf6s0XnSmmdacUESK2ctSYTgea beAZnr8itfNvCNDhNoz6FeMSeAUiAV0IhVDnAZ7PY8NcV8Qpsf4pRIcS24ZDxS4lnqX1Sb5hAETN 8Dr1Z/WjpADu1jBZcjhYsYIiJvR0MBRWf7x0mO7+2dX6hE4LtxlXW5PsPR+kXfhe7hQWQ25o9LHh Z+8V6ETPtB/d4WnnpB35/9yCjveYx07sQPZorR+uBXAIy1wQ77bfNfjz5mIt/zg88U4Jq9hk6VDl 42dwliT8ilzsmPLJzNRAxefp+d3IOfsmU+fmBcQCWsu6HVDDPa9VrKmkpoInU8j5QTqMBzOwif+w av9QKAFIZa3hrtLfSmyNAQubmfnlfAmaj+PDmvskg8N1Dm2C62W04L7Hj/WBdx9bxsWn0i70Nr+c yrRKgbseTYz9Y8BQteUTCYtcg4N/U3bzqiAwC0PftqWQFLC81rCYQmtVTMeqUKdLIwd9QnnUEtgT HilZ5Q/fGHMLJgsAXPV6m81YhUTvcYZ24OvBH0sRV3CYoaGm85jDi6wW/jWZ+uTglQVvEqIasSUL pO7bMXZKuAW7D4RVZRriHwh84s9J8UXjxT9cB7u75quTMM56eeXxU= From: 835703180@qq.com To: jingjing.wu@intel.com Cc: dev@dpdk.org, Shiqi Liu <835703180@qq.com> Subject: [V2] common/iavf:replace zero-length arrays with flexible ones Date: Thu, 16 Jun 2022 01:42:18 +0800 X-OQ-MSGID: <20220615174218.2308-1-835703180@qq.com> X-Mailer: git-send-email 2.35.1.windows.2 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org From: Shiqi Liu <835703180@qq.com> This patch replaces instances of zero-sized arrays i.e. those at the end of structures with "[0]" with the more standard syntax of "[]". Replacement was done using coccinelle script, with some revert and cleanup of whitespace afterwards. v2: * revise a error with BRACKET_SPACE Signed-off-by: Shiqi Liu <835703180@qq.com> --- ...ace-zero-length-arrays-with-flexible.patch | 76 +++++++++++++++++++ drivers/common/iavf/virtchnl_inline_ipsec.h | 2 +- 2 files changed, 77 insertions(+), 1 deletion(-) create mode 100644 0001-common-iavf-replace-zero-length-arrays-with-flexible.patch diff --git a/0001-common-iavf-replace-zero-length-arrays-with-flexible.patch b/0001-common-iavf-replace-zero-length-arrays-with-flexible.patch new file mode 100644 index 0000000000..25c5658791 --- /dev/null +++ b/0001-common-iavf-replace-zero-length-arrays-with-flexible.patch @@ -0,0 +1,76 @@ +From e680b5d91aad9e22d45652efdd85cd2c40105247 Mon Sep 17 00:00:00 2001 +From: Shiqi Liu <835703180@qq.com> +Date: Thu, 16 Jun 2022 01:12:47 +0800 +Subject: [PATCH] common/iavf: replace zero-length arrays with flexible ones + + This patch replaces instances of zero-sized arrays i.e. those at the + end of structures with "[0]" with the more standard syntax of "[]". + Replacement was done using coccinelle script, with some revert and + cleanup of whitespace afterwards. + +Signed-off-by: Shiqi Liu <835703180@qq.com> +--- + drivers/common/iavf/virtchnl_inline_ipsec.h | 24 ++++++++++----------- + drivers/common/mlx5/mlx5_prm.h | 4 ++-- + 2 files changed, 14 insertions(+), 14 deletions(-) + +diff --git a/drivers/common/iavf/virtchnl_inline_ipsec.h b/drivers/common/iavf/virtchnl_inline_ipsec.h +index 2f4bf15725..9d5c433d32 100644 +--- a/drivers/common/iavf/virtchnl_inline_ipsec.h ++++ b/drivers/common/iavf/virtchnl_inline_ipsec.h +@@ -500,25 +500,25 @@ struct inline_ipsec_msg { + + union { + /* IPsec request */ +- struct virtchnl_ipsec_sa_cfg sa_cfg[0]; +- struct virtchnl_ipsec_sp_cfg sp_cfg[0]; +- struct virtchnl_ipsec_sa_update sa_update[0]; +- struct virtchnl_ipsec_sa_destroy sa_destroy[0]; +- struct virtchnl_ipsec_sp_destroy sp_destroy[0]; ++ struct virtchnl_ipsec_sa_cfg sa_cfg[]; ++ struct virtchnl_ipsec_sp_cfg sp_cfg[]; ++ struct virtchnl_ipsec_sa_update sa_update []; ++ struct virtchnl_ipsec_sa_destroy sa_destroy[]; ++ struct virtchnl_ipsec_sp_destroy sp_destroy[]; + + /* IPsec response */ +- struct virtchnl_ipsec_sa_cfg_resp sa_cfg_resp[0]; +- struct virtchnl_ipsec_sp_cfg_resp sp_cfg_resp[0]; +- struct virtchnl_ipsec_cap ipsec_cap[0]; +- struct virtchnl_ipsec_status ipsec_status[0]; ++ struct virtchnl_ipsec_sa_cfg_resp sa_cfg_resp[]; ++ struct virtchnl_ipsec_sp_cfg_resp sp_cfg_resp[]; ++ struct virtchnl_ipsec_cap ipsec_cap[]; ++ struct virtchnl_ipsec_status ipsec_status[]; + /* response to del_sa, del_sp, update_sa */ +- struct virtchnl_ipsec_resp ipsec_resp[0]; ++ struct virtchnl_ipsec_resp ipsec_resp[]; + + /* IPsec event (no req_id is required) */ +- struct virtchnl_ipsec_event event[0]; ++ struct virtchnl_ipsec_event event[]; + + /* Reserved */ +- struct virtchnl_ipsec_sa_read sa_read[0]; ++ struct virtchnl_ipsec_sa_read sa_read[]; + } ipsec_data; + } __rte_packed; + +diff --git a/drivers/common/mlx5/mlx5_prm.h b/drivers/common/mlx5/mlx5_prm.h +index 654e5f44ee..c841d34216 100644 +--- a/drivers/common/mlx5/mlx5_prm.h ++++ b/drivers/common/mlx5/mlx5_prm.h +@@ -552,8 +552,8 @@ struct mlx5_umr_wqe { + struct mlx5_wqe_umr_cseg ucseg; + struct mlx5_wqe_mkey_cseg mkc; + union { +- struct mlx5_wqe_dseg kseg[0]; +- struct mlx5_wqe_umr_bsf_seg bsf[0]; ++ struct mlx5_wqe_dseg kseg[]; ++ struct mlx5_wqe_umr_bsf_seg bsf[]; + }; + } __rte_packed; + +-- +2.35.1.windows.2 + diff --git a/drivers/common/iavf/virtchnl_inline_ipsec.h b/drivers/common/iavf/virtchnl_inline_ipsec.h index 9d5c433d32..0781de1fe1 100644 --- a/drivers/common/iavf/virtchnl_inline_ipsec.h +++ b/drivers/common/iavf/virtchnl_inline_ipsec.h @@ -502,7 +502,7 @@ struct inline_ipsec_msg { /* IPsec request */ struct virtchnl_ipsec_sa_cfg sa_cfg[]; struct virtchnl_ipsec_sp_cfg sp_cfg[]; - struct virtchnl_ipsec_sa_update sa_update []; + struct virtchnl_ipsec_sa_update sa_update[]; struct virtchnl_ipsec_sa_destroy sa_destroy[]; struct virtchnl_ipsec_sp_destroy sp_destroy[];