]> www.infradead.org Git - users/dwmw2/linux.git/commit
selftest: hugetlb_dio: fix test naming
authorMark Brown <broonie@kernel.org>
Wed, 27 Nov 2024 16:14:22 +0000 (16:14 +0000)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 6 Dec 2024 03:54:44 +0000 (19:54 -0800)
commit4ae132c693896b0713db572676c90ffd855a4246
tree0d50918df8989587e58f3ed7a4eb8beda9f61ea9
parent965b5dd1894f4525f38c1b5f99b0106a07dbb5db
selftest: hugetlb_dio: fix test naming

The string logged when a test passes or fails is used by the selftest
framework to identify which test is being reported.  The hugetlb_dio test
not only uses the same strings for every test that is run but it also uses
different strings for test passes and failures which means that test
automation is unable to follow what the test is doing at all.

Pull the existing duplicated logging of the number of free huge pages
before and after the test out of the conditional and replace that and the
logging of the result with a single ksft_print_result() which incorporates
the parameters passed into the test into the output.

Link: https://lkml.kernel.org/r/20241127-kselftest-mm-hugetlb-dio-names-v1-1-22aab01bf550@kernel.org
Fixes: fae1980347bf ("selftests: hugetlb_dio: fixup check for initial conditions to skip in the start")
Signed-off-by: Mark Brown <broonie@kernel.org>
Reviewed-by: Muhammad Usama Anjum <usama.anjum@collabora.com>
Cc: Donet Tom <donettom@linux.ibm.com>
Cc: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Cc: Shuah Khan <shuah@kernel.org>
Cc: <stable@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
tools/testing/selftests/mm/hugetlb_dio.c