[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAEf4BzYHycMkdkYdNWjd+XOaHUfJnmbAAKOh_Gu1WpN=MJZ-wA@mail.gmail.com>
Date: Thu, 2 Dec 2021 12:11:45 -0800
From: Andrii Nakryiko <andrii.nakryiko@...il.com>
To: "Paul E . McKenney" <paulmck@...nel.org>
Cc: "open list:KERNEL SELFTEST FRAMEWORK"
<linux-kselftest@...r.kernel.org>,
Networking <netdev@...r.kernel.org>, bpf <bpf@...r.kernel.org>,
open list <linux-kernel@...r.kernel.org>,
Alexei Starovoitov <ast@...nel.org>,
Shuah Khan <shuah@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
Andrii Nakryiko <andrii@...nel.org>, Martin Lau <kafai@...com>,
Song Liu <songliubraving@...com>, Yonghong Song <yhs@...com>,
john fastabend <john.fastabend@...il.com>,
KP Singh <kpsingh@...nel.org>
Subject: Re: [PATCH] testing/bpf: Update test names for xchg and cmpxchg
On Tue, Nov 30, 2021 at 4:50 PM Paul E. McKenney <paulmck@...nel.org> wrote:
>
> The test_cmpxchg() and test_xchg() functions say "test_run add".
> Therefore, make them say "test_run cmpxchg" and "test_run xchg",
> respectively.
>
> Cc: Shuah Khan <shuah@...nel.org>
> Cc: Alexei Starovoitov <ast@...nel.org>
> Cc: Daniel Borkmann <daniel@...earbox.net>
> Cc: Andrii Nakryiko <andrii@...nel.org>
> Cc: Martin KaFai Lau <kafai@...com>
> Cc: Song Liu <songliubraving@...com>
> Cc: Yonghong Song <yhs@...com>
> Cc: John Fastabend <john.fastabend@...il.com>
> Cc: KP Singh <kpsingh@...nel.org>
> Cc: <linux-kselftest@...r.kernel.org>
> Cc: <netdev@...r.kernel.org>
> Cc: <bpf@...r.kernel.org>
> Cc: <linux-kernel@...r.kernel.org>
> Signed-off-by: Paul E. McKenney <paulmck@...nel.org>
>
fixed up "testing/bpf" to more commonly used "selftests/bpf" and
pushed to bpf-next, thanks.
> diff --git a/tools/testing/selftests/bpf/prog_tests/atomics.c b/tools/testing/selftests/bpf/prog_tests/atomics.c
> index 0f9525293881b..86b7d5d84eec4 100644
> --- a/tools/testing/selftests/bpf/prog_tests/atomics.c
> +++ b/tools/testing/selftests/bpf/prog_tests/atomics.c
> @@ -167,7 +167,7 @@ static void test_cmpxchg(struct atomics_lskel *skel)
> prog_fd = skel->progs.cmpxchg.prog_fd;
> err = bpf_prog_test_run(prog_fd, 1, NULL, 0,
> NULL, NULL, &retval, &duration);
> - if (CHECK(err || retval, "test_run add",
> + if (CHECK(err || retval, "test_run cmpxchg",
> "err %d errno %d retval %d duration %d\n", err, errno, retval, duration))
> goto cleanup;
>
> @@ -196,7 +196,7 @@ static void test_xchg(struct atomics_lskel *skel)
> prog_fd = skel->progs.xchg.prog_fd;
> err = bpf_prog_test_run(prog_fd, 1, NULL, 0,
> NULL, NULL, &retval, &duration);
> - if (CHECK(err || retval, "test_run add",
> + if (CHECK(err || retval, "test_run xchg",
> "err %d errno %d retval %d duration %d\n", err, errno, retval, duration))
> goto cleanup;
>
Powered by blists - more mailing lists