From:   Guo Zhengkui <guozhengkui@vivo.com>
To:     Shuah Khan <shuah@kernel.org>, Alexei Starovoitov <ast@kernel.org>,
        Daniel Borkmann <daniel@iogearbox.net>,
        Andrii Nakryiko <andrii@kernel.org>,
        Martin KaFai Lau <kafai@fb.com>,
        Song Liu <songliubraving@fb.com>, Yonghong Song <yhs@fb.com>,
        John Fastabend <john.fastabend@gmail.com>,
        KP Singh <kpsingh@kernel.org>,
        Guo Zhengkui <guozhengkui@vivo.com>,
        Yucong Sun <sunyucong@gmail.com>,
        Dave Marchevsky <davemarchevsky@fb.com>,
        linux-kselftest@vger.kernel.org, netdev@vger.kernel.org,
        bpf@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     kernel@vivo.com
Subject: [PATCH] selftests/bpf: fix array_size.cocci warning:
Date:   Wed, 17 Nov 2021 21:20:21 +0800
Message-Id: <20211117132024.11509-1-guozhengkui@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?l7kOTOTdkNtT2/z4sq0HQijwHP0J8dmfenCY5g9K6q20o91JQ96pPwtQNtPn?=
 =?us-ascii?Q?ohHHadScgo9WPG7IIh9kbM9G9Y+s9H6XbNlOChDT0WrTbkwL2waPs3y5XucE?=
 =?us-ascii?Q?qciIc9NA9+nJ3gIQzrqtPzBzWth21CNPQ6EhqnpKh/5uQ6OEdDB6fanKqTu5?=
 =?us-ascii?Q?ETzWekiuCrfcPEsfdjFX3eKtdIgrrK15Thz6+Kj5z2PpatU4+986uqCopmyz?=
 =?us-ascii?Q?DaIX4OUhWC1vsgPclkIodjBxViRjJRUIxCtB4Zw+QnBABdv898l2rKidJ8kn?=
 =?us-ascii?Q?cjL+TGvuB9F/3VycAs22yCIhk1LUx34Yk8LbKcdXuBMWpfsYanX4jnS3ZZ5F?=
 =?us-ascii?Q?Uel8jy44XdMLUG6KG+Zk4EThCz7jF1zFqewFUSBBRgmaJHApNpuVkliq5Dmg?=
 =?us-ascii?Q?LNf1/ns5U0KjQtXq8bnQ5aX6ZvC+w5bOfSnyf4KRvWILa0c47LxLlH1f2dge?=
 =?us-ascii?Q?lWlEjVsclEu19F0Z7+T1rhEp06e6zDNpDipmmNgCw/bjutOiqUeO719bPeIk?=
 =?us-ascii?Q?KaQMplKYduBVMVi4j+GWXVqtXf39IUfh9XS5Y4U+1TIhSS2P0m41eIV7EJ6+?=
 =?us-ascii?Q?wbVx/Os2jvLX0p5YqBl2wzaPGJ7hJLeAgn6TgB3l7O0vXkjz2mpwd7rxc7bN?=
 =?us-ascii?Q?USXp+mftcNnHUQCIAC5zYPVecZDLNBVtsEI435qFLXsbswFni7LA9aTI0C/F?=
 =?us-ascii?Q?uReR+CDoGFQe09cmXvOOoUhqCFVxDcBa6l+nII5r8jJsrd1JrFbWI66wDDoN?=
 =?us-ascii?Q?mxiEFZXHNr6lpHUWoOQsLS5dKqt2k01HTr9unqBovqSEzLZyi8/myinpaaNJ?=
 =?us-ascii?Q?AqRM/353nqj+GjOXmP1wUuTuXmG6Un9PIfcVYoXI48D3uNyK+4/T60KBkmFb?=
 =?us-ascii?Q?Si9Qedvv5duYmrd0s5abAx1z3M689RJjrARQaSdAL1NkbftLCYBXJo7SKf8D?=
 =?us-ascii?Q?0wjWjS4EQRO95srRL7w/Hm/gVNF2lmszLgJjDNux7QLjIuVKzq0hcJfwC9Ez?=
 =?us-ascii?Q?OMRmWMqS9xPVCV7MeNN4B8MBRW4Neit5hk3KNe+8TTbAICY7QjT8rM9pIzcr?=
 =?us-ascii?Q?WNGwUmxNP1ohxww7nO+Lg6up5MAvDoQDqMWAcNlWqrzuQ8A0+AcGZ2truApc?=
 =?us-ascii?Q?H9a2RcaZSFaZ5H2fv4VoaeOhuFy2CNJeR/wZZoYlzRt/A+8Oh3D9fS9jYdxD?=
 =?us-ascii?Q?K9r6aJw7eIhBjh3TxlKp+3mU1Jo0qUY7SH9JP+h1KsI5hKSWjv40MGktSog3?=
 =?us-ascii?Q?lMZFxpYFeVJvj41BJ31mEGPqf0Cgw2zLejjOpBJvMr2aJZTgkgfKyyUJNrAA?=
 =?us-ascii?Q?yipirs66TqIiC8IWZC5mq6JcshnMc6PPHiQpHQ7mkldl0ML/TGill0QWlbO8?=
 =?us-ascii?Q?c4zah1SiwekiON2oi7NxwQkgkKxNZibf4tY+MI55fHkAWnwglKMxrBO5W7f4?=
 =?us-ascii?Q?54L2a5FRj8hROcDB14pdO/cB7tQJeqJtN770ijm+xdpOWmO2viip+hyHLmct?=
 =?us-ascii?Q?G6A/TvsResvg6VkaI5JRYbkpCHzEjRSajG6V+PHxSqnFJRA+iZcMoxGgLDsM?=
 =?us-ascii?Q?uM5o6ahNKxuxGUVcycFPkOPjoPO0iP+AbGoiQtZacGJzK052OL7ocEIhF4hV?=
 =?us-ascii?Q?DHkj1/zifXRRrnwcIWQctSE=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b0f05955-5f96-4827-24b3-08d9a9cd212f
