On 11/1/24 4:15 PM, Mirsad Todorovac wrote:
> Coccinelle gives WARNING recommending the use of ARRAY_SIZE() macro definition
> to improve the code readability:
>
> ./tools/testing/selftests/x86/syscall_numbering.c:316:35-36: WARNING: Use
> ARRAY_SIZE
>
> Fixes: 15c82d98a0f78 ("selftests/x86/syscall: Update and extend
> syscall_numbering_64")
> Cc: Thomas Gleixner <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: Borislav Petkov <[email protected]>
> Cc: Dave Hansen <[email protected]>
> Cc: [email protected]
> Cc: "H. Peter Anvin" <[email protected]>
> Cc: Shuah Khan <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Mirsad Todorovac <[email protected]>
Reviewed-by: Muhammad Usama Anjum <[email protected]>
> ---
> v1: initial version.
>
> tools/testing/selftests/x86/syscall_numbering.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/x86/syscall_numbering.c
> b/tools/testing/selftests/x86/syscall_numbering.c
> index 991591718bb0..41c42b7b54a6 100644
> --- a/tools/testing/selftests/x86/syscall_numbering.c
> +++ b/tools/testing/selftests/x86/syscall_numbering.c
> @@ -25,6 +25,7 @@
> #include <sys/mman.h>
>
> #include <linux/ptrace.h>
> +#include "../kselftest.h"
>
> /* Common system call numbers */
> #define SYS_READ 0
> @@ -313,7 +314,7 @@ static void test_syscall_numbering(void)
> * The MSB is supposed to be ignored, so we loop over a few
> * to test that out.
> */
> - for (size_t i = 0; i < sizeof(msbs)/sizeof(msbs[0]); i++) {
> + for (size_t i = 0; i < ARRAY_SIZE(msbs); i++) {
> int msb = msbs[i];
> run("Checking system calls with msb = %d (0x%x)\n",
> msb, msb);
--
BR,
Muhammad Usama Anjum
