From: "Mike Rapoport (Microsoft)" <[email protected]>

Convert hugetlb_madv_vs_map test to use kselftest framework for
reporting and tracking successful and failing runs.

Reviewed-by: Mark Brown <[email protected]>
Reviewed-by: Donet Tom <[email protected]>
Signed-off-by: Mike Rapoport (Microsoft) <[email protected]>
---
 .../testing/selftests/mm/hugetlb_madv_vs_map.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/tools/testing/selftests/mm/hugetlb_madv_vs_map.c 
b/tools/testing/selftests/mm/hugetlb_madv_vs_map.c
index efd774b41389..c7105c6d319b 100644
--- a/tools/testing/selftests/mm/hugetlb_madv_vs_map.c
+++ b/tools/testing/selftests/mm/hugetlb_madv_vs_map.c
@@ -25,7 +25,6 @@
 #include <unistd.h>
 
 #include "vm_util.h"
-#include "kselftest.h"
 
 #define INLOOP_ITER 100
 
@@ -86,12 +85,14 @@ int main(void)
         */
        int max = 10;
 
+       ksft_print_header();
+       ksft_set_plan(1);
+
        free_hugepages = get_free_hugepages();
 
-       if (free_hugepages != 1) {
+       if (free_hugepages != 1)
                ksft_exit_skip("This test needs one and only one page to 
execute. Got %lu\n",
                               free_hugepages);
-       }
 
        mmap_size = default_huge_page_size();
 
@@ -100,10 +101,8 @@ int main(void)
                                MAP_PRIVATE | MAP_ANONYMOUS | MAP_HUGETLB,
                                -1, 0);
 
-               if ((unsigned long)huge_ptr == -1) {
-                       ksft_test_result_fail("Failed to allocate huge page\n");
-                       return KSFT_FAIL;
-               }
+               if ((unsigned long)huge_ptr == -1)
+                       ksft_exit_fail_msg("Failed to allocate huge page\n");
 
                pthread_create(&thread1, NULL, madv, NULL);
                pthread_create(&thread2, NULL, touch, NULL);
@@ -115,12 +114,13 @@ int main(void)
 
                if (ret) {
                        ksft_test_result_fail("Unexpected huge page 
allocation\n");
-                       return KSFT_FAIL;
+                       ksft_finished();
                }
 
                /* Unmap and restart */
                munmap(huge_ptr, mmap_size);
        }
 
-       return KSFT_PASS;
+       ksft_test_result_pass("No unexpected huge page allocations\n");
+       ksft_finished();
 }
-- 
2.53.0


Reply via email to