* [PATCH v2 0/2] drop duplicate max/min definitions @ 2022-04-08 13:36 Geliang Tang 2022-04-08 13:36 ` [PATCH v2 1/2] selftests: bpf: " Geliang Tang 2022-04-08 13:36 ` [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition Geliang Tang 0 siblings, 2 replies; 7+ messages in thread From: Geliang Tang @ 2022-04-08 13:36 UTC (permalink / raw) To: Shuah Khan, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: Geliang Tang, bpf, linux-kselftest Two small cleanups for selftests, drop duplicate max/min definitions. v2: - do more cleanups as Daniel suggested. v1: - "selftests: bpf: use MIN for TCP CC tests" Geliang Tang (2): selftests: bpf: drop duplicate max/min definitions selftests: mqueue: drop duplicate min definition tools/testing/selftests/bpf/prog_tests/bpf_iter.c | 5 ++--- tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c | 7 +++---- tools/testing/selftests/bpf/prog_tests/snprintf.c | 5 ++--- tools/testing/selftests/bpf/prog_tests/tc_redirect.c | 2 +- tools/testing/selftests/mqueue/mq_perf_tests.c | 4 ++-- 5 files changed, 10 insertions(+), 13 deletions(-) -- 2.34.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] selftests: bpf: drop duplicate max/min definitions 2022-04-08 13:36 [PATCH v2 0/2] drop duplicate max/min definitions Geliang Tang @ 2022-04-08 13:36 ` Geliang Tang 2022-04-08 19:15 ` Shuah Khan 2022-04-08 20:31 ` Daniel Borkmann 2022-04-08 13:36 ` [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition Geliang Tang 1 sibling, 2 replies; 7+ messages in thread From: Geliang Tang @ 2022-04-08 13:36 UTC (permalink / raw) To: Shuah Khan, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: Geliang Tang, bpf, linux-kselftest Drop duplicate macros min() and MAX() definitions in prog_tests, use MIN() or MAX() in sys/param.h instead. Signed-off-by: Geliang Tang <geliang.tang@suse.com> --- tools/testing/selftests/bpf/prog_tests/bpf_iter.c | 5 ++--- tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c | 7 +++---- tools/testing/selftests/bpf/prog_tests/snprintf.c | 5 ++--- tools/testing/selftests/bpf/prog_tests/tc_redirect.c | 2 +- 4 files changed, 8 insertions(+), 11 deletions(-) diff --git a/tools/testing/selftests/bpf/prog_tests/bpf_iter.c b/tools/testing/selftests/bpf/prog_tests/bpf_iter.c index 5142a7d130b2..86561c0b0dea 100644 --- a/tools/testing/selftests/bpf/prog_tests/bpf_iter.c +++ b/tools/testing/selftests/bpf/prog_tests/bpf_iter.c @@ -1,5 +1,6 @@ // SPDX-License-Identifier: GPL-2.0 /* Copyright (c) 2020 Facebook */ +#include <sys/param.h> #include <test_progs.h> #include "bpf_iter_ipv6_route.skel.h" #include "bpf_iter_netlink.skel.h" @@ -1192,8 +1193,6 @@ static void str_strip_first_line(char *str) *dst = '\0'; } -#define min(a, b) ((a) < (b) ? (a) : (b)) - static void test_task_vma(void) { int err, iter_fd = -1, proc_maps_fd = -1; @@ -1229,7 +1228,7 @@ static void test_task_vma(void) len = 0; while (len < CMP_BUFFER_SIZE) { err = read_fd_into_buffer(iter_fd, task_vma_output + len, - min(read_size, CMP_BUFFER_SIZE - len)); + MIN(read_size, CMP_BUFFER_SIZE - len)); if (!err) break; if (CHECK(err < 0, "read_iter_fd", "read_iter_fd failed\n")) diff --git a/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c b/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c index 8f7a1cef7d87..ceed369361fc 100644 --- a/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c +++ b/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c @@ -3,6 +3,7 @@ #include <linux/err.h> #include <netinet/tcp.h> +#include <sys/param.h> #include <test_progs.h> #include "network_helpers.h" #include "bpf_dctcp.skel.h" @@ -10,8 +11,6 @@ #include "bpf_tcp_nogpl.skel.h" #include "bpf_dctcp_release.skel.h" -#define min(a, b) ((a) < (b) ? (a) : (b)) - #ifndef ENOTSUPP #define ENOTSUPP 524 #endif @@ -53,7 +52,7 @@ static void *server(void *arg) while (bytes < total_bytes && !READ_ONCE(stop)) { nr_sent = send(fd, &batch, - min(total_bytes - bytes, sizeof(batch)), 0); + MIN(total_bytes - bytes, sizeof(batch)), 0); if (nr_sent == -1 && errno == EINTR) continue; if (nr_sent == -1) { @@ -146,7 +145,7 @@ static void do_test(const char *tcp_ca, const struct bpf_map *sk_stg_map) /* recv total_bytes */ while (bytes < total_bytes && !READ_ONCE(stop)) { nr_recv = recv(fd, &batch, - min(total_bytes - bytes, sizeof(batch)), 0); + MIN(total_bytes - bytes, sizeof(batch)), 0); if (nr_recv == -1 && errno == EINTR) continue; if (nr_recv == -1) diff --git a/tools/testing/selftests/bpf/prog_tests/snprintf.c b/tools/testing/selftests/bpf/prog_tests/snprintf.c index 394ebfc3bbf3..5ca70aa15c4a 100644 --- a/tools/testing/selftests/bpf/prog_tests/snprintf.c +++ b/tools/testing/selftests/bpf/prog_tests/snprintf.c @@ -1,6 +1,7 @@ // SPDX-License-Identifier: GPL-2.0 /* Copyright (c) 2021 Google LLC. */ +#include <sys/param.h> #include <test_progs.h> #include "test_snprintf.skel.h" #include "test_snprintf_single.skel.h" @@ -83,8 +84,6 @@ static void test_snprintf_positive(void) test_snprintf__destroy(skel); } -#define min(a, b) ((a) < (b) ? (a) : (b)) - /* Loads an eBPF object calling bpf_snprintf with up to 10 characters of fmt */ static int load_single_snprintf(char *fmt) { @@ -95,7 +94,7 @@ static int load_single_snprintf(char *fmt) if (!skel) return -EINVAL; - memcpy(skel->rodata->fmt, fmt, min(strlen(fmt) + 1, 10)); + memcpy(skel->rodata->fmt, fmt, MIN(strlen(fmt) + 1, 10)); ret = test_snprintf_single__load(skel); test_snprintf_single__destroy(skel); diff --git a/tools/testing/selftests/bpf/prog_tests/tc_redirect.c b/tools/testing/selftests/bpf/prog_tests/tc_redirect.c index 7ad66a247c02..52f1b9139145 100644 --- a/tools/testing/selftests/bpf/prog_tests/tc_redirect.c +++ b/tools/testing/selftests/bpf/prog_tests/tc_redirect.c @@ -20,6 +20,7 @@ #include <stdbool.h> #include <stdio.h> #include <sys/stat.h> +#include <sys/param.h> #include <unistd.h> #include "test_progs.h" @@ -949,7 +950,6 @@ static int tun_open(char *name) return -1; } -#define MAX(a, b) ((a) > (b) ? (a) : (b)) enum { SRC_TO_TARGET = 0, TARGET_TO_SRC = 1, -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] selftests: bpf: drop duplicate max/min definitions 2022-04-08 13:36 ` [PATCH v2 1/2] selftests: bpf: " Geliang Tang @ 2022-04-08 19:15 ` Shuah Khan 2022-04-08 20:31 ` Daniel Borkmann 1 sibling, 0 replies; 7+ messages in thread From: Shuah Khan @ 2022-04-08 19:15 UTC (permalink / raw) To: Geliang Tang, Shuah Khan, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: bpf, linux-kselftest, Shuah Khan On 4/8/22 7:36 AM, Geliang Tang wrote: > Drop duplicate macros min() and MAX() definitions in prog_tests, use MIN() > or MAX() in sys/param.h instead. > > Signed-off-by: Geliang Tang <geliang.tang@suse.com> > --- > tools/testing/selftests/bpf/prog_tests/bpf_iter.c | 5 ++--- > tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c | 7 +++---- > tools/testing/selftests/bpf/prog_tests/snprintf.c | 5 ++--- > tools/testing/selftests/bpf/prog_tests/tc_redirect.c | 2 +- > 4 files changed, 8 insertions(+), 11 deletions(-) > Looks good to me. Thanks for cleaning these up. Reviewed-by: Shuah Khan <skhan@linuxfoundation.org> thanks, -- Shuah ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] selftests: bpf: drop duplicate max/min definitions 2022-04-08 13:36 ` [PATCH v2 1/2] selftests: bpf: " Geliang Tang 2022-04-08 19:15 ` Shuah Khan @ 2022-04-08 20:31 ` Daniel Borkmann 1 sibling, 0 replies; 7+ messages in thread From: Daniel Borkmann @ 2022-04-08 20:31 UTC (permalink / raw) To: Geliang Tang, Shuah Khan, Alexei Starovoitov, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: bpf, linux-kselftest On 4/8/22 3:36 PM, Geliang Tang wrote: > Drop duplicate macros min() and MAX() definitions in prog_tests, use MIN() > or MAX() in sys/param.h instead. > > Signed-off-by: Geliang Tang <geliang.tang@suse.com> [...] Thanks Geliang! One small final nit and then it's good to go to bpf-next: > diff --git a/tools/testing/selftests/bpf/prog_tests/bpf_iter.c b/tools/testing/selftests/bpf/prog_tests/bpf_iter.c > index 5142a7d130b2..86561c0b0dea 100644 > --- a/tools/testing/selftests/bpf/prog_tests/bpf_iter.c > +++ b/tools/testing/selftests/bpf/prog_tests/bpf_iter.c > @@ -1,5 +1,6 @@ > // SPDX-License-Identifier: GPL-2.0 > /* Copyright (c) 2020 Facebook */ > +#include <sys/param.h> > #include <test_progs.h> > #include "bpf_iter_ipv6_route.skel.h" > #include "bpf_iter_netlink.skel.h" [...] > diff --git a/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c b/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c > index 8f7a1cef7d87..ceed369361fc 100644 > --- a/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c > +++ b/tools/testing/selftests/bpf/prog_tests/bpf_tcp_ca.c > @@ -3,6 +3,7 @@ > > #include <linux/err.h> > #include <netinet/tcp.h> > +#include <sys/param.h> > #include <test_progs.h> > #include "network_helpers.h" > #include "bpf_dctcp.skel.h" [...] > diff --git a/tools/testing/selftests/bpf/prog_tests/snprintf.c b/tools/testing/selftests/bpf/prog_tests/snprintf.c > index 394ebfc3bbf3..5ca70aa15c4a 100644 > --- a/tools/testing/selftests/bpf/prog_tests/snprintf.c > +++ b/tools/testing/selftests/bpf/prog_tests/snprintf.c > @@ -1,6 +1,7 @@ > // SPDX-License-Identifier: GPL-2.0 > /* Copyright (c) 2021 Google LLC. */ > > +#include <sys/param.h> > #include <test_progs.h> > #include "test_snprintf.skel.h" > #include "test_snprintf_single.skel.h" > @@ -83,8 +84,6 @@ static void test_snprintf_positive(void) [...] > diff --git a/tools/testing/selftests/bpf/prog_tests/tc_redirect.c b/tools/testing/selftests/bpf/prog_tests/tc_redirect.c > index 7ad66a247c02..52f1b9139145 100644 > --- a/tools/testing/selftests/bpf/prog_tests/tc_redirect.c > +++ b/tools/testing/selftests/bpf/prog_tests/tc_redirect.c > @@ -20,6 +20,7 @@ > #include <stdbool.h> > #include <stdio.h> > #include <sys/stat.h> > +#include <sys/param.h> > #include <unistd.h> > > #include "test_progs.h" Just add the sys/param.h to the test_progs.h header in BPF selftests where we also pull in other sys headers; then all the duplicate includes above are not needed. Thanks, Daniel ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition 2022-04-08 13:36 [PATCH v2 0/2] drop duplicate max/min definitions Geliang Tang 2022-04-08 13:36 ` [PATCH v2 1/2] selftests: bpf: " Geliang Tang @ 2022-04-08 13:36 ` Geliang Tang 2022-04-08 19:23 ` Shuah Khan 1 sibling, 1 reply; 7+ messages in thread From: Geliang Tang @ 2022-04-08 13:36 UTC (permalink / raw) To: Shuah Khan, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: Geliang Tang, bpf, linux-kselftest Drop duplicate macro min() definition in mq_perf_tests.c, use MIN() in sys/param.h instead. Signed-off-by: Geliang Tang <geliang.tang@suse.com> --- tools/testing/selftests/mqueue/mq_perf_tests.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/mqueue/mq_perf_tests.c b/tools/testing/selftests/mqueue/mq_perf_tests.c index b019e0b8221c..6df1b2a7ec88 100644 --- a/tools/testing/selftests/mqueue/mq_perf_tests.c +++ b/tools/testing/selftests/mqueue/mq_perf_tests.c @@ -35,6 +35,7 @@ #include <sys/time.h> #include <sys/resource.h> #include <sys/stat.h> +#include <sys/param.h> #include <mqueue.h> #include <popt.h> #include <error.h> @@ -73,7 +74,6 @@ static char *usage = char *MAX_MSGS = "/proc/sys/fs/mqueue/msg_max"; char *MAX_MSGSIZE = "/proc/sys/fs/mqueue/msgsize_max"; -#define min(a, b) ((a) < (b) ? (a) : (b)) #define MAX_CPUS 64 char *cpu_option_string; int cpus_to_pin[MAX_CPUS]; @@ -551,7 +551,7 @@ int main(int argc, char *argv[]) perror("sysconf(_SC_NPROCESSORS_ONLN)"); exit(1); } - cpus_online = min(MAX_CPUS, sysconf(_SC_NPROCESSORS_ONLN)); + cpus_online = MIN(MAX_CPUS, sysconf(_SC_NPROCESSORS_ONLN)); cpu_set = CPU_ALLOC(cpus_online); if (cpu_set == NULL) { perror("CPU_ALLOC()"); -- 2.34.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition 2022-04-08 13:36 ` [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition Geliang Tang @ 2022-04-08 19:23 ` Shuah Khan 2022-04-13 10:51 ` Geliang Tang 0 siblings, 1 reply; 7+ messages in thread From: Shuah Khan @ 2022-04-08 19:23 UTC (permalink / raw) To: Geliang Tang, Shuah Khan, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh Cc: bpf, linux-kselftest, Shuah Khan On 4/8/22 7:36 AM, Geliang Tang wrote: > Drop duplicate macro min() definition in mq_perf_tests.c, use MIN() in > sys/param.h instead. > > Signed-off-by: Geliang Tang <geliang.tang@suse.com> > --- Thank you. I will apply this for next. thanks, -- Shuah ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition 2022-04-08 19:23 ` Shuah Khan @ 2022-04-13 10:51 ` Geliang Tang 0 siblings, 0 replies; 7+ messages in thread From: Geliang Tang @ 2022-04-13 10:51 UTC (permalink / raw) To: Shuah Khan Cc: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song, John Fastabend, KP Singh, bpf, linux-kselftest On Fri, Apr 08, 2022 at 01:23:52PM -0600, Shuah Khan wrote: > On 4/8/22 7:36 AM, Geliang Tang wrote: > > Drop duplicate macro min() definition in mq_perf_tests.c, use MIN() in > > sys/param.h instead. > > > > Signed-off-by: Geliang Tang <geliang.tang@suse.com> > > --- > > Thank you. I will apply this for next. Thanks very much. I had sent a v3 of this patch, but it's the same as this one. So apply any one is OK. Thanks, -Geliang > > thanks, > -- Shuah > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-04-13 10:51 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-04-08 13:36 [PATCH v2 0/2] drop duplicate max/min definitions Geliang Tang 2022-04-08 13:36 ` [PATCH v2 1/2] selftests: bpf: " Geliang Tang 2022-04-08 19:15 ` Shuah Khan 2022-04-08 20:31 ` Daniel Borkmann 2022-04-08 13:36 ` [PATCH v2 2/2] selftests: mqueue: drop duplicate min definition Geliang Tang 2022-04-08 19:23 ` Shuah Khan 2022-04-13 10:51 ` Geliang Tang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).