Re: [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start

2024-11-09 Thread Donet Tom
On 11/10/24 12:19, Donet Tom wrote: This test verifies that a hugepage, used as a user buffer for DIO operations, is correctly freed upon unmapping. To test this, we read the count of free hugepages before and after the mmap, DIO, and munmap operations, then check if the free hugepage count is

[PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start

2024-11-09 Thread Donet Tom
"selftests: hugetlb_dio: check for initial conditions to skip in the start") Signed-off-by: Donet Tom --- tools/testing/selftests/mm/hugetlb_dio.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.

Re: [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start

2024-11-09 Thread Donet Tom
On 11/10/24 12:19, Donet Tom wrote: This test verifies that a hugepage, used as a user buffer for DIO operations, is correctly freed upon unmapping. To test this, we read the count of free hugepages before and after the mmap, DIO, and munmap operations, then check if the free hugepage count is

[PATCH] selftests/mm: Fixed incorrect buffer->mirror size in hmm2 double_map test

2024-09-26 Thread Donet Tom
#OK hmm2.hmm2_device_private.double_map ok 53 hmm2.hmm2_device_private.double_map Signed-off-by: Donet Tom --- tools/testing/selftests/mm/hmm-tests.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/mm/hmm-tests.c b/tools/testing/selftests/mm/hmm-tests.c index d2c

Re: [PATCH] selftests/mm: Fixed incorrect buffer->mirror size in hmm2 double_map test

2024-09-27 Thread Donet Tom
On 9/27/24 12:48, Muhammad Usama Anjum wrote: On 9/27/24 10:07 AM, Donet Tom wrote: The hmm2 double_map test was failing due to an incorrect buffer->mirror size. The buffer->mirror size was 6, while buffer->ptr size was 6 * PAGE_SIZE. The test failed because the kernel's copy_to

Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start

2024-11-08 Thread Donet Tom
On 11/1/24 19:45, Muhammad Usama Anjum wrote: The test should be skipped if initial conditions aren't fulfilled in the start instead of failing and outputting non-compliant TAP logs. This kind of failure pollutes the results. The initial conditions are: - The test should only execute if /tmp fi

Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start

2024-11-08 Thread Donet Tom
On 11/8/24 16:05, Donet Tom wrote: On 11/1/24 19:45, Muhammad Usama Anjum wrote: The test should be skipped if initial conditions aren't fulfilled in the start instead of failing and outputting non-compliant TAP logs. This kind of failure pollutes the results. The initial condition

Re: [PATCH] selftest: hugetlb_dio: Fix test naming

2024-11-28 Thread Donet Tom
On 11/28/24 18:14, Mark Brown wrote: On Thu, Nov 28, 2024 at 10:46:56AM +0530, Donet Tom wrote: On 11/27/24 21:44, Mark Brown wrote: + ksft_test_result(free_hpage_a == free_hpage_b, +"free huge pages from %u-%u\n", start_off, end_off); This test a

Re: [PATCH] selftest: hugetlb_dio: Fix test naming

2024-11-27 Thread Donet Tom
On 11/27/24 21:44, Mark Brown wrote: 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 fa

[PATCH] selftests/mm: Added new test cases to the migration test

2024-12-19 Thread Donet Tom
# Signed-off-by: Donet Tom --- tools/testing/selftests/mm/migration.c | 99 ++ 1 file changed, 99 insertions(+) diff --git a/tools/testing/selftests/mm/migration.c b/tools/testing/selftests/mm/migration.c index 64bcbb7151cf..1e3a595fbf01 100644 --- a/tools/testing/selftests