The hardware breakpoint counters breaks, needed, breaks1 and breaks2 are unsigned long long, but perf-hwbreak.c prints them using signed integer format specifiers.
Use %llu to match their types and avoid interpreting large counter values as negative numbers. Signed-off-by: zhangchujun <[email protected]> diff --git a/tools/testing/selftests/powerpc/ptrace/perf-hwbreak.c b/tools/testing/selftests/powerpc/ptrace/perf-hwbreak.c index e374c6b7ace6..5b255c1fa67e 100644 --- a/tools/testing/selftests/powerpc/ptrace/perf-hwbreak.c +++ b/tools/testing/selftests/powerpc/ptrace/perf-hwbreak.c @@ -277,10 +277,10 @@ static int runtestsingle(int readwriteflag, int exclude_user, int arraytest) if (readwriteflag & HW_BREAKPOINT_W) needed += loop_num; needed = needed * (1 - exclude_user); - printf("TESTED: addr:0x%lx brks:% 8lld loops:% 8i rw:%i !user:%i array:%i\n", + printf("TESTED: addr:0x%lx brks:% 8llu loops:% 8i rw:%i !user:%i array:%i\n", (unsigned long int)ptr, breaks, loop_num, readwriteflag, exclude_user, arraytest); if (breaks != needed) { - printf("FAILED: 0x%lx brks:%lld needed:%lli %i %i %i\n\n", + printf("FAILED: 0x%lx brks:%llu needed:%llu %i %i %i\n\n", (unsigned long int)ptr, breaks, needed, loop_num, readwriteflag, exclude_user); return 1; } @@ -324,7 +324,7 @@ static int runtest_dar_outside(void) if (breaks == 0) { printf("TESTED: No overlap\n"); } else { - printf("FAILED: No overlap: %lld != 0\n", breaks); + printf("FAILED: No overlap: %llu != 0\n", breaks); fail = 1; } @@ -339,7 +339,7 @@ static int runtest_dar_outside(void) if (breaks == 2) { printf("TESTED: Partial overlap\n"); } else { - printf("FAILED: Partial overlap: %lld != 2\n", breaks); + printf("FAILED: Partial overlap: %llu != 2\n", breaks); fail = 1; } @@ -354,7 +354,7 @@ static int runtest_dar_outside(void) if (breaks == 2) { printf("TESTED: Partial overlap\n"); } else { - printf("FAILED: Partial overlap: %lld != 2\n", breaks); + printf("FAILED: Partial overlap: %llu != 2\n", breaks); fail = 1; } @@ -369,7 +369,7 @@ static int runtest_dar_outside(void) if (breaks == 0) { printf("TESTED: No overlap\n"); } else { - printf("FAILED: No overlap: %lld != 0\n", breaks); + printf("FAILED: No overlap: %llu != 0\n", breaks); fail = 1; } @@ -384,7 +384,7 @@ static int runtest_dar_outside(void) if (breaks == 2) { printf("TESTED: Full overlap\n"); } else { - printf("FAILED: Full overlap: %lld != 2\n", breaks); + printf("FAILED: Full overlap: %llu != 2\n", breaks); fail = 1; } @@ -437,7 +437,7 @@ static int test_process_multi_diff_addr(void) close(fd2); if (breaks1 != 2 || breaks2 != 2) { - printf("FAILED: %s: %lld != 2 || %lld != 2\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 2 || %llu != 2\n", desc, breaks1, breaks2); return 1; } @@ -482,7 +482,7 @@ static int test_process_multi_same_addr(void) close(fd2); if (breaks1 != 2 || breaks2 != 2) { - printf("FAILED: %s: %lld != 2 || %lld != 2\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 2 || %llu != 2\n", desc, breaks1, breaks2); return 1; } @@ -527,7 +527,7 @@ static int test_process_multi_diff_addr_ro_wo(void) close(fd2); if (breaks1 != 1 || breaks2 != 1) { - printf("FAILED: %s: %lld != 1 || %lld != 1\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 1 || %llu != 1\n", desc, breaks1, breaks2); return 1; } @@ -572,7 +572,7 @@ static int test_process_multi_same_addr_ro_wo(void) close(fd2); if (breaks1 != 1 || breaks2 != 1) { - printf("FAILED: %s: %lld != 1 || %lld != 1\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 1 || %llu != 1\n", desc, breaks1, breaks2); return 1; } @@ -616,7 +616,7 @@ static int test_syswide_multi_diff_addr(void) free(fd2); if (breaks1 != 2 || breaks2 != 2) { - printf("FAILED: %s: %lld != 2 || %lld != 2\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 2 || %llu != 2\n", desc, breaks1, breaks2); return 1; } @@ -660,7 +660,7 @@ static int test_syswide_multi_same_addr(void) free(fd2); if (breaks1 != 2 || breaks2 != 2) { - printf("FAILED: %s: %lld != 2 || %lld != 2\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 2 || %llu != 2\n", desc, breaks1, breaks2); return 1; } @@ -704,7 +704,7 @@ static int test_syswide_multi_diff_addr_ro_wo(void) free(fd2); if (breaks1 != 1 || breaks2 != 1) { - printf("FAILED: %s: %lld != 1 || %lld != 1\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 1 || %llu != 1\n", desc, breaks1, breaks2); return 1; } @@ -748,7 +748,7 @@ static int test_syswide_multi_same_addr_ro_wo(void) free(fd2); if (breaks1 != 1 || breaks2 != 1) { - printf("FAILED: %s: %lld != 1 || %lld != 1\n", desc, breaks1, breaks2); + printf("FAILED: %s: %llu != 1 || %llu != 1\n", desc, breaks1, breaks2); return 1; } @@ -797,7 +797,7 @@ static int runtest_unaligned_512bytes(void) close(fd); if (breaks != 2) { - printf("FAILED: %s: %lld != 2\n", desc, breaks); + printf("FAILED: %s: %llu != 2\n", desc, breaks); return 1; } -- 2.50.1.windows.1
