From nobody Mon Sep 28 18:33:17 2026 Received: from mta1.migadu.com (out-252.mta1.migadu.com [95.215.58.252]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 550E73812FB for ; Tue, 18 Aug 2026 15:32:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.252 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787067138; cv=none; b=YdDLHtBmuBMjZJu6/RcoewyL7bEdJGd9nRT8dI/TwGp7NO52XDYQPsAyCBFi8WvzZcPRQpjbGuIN6qzPAwxPZ1RUNH7W2bmi6HxjBN1p/GI/1xBZzCC0wNrUiKznAXGcl3g/VOoUaPzIda6Xpe9Ot9ZjRZ9LU2EnUcLfoXOrLwI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787067138; c=relaxed/simple; bh=/f0b5nqYvRuYRrnGgcST9+/5ojWfBeBx/7ogSsZtBdM=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=MYV0P12q2vQBq0CZw68w7HUjnf2g8zv/Q132Fqqg+AWTeXqVmiuZpCh1anupYxgUO95hLy1O6ZWLVC9Wc8z8cglTky08hXv+uUiin2Ptdud013RyTV3zDenWa9oXvEtZzovElPqtZLYaBX7bRSlrcoUfFx08Se4EVYFnfl5mQyM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=rD3dF5i+; arc=none smtp.client-ip=95.215.58.252 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="rD3dF5i+" X-Envelope-To: linux-kernel@vger.kernel.org DKIM-Signature: a=rsa-sha256; bh=/f0b5nqYvRuYRrnGgcST9+/5ojWfBeBx/7ogSsZtBdM=; c=simple/simple; d=linux.dev; h=from:to:subject:date:message-id:mime-version:content-type; s=key1; t=1787067134; v=1; x=1787671934; b=rD3dF5i+d1XXkZUx5ciE56C+7Apl5g/Ls3zbpVL7WRjMkw4Qukkye2Y8ukXUFrLdA6aAKLc4 yqK/2jpfTs1HxEwjRz51tDT+syAmEEBoLCxmV5ZgqtpSltb7S+uqWNvdju6eTWF7VlvZqOsOL7j 4ARO+m+FTRVGd6kG7/8Qz+3g= X-Envelope-To: linux-kernel@vger.kernel.org Received: from .linux.dev (117.20.149.64) by smtp.migadu.com with ESMTPS id 07ad087d0c7f2357; Tue, 18 Aug 2026 15:32:13 +0000 X-Migadu-Flow: FLOW_OUT From: Leon Hwang To: bpf@vger.kernel.org Cc: Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Eduard Zingerman , Kumar Kartikeya Dwivedi , Martin KaFai Lau , Song Liu , Yonghong Song , Jiri Olsa , Emil Tsalapatis , Ihor Solodrai , Shuah Khan , Leon Hwang , linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH bpf-next v2] selftests/bpf: Avoid flaky resize value test for percpu data Date: Tue, 18 Aug 2026 23:32:01 +0800 Message-ID: <20260818153202.90714-1-leon.hwang@linux.dev> X-Mailer: git-send-email 2.55.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The bpf_map__set_value_size() test for percpu data relies on the tail 'cpu_id' field in the ".percpu" map. The test would fail, when 'cpu_id' is not the last field in the map. To avoid the flaky test, factor out a subtest to use a dedicated ".percpu.arr" map that only has one field 'int arr[1]' to ensure that bpf_map__set_value_size() won't fail on 4K page systems. And, drop 'args' in test_percpu_data_on_cpus(), which isn't used in bpf prog side. And, enhance the test to cover 'set' and 'nums[6]'. Fixes: 4c9241bd731a ("selftests/bpf: Add tests to verify global percpu data= ") Signed-off-by: Leon Hwang --- v1 -> v2: * Factor out a subtest to test bpf_map__set_value_size(). * Run the new subtest only on 4K page systems. (Sashiko) * v1: https://lore.kernel.org/bpf/20260814173206.93082-5-leon.hwang@linux.d= ev/ --- .../bpf/prog_tests/global_data_init.c | 86 ++++++++++++++----- .../bpf/progs/test_global_percpu_data.c | 10 ++- 2 files changed, 70 insertions(+), 26 deletions(-) diff --git a/tools/testing/selftests/bpf/prog_tests/global_data_init.c b/to= ols/testing/selftests/bpf/prog_tests/global_data_init.c index 5671c31085cd..2b69290bcc3e 100644 --- a/tools/testing/selftests/bpf/prog_tests/global_data_init.c +++ b/tools/testing/selftests/bpf/prog_tests/global_data_init.c @@ -68,12 +68,9 @@ static void test_percpu_data_on_cpus(struct bpf_map *map= , int map_fd, int prog_f { struct test_global_percpu_data__percpu *data =3D NULL; int i, err, key =3D 0, num_online, run =3D 0; - __u64 args[2] =3D {0x1234ULL, 0x5678ULL}; size_t data_sz; bool *online; LIBBPF_OPTS(bpf_test_run_opts, topts, - .ctx_in =3D args, - .ctx_size_in =3D sizeof(args), .flags =3D BPF_F_TEST_RUN_ON_CPU, ); =20 @@ -81,7 +78,7 @@ static void test_percpu_data_on_cpus(struct bpf_map *map,= int map_fd, int prog_f if (!ASSERT_OK(err, "parse_cpu_mask_file")) return; =20 - data_sz =3D map ? bpf_map__value_size(map) : sizeof(*data); + data_sz =3D sizeof(*data); data =3D calloc(1, data_sz); if (!ASSERT_OK_PTR(data, "calloc percpu data")) goto out; @@ -109,7 +106,7 @@ static void test_percpu_data_on_cpus(struct bpf_map *ma= p, int map_fd, int prog_f break; =20 ASSERT_EQ(*runp, ++run, "run"); - ASSERT_EQ(data->cpu_id[0], i, "cpu_id"); + ASSERT_EQ(data->cpu_id, i, "cpu_id"); ASSERT_EQ(data->data, 1, "data"); ASSERT_TRUE(data->set, "set"); ASSERT_EQ(data->nums[6], 0xc0de, "nums[6]"); @@ -127,7 +124,6 @@ static void test_global_percpu_data_init(void) { struct test_global_percpu_data__percpu init_value =3D {}; struct test_global_percpu_data__percpu *init_data; - const __u32 desired_sz =3D sysconf(_SC_PAGE_SIZE); struct test_global_percpu_data *skel =3D NULL; size_t init_data_sz; struct bpf_map *map; @@ -163,9 +159,11 @@ static void test_global_percpu_data_init(void) if (!ASSERT_EQ(bpf_map__type(map), BPF_MAP_TYPE_PERCPU_ARRAY, "bpf_map__t= ype")) goto out; =20 + init_value.set =3D 1; init_value.data =3D 2; init_value.nums[6] =3D -1; init_value.struct_data.i =3D 2; + init_value.struct_data.set =3D 1; init_value.struct_data.nums[6] =3D -1; err =3D bpf_map__set_initial_value(map, &init_value, sizeof(init_value)); if (!ASSERT_OK(err, "bpf_map__set_initial_value")) @@ -177,35 +175,24 @@ static void test_global_percpu_data_init(void) =20 ASSERT_EQ(init_data->data, init_value.data, "init_value data"); ASSERT_EQ(init_data->set, init_value.set, "init_value set"); + ASSERT_EQ(init_data->nums[6], init_value.nums[6], "init_value nums[6]"); ASSERT_EQ(init_data->struct_data.i, init_value.struct_data.i, "init_value= struct_data.i"); + ASSERT_EQ(init_data->struct_data.set, init_value.struct_data.set, + "init_value struct_data.set"); ASSERT_EQ(init_data->struct_data.nums[6], init_value.struct_data.nums[6], "init_value struct_data.nums[6]"); ASSERT_EQ(init_data_sz, sizeof(init_value), "init_value size"); ASSERT_EQ((void *) init_data, (void *) skel->percpu, "skel->percpu eq ini= t_data"); ASSERT_EQ(skel->percpu->data, init_value.data, "skel->percpu->data"); ASSERT_EQ(skel->percpu->set, init_value.set, "skel->percpu->set"); + ASSERT_EQ(skel->percpu->nums[6], init_value.nums[6], "skel->percpu->nums[= 6]"); ASSERT_EQ(skel->percpu->struct_data.i, init_value.struct_data.i, "skel->percpu->struct_data.i"); + ASSERT_EQ(skel->percpu->struct_data.set, init_value.struct_data.set, + "skel->percpu->struct_data.set"); ASSERT_EQ(skel->percpu->struct_data.nums[6], init_value.struct_data.nums[= 6], "skel->percpu->struct_data.nums[6]"); =20 - ASSERT_GT(desired_sz, sizeof(init_value), "desired_sz"); - err =3D bpf_map__set_value_size(map, desired_sz); - if (!ASSERT_OK(err, "bpf_map__set_value_size")) - goto out; - if (!ASSERT_EQ(bpf_map__value_size(map), desired_sz, "percpu value size")) - goto out; - if (!ASSERT_NEQ(bpf_map__btf_value_type_id(map), 0, "percpu BTF value typ= e")) - goto out; - - init_data =3D bpf_map__initial_value(map, &init_data_sz); - if (!ASSERT_OK_PTR(init_data, "resized bpf_map__initial_value")) - goto out; - if (!ASSERT_EQ(init_data_sz, desired_sz, "resized initial value size")) - goto out; - if (!ASSERT_EQ(init_data->data, init_value.data, "resized initial value d= ata")) - goto out; - err =3D test_global_percpu_data__load(skel); if (!ASSERT_OK(err, "test_global_percpu_data__load")) goto out; @@ -236,6 +223,57 @@ static void test_global_percpu_data_lskel(void) test_global_percpu_data_lskel__destroy(lskel); } =20 +#define PAGE_SIZE 4096 + +static void test_global_percpu_data_map_resize(void) +{ + const __u32 desired_sz =3D sysconf(_SC_PAGE_SIZE) * 2; + struct test_global_percpu_data *skel; + const int arr_value =3D -1; + struct bpf_map *map; + int err, prog_fd; + size_t data_sz; + + if (sysconf(_SC_PAGE_SIZE) !=3D PAGE_SIZE) { + test__skip(); + return; + } + + skel =3D test_global_percpu_data__open(); + if (!ASSERT_OK_PTR(skel, "test_global_percpu_data__open")) + return; + + map =3D skel->maps.percpu_arr; + skel->percpu_arr->arr[0] =3D arr_value; + err =3D bpf_map__set_value_size(map, desired_sz); + if (!ASSERT_OK(err, "bpf_map__set_value_size")) + goto out; + if (!ASSERT_EQ(bpf_map__value_size(map), desired_sz, "bpf_map__value_size= ")) + goto out; + if (!ASSERT_NEQ(bpf_map__btf_value_type_id(map), 0, "bpf_map__btf_value_t= ype_id")) + goto out; + + skel->percpu_arr =3D bpf_map__initial_value(map, &data_sz); + if (!ASSERT_EQ(data_sz, desired_sz, "data_sz")) + goto out; + if (!ASSERT_OK_PTR(skel->percpu_arr, "skel->percpu_arr")) + goto out; + if (!ASSERT_EQ(skel->percpu_arr->arr[0], arr_value, "arr[0]")) + goto out; + + err =3D test_global_percpu_data__load(skel); + if (!ASSERT_OK(err, "test_global_percpu_data__load")) + goto out; + + map =3D skel->maps.percpu; + prog_fd =3D bpf_program__fd(skel->progs.update_percpu_data); + test_percpu_data_on_cpus(map, bpf_map__fd(map), prog_fd, &skel->bss->run); + ASSERT_EQ(skel->bss->arr_sum, arr_value * skel->bss->run, "arr_sum"); + +out: + test_global_percpu_data__destroy(skel); +} + static int create_rdonly_percpu_array(void) { LIBBPF_OPTS(bpf_map_create_opts, map_opts, @@ -388,6 +426,8 @@ void test_global_percpu_data(void) test_global_percpu_data_init(); if (test__start_subtest("lskel")) test_global_percpu_data_lskel(); + if (test__start_subtest("map_resize")) + test_global_percpu_data_map_resize(); if (test__start_subtest("rdonly_direct_read")) test_global_percpu_data_rdonly_direct_read(); if (test__start_subtest("rdonly_direct_write")) diff --git a/tools/testing/selftests/bpf/progs/test_global_percpu_data.c b/= tools/testing/selftests/bpf/progs/test_global_percpu_data.c index 5dc21b3b4cb5..2765cd46e9af 100644 --- a/tools/testing/selftests/bpf/progs/test_global_percpu_data.c +++ b/tools/testing/selftests/bpf/progs/test_global_percpu_data.c @@ -8,9 +8,12 @@ int loong SEC(".percpu.looooooooong"); int data3 SEC(".data.percpu"); int data2 SEC(".percpu.data"); =20 +/* Used for testing bpf_map__set_value_size(). */ +int arr[1] SEC(".percpu.arr"); +int arr_sum; + int run; -/* cpu_id as array to verify map value resizing. */ -int cpu_id[1] SEC(".percpu"); +int cpu_id SEC(".percpu"); int data SEC(".percpu") =3D -1; int nums[7] SEC(".percpu"); bool set SEC(".percpu") =3D false; @@ -34,7 +37,8 @@ int update_percpu_data(void *ctx) data =3D 1; run++; set =3D true; - cpu_id[0] =3D bpf_get_smp_processor_id(); + cpu_id =3D bpf_get_smp_processor_id(); + arr_sum +=3D arr[0]; return 0; } =20 --=20 2.55.0