mbox series

[v1,0/2] selftests/mm: hugetlb_fault_after_madv improvements

Message ID 20240926152044.2205129-1-david@redhat.com (mailing list archive)
Headers show
Series selftests/mm: hugetlb_fault_after_madv improvements | expand

Message

David Hildenbrand Sept. 26, 2024, 3:20 p.m. UTC
Mario brought to my attention that the hugetlb_fault_after_madv test
is currently always skipped on s390x.

Let's adjust the test to be independent of the default hugetlb page size
and while at it, also improve the test output.

Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Shuah Khan <shuah@kernel.org>
Cc: Mario Casquero <mcasquer@redhat.com>
Cc: Breno Leitao <leitao@debian.org>

David Hildenbrand (2):
  selftests/mm: hugetlb_fault_after_madv: use default hguetlb page size
  selftests/mm: hugetlb_fault_after_madv: improve test output

 .../selftests/mm/hugetlb_fault_after_madv.c   | 48 ++++++++++++++++---
 1 file changed, 42 insertions(+), 6 deletions(-)

Comments

Mario Casquero Oct. 1, 2024, 10:31 a.m. UTC | #1
This series has been successfully tested. Now when executing the
hugetlb_fault_after_madv selftest the benefits of both patches could
be observed.
# ./hugetlb_fault_after_madv
TAP version 13
1..1
# [INFO] detected default hugetlb page size: 1024 KiB
ok 1 SIGBUS behavior
# Totals: pass:1 fail:0 xfail:0 xpass:0 skip:0 error:0

Tested-by: Mario Casquero <mcasquer@redhat.com>


On Thu, Sep 26, 2024 at 5:20 PM David Hildenbrand <david@redhat.com> wrote:
>
> Mario brought to my attention that the hugetlb_fault_after_madv test
> is currently always skipped on s390x.
>
> Let's adjust the test to be independent of the default hugetlb page size
> and while at it, also improve the test output.
>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Shuah Khan <shuah@kernel.org>
> Cc: Mario Casquero <mcasquer@redhat.com>
> Cc: Breno Leitao <leitao@debian.org>
>
> David Hildenbrand (2):
>   selftests/mm: hugetlb_fault_after_madv: use default hguetlb page size
>   selftests/mm: hugetlb_fault_after_madv: improve test output
>
>  .../selftests/mm/hugetlb_fault_after_madv.c   | 48 ++++++++++++++++---
>  1 file changed, 42 insertions(+), 6 deletions(-)
>
> --
> 2.46.1
>