X-MS-Exchange-CrossTenant-AuthSource: HK2PR06MB3492.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Nov 2021 13:21:14.1462
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: PH1TmmeZClLozx3v/KBYBy5ISOlYuOa7pwmY/rfsZ0QNsDy7F/tRKHvKQqtUi0GR+j2yXlO3V8xeJuu30IgmUQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HK0PR06MB3841
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Use ARRAY_SIZE() because it uses __must_be_array(arr) to make sure
arr is really an array.

Signed-off-by: Guo Zhengkui <guozhengkui@vivo.com>
---
 .../testing/selftests/bpf/prog_tests/cgroup_attach_override.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/cgroup_attach_override.c b/tools/testing/selftests/bpf/prog_tests/cgroup_attach_override.c
index 356547e849e2..1921c5040d8c 100644
--- a/tools/testing/selftests/bpf/prog_tests/cgroup_attach_override.c
+++ b/tools/testing/selftests/bpf/prog_tests/cgroup_attach_override.c
@@ -1,5 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 
+#include <linux/kernel.h>
 #include <test_progs.h>
 
 #include "cgroup_helpers.h"
@@ -16,10 +17,9 @@ static int prog_load(int verdict)
 		BPF_MOV64_IMM(BPF_REG_0, verdict), /* r0 = verdict */
 		BPF_EXIT_INSN(),
 	};
-	size_t insns_cnt = sizeof(prog) / sizeof(struct bpf_insn);
 
 	return bpf_test_load_program(BPF_PROG_TYPE_CGROUP_SKB,
-			       prog, insns_cnt, "GPL", 0,
+			       prog, ARRAY_SIZE(prog), "GPL", 0,
 			       bpf_log_buf, BPF_LOG_BUF_SIZE);
 }
 
-- 
2.20.1