kselftest/arm64: fix array_size.cocci warning
authorKang Minchul <tegongkang@gmail.com>
Sat, 5 Nov 2022 07:31:43 +0000 (16:31 +0900)
committerWill Deacon <will@kernel.org>
Tue, 8 Nov 2022 16:05:31 +0000 (16:05 +0000)
Use ARRAY_SIZE to fix the following coccicheck warnings:
tools/testing/selftests/arm64/mte/check_buffer_fill.c:341:20-21:
WARNING: Use ARRAY_SIZE
tools/testing/selftests/arm64/mte/check_buffer_fill.c:35:20-21:
WARNING: Use ARRAY_SIZE
tools/testing/selftests/arm64/mte/check_buffer_fill.c:168:20-21:
WARNING: Use ARRAY_SIZE
tools/testing/selftests/arm64/mte/check_buffer_fill.c:72:20-21:
WARNING: Use ARRAY_SIZE
tools/testing/selftests/arm64/mte/check_buffer_fill.c:369:25-26:
WARNING: Use ARRAY_SIZE

Signed-off-by: Kang Minchul <tegongkang@gmail.com>
Link: https://lore.kernel.org/r/20221105073143.78521-1-tegongkang@gmail.com
Signed-off-by: Will Deacon <will@kernel.org>
tools/testing/selftests/arm64/mte/check_buffer_fill.c

index 75fc482d63b686b47501e1c709710adf3b0d9901..1dbbbd47dd501958dae58e4bc850422df87dcdff 100644 (file)
@@ -32,7 +32,7 @@ static int check_buffer_by_byte(int mem_type, int mode)
        bool err;
 
        mte_switch_mode(mode, MTE_ALLOW_NON_ZERO_TAG);
-       item = sizeof(sizes)/sizeof(int);
+       item = ARRAY_SIZE(sizes);
 
        for (i = 0; i < item; i++) {
                ptr = (char *)mte_allocate_memory(sizes[i], mem_type, 0, true);
@@ -69,7 +69,7 @@ static int check_buffer_underflow_by_byte(int mem_type, int mode,
        char *und_ptr = NULL;
 
        mte_switch_mode(mode, MTE_ALLOW_NON_ZERO_TAG);
-       item = sizeof(sizes)/sizeof(int);
+       item = ARRAY_SIZE(sizes);
        for (i = 0; i < item; i++) {
                ptr = (char *)mte_allocate_memory_tag_range(sizes[i], mem_type, 0,
                                                            underflow_range, 0);
@@ -165,7 +165,7 @@ static int check_buffer_overflow_by_byte(int mem_type, int mode,
        char *over_ptr = NULL;
 
        mte_switch_mode(mode, MTE_ALLOW_NON_ZERO_TAG);
-       item = sizeof(sizes)/sizeof(int);
+       item = ARRAY_SIZE(sizes);
        for (i = 0; i < item; i++) {
                ptr = (char *)mte_allocate_memory_tag_range(sizes[i], mem_type, 0,
                                                            0, overflow_range);
@@ -338,7 +338,7 @@ static int check_buffer_by_block(int mem_type, int mode)
        int i, item, result = KSFT_PASS;
 
        mte_switch_mode(mode, MTE_ALLOW_NON_ZERO_TAG);
-       item = sizeof(sizes)/sizeof(int);
+       item = ARRAY_SIZE(sizes);
        cur_mte_cxt.fault_valid = false;
        for (i = 0; i < item; i++) {
                result = check_buffer_by_block_iterate(mem_type, mode, sizes[i]);
@@ -366,7 +366,7 @@ static int check_memory_initial_tags(int mem_type, int mode, int mapping)
 {
        char *ptr;
        int run, fd;
-       int total = sizeof(sizes)/sizeof(int);
+       int total = ARRAY_SIZE(sizes);
 
        mte_switch_mode(mode, MTE_ALLOW_NON_ZERO_TAG);
        for (run = 0; run < total; run++) {
@@ -404,7 +404,7 @@ int main(int argc, char *argv[])
 {
        int err;
        size_t page_size = getpagesize();
-       int item = sizeof(sizes)/sizeof(int);
+       int item = ARRAY_SIZE(sizes);
 
        sizes[item - 3] = page_size - 1;
        sizes[item - 2] = page_size;