| Message ID | 80fa7078e0645649b6e31be4844a3cffbe67a79b.1639156389.git.skhan@linuxfoundation.org |
|---|---|
| State | Accepted |
| Commit | 6e5eba2e336694b88e3ef6584a25e0723c173ae2 |
| Headers | show |
| Series | selftests: Remove ARRAY_SIZE duplicate defines | expand |
On Fri, Dec 10, 2021 at 10:33:19AM -0700, Shuah Khan wrote: > ARRAY_SIZE is defined in several selftests. Remove definitions from > individual test files and include header file for the define instead. > ARRAY_SIZE define is added in a separate patch to prepare for this > change. > > Remove ARRAY_SIZE from seccomp_benchmark and pickup the one defined in > kselftest.h. > > Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> Good idea to clean these up; thanks! Reviewed-by: Kees Cook <keescook@chromium.org>
diff --git a/tools/testing/selftests/seccomp/seccomp_benchmark.c b/tools/testing/selftests/seccomp/seccomp_benchmark.c index 6e5102a7d7c9..5b5c9d558dee 100644 --- a/tools/testing/selftests/seccomp/seccomp_benchmark.c +++ b/tools/testing/selftests/seccomp/seccomp_benchmark.c @@ -18,7 +18,7 @@ #include <sys/syscall.h> #include <sys/types.h> -#define ARRAY_SIZE(a) (sizeof(a) / sizeof(a[0])) +#include "../kselftest.h" unsigned long long timing(clockid_t clk_id, unsigned long long samples) {
ARRAY_SIZE is defined in several selftests. Remove definitions from individual test files and include header file for the define instead. ARRAY_SIZE define is added in a separate patch to prepare for this change. Remove ARRAY_SIZE from seccomp_benchmark and pickup the one defined in kselftest.h. Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> --- tools/testing/selftests/seccomp/seccomp_benchmark.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)