]> Git Repo - J-linux.git/commitdiff
KVM: selftests: Actually print out magic token in NX hugepages skip message
authorangquan yu <[email protected]>
Tue, 28 Nov 2023 22:11:05 +0000 (16:11 -0600)
committerPaolo Bonzini <[email protected]>
Fri, 8 Dec 2023 18:18:45 +0000 (13:18 -0500)
Pass MAGIC_TOKEN to __TEST_REQUIRE() when printing the help message about
needing to pass a magic value to manually run the NX hugepages test,
otherwise the help message will contain garbage.

  In file included from x86_64/nx_huge_pages_test.c:15:
  x86_64/nx_huge_pages_test.c: In function ‘main’:
  include/test_util.h:40:32: error: format ‘%d’ expects a matching ‘int’ argument [-Werror=format=]
     40 |                 ksft_exit_skip("- " fmt "\n", ##__VA_ARGS__);   \
        |                                ^~~~
  x86_64/nx_huge_pages_test.c:259:9: note: in expansion of macro ‘__TEST_REQUIRE’
    259 |         __TEST_REQUIRE(token == MAGIC_TOKEN,
        |         ^~~~~~~~~~~~~~

Signed-off-by: angquan yu <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
[sean: rewrite shortlog+changelog]
Signed-off-by: Sean Christopherson <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
tools/testing/selftests/kvm/x86_64/nx_huge_pages_test.c

index 18ac5c1952a3a382590d799c8959f600cafc7cd2..83e25bccc139decff79249e99a336ef2cb8cc820 100644 (file)
@@ -259,7 +259,7 @@ int main(int argc, char **argv)
        __TEST_REQUIRE(token == MAGIC_TOKEN,
                       "This test must be run with the magic token %d.\n"
                       "This is done by nx_huge_pages_test.sh, which\n"
-                      "also handles environment setup for the test.");
+                      "also handles environment setup for the test.", MAGIC_TOKEN);
 
        run_test(reclaim_period_ms, false, reboot_permissions);
        run_test(reclaim_period_ms, true, reboot_permissions);
This page took 0.057209 seconds and 4 git commands to generate.