Skip to content

Commit 3d5367a

Browse files
Dan Carpenterakpm00
authored andcommitted
tools/testing/selftests/vm/hugetlb-madvise.c: silence uninitialized variable warning
This code just reads from memory without caring about the data itself. However static checkers complain that "tmp" is never properly initialized. Initialize it to zero and change the name to "dummy" to show that we don't care about the value stored in it. Link: https://lkml.kernel.org/r/YtZ8mKJmktA2GaHB@kili Fixes: c4b6cb8 ("selftests/vm: add hugetlb madvise MADV_DONTNEED MADV_REMOVE test") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Souptick Joarder (HPE) <jrdr.linux@gmail.com> Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com> Cc: Shuah Khan <shuah@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
1 parent 6d97cf8 commit 3d5367a

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

tools/testing/selftests/vm/hugetlb-madvise.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,11 @@ void write_fault_pages(void *addr, unsigned long nr_pages)
8989

9090
void read_fault_pages(void *addr, unsigned long nr_pages)
9191
{
92-
unsigned long i, tmp;
92+
unsigned long dummy = 0;
93+
unsigned long i;
9394

9495
for (i = 0; i < nr_pages; i++)
95-
tmp += *((unsigned long *)(addr + (i * huge_page_size)));
96+
dummy += *((unsigned long *)(addr + (i * huge_page_size)));
9697
}
9798

9899
int main(int argc, char **argv)

0 commit comments

Comments
 (0)