* [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start @ 2024-11-01 14:15 Muhammad Usama Anjum 2024-11-01 18:54 ` Andrew Morton 2024-11-08 10:35 ` Donet Tom 0 siblings, 2 replies; 11+ messages in thread From: Muhammad Usama Anjum @ 2024-11-01 14:15 UTC (permalink / raw) To: Andrew Morton, Shuah Khan Cc: Muhammad Usama Anjum, kernel, linux-mm, linux-kselftest, linux-kernel 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 file can be allocated. - The test should only execute if huge pages are free. Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> --- Before: TAP version 13 1..4 Bail out! Error opening file : Read-only file system (30) # Planned tests != run tests (4 != 0) # Totals: pass:0 fail:0 xfail:0 xpass:0 skip:0 error:0 After: TAP version 13 1..0 # SKIP Unable to allocate file: Read-only file system --- tools/testing/selftests/mm/hugetlb_dio.c | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c index f9ac20c657ec6..60001c142ce99 100644 --- a/tools/testing/selftests/mm/hugetlb_dio.c +++ b/tools/testing/selftests/mm/hugetlb_dio.c @@ -44,13 +44,6 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) if (fd < 0) ksft_exit_fail_perror("Error opening file\n"); - /* Get the free huge pages before allocation */ - free_hpage_b = get_free_hugepages(); - if (free_hpage_b == 0) { - close(fd); - ksft_exit_skip("No free hugepage, exiting!\n"); - } - /* Allocate a hugetlb page */ orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); if (orig_buffer == MAP_FAILED) { @@ -94,8 +87,20 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) int main(void) { size_t pagesize = 0; + int fd; ksft_print_header(); + + /* Open the file to DIO */ + fd = open("/tmp", O_TMPFILE | O_RDWR | O_DIRECT, 0664); + if (fd < 0) + ksft_exit_skip("Unable to allocate file: %s\n", strerror(errno)); + close(fd); + + /* Check if huge pages are free */ + if (!get_free_hugepages()) + ksft_exit_skip("No free hugepage, exiting\n"); + ksft_set_plan(4); /* Get base page size */ -- 2.39.5 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-01 14:15 [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start Muhammad Usama Anjum @ 2024-11-01 18:54 ` Andrew Morton 2024-11-08 10:35 ` Donet Tom 1 sibling, 0 replies; 11+ messages in thread From: Andrew Morton @ 2024-11-01 18:54 UTC (permalink / raw) To: Muhammad Usama Anjum Cc: Shuah Khan, kernel, linux-mm, linux-kselftest, linux-kernel, Donet Tom On Fri, 1 Nov 2024 19:15:57 +0500 Muhammad Usama Anjum <usama.anjum@collabora.com> 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 file can be allocated. > - The test should only execute if huge pages are free. Cc Donet Tom. I added Fixes: 3a103b5315b7 ("selftest: mm: Test if hugepage does not get leaked during __bio_release_pages()") Cc: <stable@vger.kernel.org> > --- > Before: > TAP version 13 > 1..4 > Bail out! Error opening file > : Read-only file system (30) > # Planned tests != run tests (4 != 0) > # Totals: pass:0 fail:0 xfail:0 xpass:0 skip:0 error:0 > > After: > TAP version 13 > 1..0 # SKIP Unable to allocate file: Read-only file system > --- And I moved this useful information to the changelog. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-01 14:15 [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start Muhammad Usama Anjum 2024-11-01 18:54 ` Andrew Morton @ 2024-11-08 10:35 ` Donet Tom 2024-11-08 10:49 ` Donet Tom 1 sibling, 1 reply; 11+ messages in thread From: Donet Tom @ 2024-11-08 10:35 UTC (permalink / raw) To: Muhammad Usama Anjum, Andrew Morton, Shuah Khan Cc: kernel, linux-mm, linux-kselftest, linux-kernel 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 file can be allocated. > - The test should only execute if huge pages are free. > > Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> > --- > Before: > TAP version 13 > 1..4 > Bail out! Error opening file > : Read-only file system (30) > # Planned tests != run tests (4 != 0) > # Totals: pass:0 fail:0 xfail:0 xpass:0 skip:0 error:0 > > After: > TAP version 13 > 1..0 # SKIP Unable to allocate file: Read-only file system > --- > tools/testing/selftests/mm/hugetlb_dio.c | 19 ++++++++++++------- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c > index f9ac20c657ec6..60001c142ce99 100644 > --- a/tools/testing/selftests/mm/hugetlb_dio.c > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > @@ -44,13 +44,6 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > if (fd < 0) > ksft_exit_fail_perror("Error opening file\n"); > > - /* Get the free huge pages before allocation */ > - free_hpage_b = get_free_hugepages(); Hi Muhammed Usman Anjum Reading the free pages is required before starting the test. This value will be compared to the free pages after the test. If they are not the same, the test will be considered a failure. Since reading the free pages before the test was removed,|free_hpage_b| is always 0, causing the test to fail. ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 1 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 2 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 3 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 4 : Huge pages not freed! # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 I think below changes are required. --- a/tools/testing/selftests/mm/hugetlb_dio.c +++ b/tools/testing/selftests/mm/hugetlb_dio.c @@ -44,6 +44,9 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) if (fd < 0) ksft_exit_fail_perror("Error opening file\n"); + /* Get the free huge pages before allocation */ + free_hpage_b = get_free_hugepages(); + /* Allocate a hugetlb page */ orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); if (orig_buffer == MAP_FAILED) { With this change the tests are passing. ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 1 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 2 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 3 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 4 : Huge pages freed successfully ! # Totals: pass:4 fail:0 xfail:0 xpass:0 skip:0 error:0 Thank Donet > - if (free_hpage_b == 0) { > - close(fd); > - ksft_exit_skip("No free hugepage, exiting!\n"); > - } > - > /* Allocate a hugetlb page */ > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > if (orig_buffer == MAP_FAILED) { > @@ -94,8 +87,20 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > int main(void) > { > size_t pagesize = 0; > + int fd; > > ksft_print_header(); > + > + /* Open the file to DIO */ > + fd = open("/tmp", O_TMPFILE | O_RDWR | O_DIRECT, 0664); > + if (fd < 0) > + ksft_exit_skip("Unable to allocate file: %s\n", strerror(errno)); > + close(fd); > + > + /* Check if huge pages are free */ > + if (!get_free_hugepages()) > + ksft_exit_skip("No free hugepage, exiting\n"); > + > ksft_set_plan(4); > > /* Get base page size */ ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-08 10:35 ` Donet Tom @ 2024-11-08 10:49 ` Donet Tom 2024-11-08 14:13 ` Muhammad Usama Anjum 0 siblings, 1 reply; 11+ messages in thread From: Donet Tom @ 2024-11-08 10:49 UTC (permalink / raw) To: Muhammad Usama Anjum, Andrew Morton, Shuah Khan Cc: kernel, linux-mm, linux-kselftest, linux-kernel 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 conditions are: >> - The test should only execute if /tmp file can be allocated. >> - The test should only execute if huge pages are free. >> >> Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> >> --- >> Before: >> TAP version 13 >> 1..4 >> Bail out! Error opening file >> : Read-only file system (30) >> # Planned tests != run tests (4 != 0) >> # Totals: pass:0 fail:0 xfail:0 xpass:0 skip:0 error:0 >> >> After: >> TAP version 13 >> 1..0 # SKIP Unable to allocate file: Read-only file system >> --- >> tools/testing/selftests/mm/hugetlb_dio.c | 19 ++++++++++++------- >> 1 file changed, 12 insertions(+), 7 deletions(-) >> >> diff --git a/tools/testing/selftests/mm/hugetlb_dio.c >> b/tools/testing/selftests/mm/hugetlb_dio.c >> index f9ac20c657ec6..60001c142ce99 100644 >> --- a/tools/testing/selftests/mm/hugetlb_dio.c >> +++ b/tools/testing/selftests/mm/hugetlb_dio.c >> @@ -44,13 +44,6 @@ void run_dio_using_hugetlb(unsigned int start_off, >> unsigned int end_off) >> if (fd < 0) >> ksft_exit_fail_perror("Error opening file\n"); >> - /* Get the free huge pages before allocation */ >> - free_hpage_b = get_free_hugepages(); > > Hi Muhammed Usman Anjum > > Reading the free pages is required before starting the test. This > value will be compared to the free pages after the test. If they are > not the same, the test will be considered a failure. > > Since reading the free pages before the test was > removed,|free_hpage_b| is always 0, causing the test to fail. > > ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. > Free pages before allocation : 0 # No. Free pages after munmap : 100 > not ok 1 : Huge pages not freed! # No. Free pages before allocation : > 0 # No. Free pages after munmap : 100 not ok 2 : Huge pages not freed! > # No. Free pages before allocation : 0 # No. Free pages after munmap : > 100 not ok 3 : Huge pages not freed! # No. Free pages before > allocation : 0 # No. Free pages after munmap : 100 not ok 4 : Huge > pages not freed! # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 > error:0 I think below changes are required. --- > a/tools/testing/selftests/mm/hugetlb_dio.c +++ > b/tools/testing/selftests/mm/hugetlb_dio.c @@ -44,6 +44,9 @@ void > run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) if > (fd < 0) ksft_exit_fail_perror("Error opening file\n"); + /* Get the > free huge pages before allocation */ + free_hpage_b = > get_free_hugepages(); + /* Allocate a hugetlb page */ orig_buffer = > mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); if (orig_buffer > == MAP_FAILED) { With this change the tests are passing. > ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. > Free pages before allocation : 100 # No. Free pages after munmap : 100 > ok 1 : Huge pages freed successfully ! # No. Free pages before > allocation : 100 # No. Free pages after munmap : 100 ok 2 : Huge pages > freed successfully ! # No. Free pages before allocation : 100 # No. > Free pages after munmap : 100 ok 3 : Huge pages freed successfully ! # > No. Free pages before allocation : 100 # No. Free pages after munmap : > 100 ok 4 : Huge pages freed successfully ! # Totals: pass:4 fail:0 > xfail:0 xpass:0 skip:0 error:0 Thank > > Donet > > Sorry. Please ignore above mail. Reading the free pages is required before starting the test. This value will be compared to the free pages after the test. If they are not the same, the test will be considered a failure. Since reading the free pages before the test was removed,free_hpage_b is always 0, causing the test to fail. ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 1 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 2 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 3 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 4 : Huge pages not freed! # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 I think below changes are required. iff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c index 60001c142ce9..4b52106b8124 100644 --- a/tools/testing/selftests/mm/hugetlb_dio.c +++ b/tools/testing/selftests/mm/hugetlb_dio.c @@ -44,6 +44,9 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) if (fd < 0) ksft_exit_fail_perror("Error opening file\n"); + /* Get the free huge pages before allocation */ + free_hpage_b = get_free_hugepages(); + /* Allocate a hugetlb page */ orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); if (orig_buffer == MAP_FAILED) { With this change the tests are passing. ./tools/testing/selftests/mm/hugetlb_dio TAP version 131..4 # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 1 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 2 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 3 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 4 : Huge pages freed successfully ! # Totals: pass:4 fail:0 xfail:0 xpass:0 skip:0 error:0 Thanks Donet >> - if (free_hpage_b == 0) { >> - close(fd); >> - ksft_exit_skip("No free hugepage, exiting!\n"); >> - } >> - >> /* Allocate a hugetlb page */ >> orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, >> 0); >> if (orig_buffer == MAP_FAILED) { >> @@ -94,8 +87,20 @@ void run_dio_using_hugetlb(unsigned int start_off, >> unsigned int end_off) >> int main(void) >> { >> size_t pagesize = 0; >> + int fd; >> ksft_print_header(); >> + >> + /* Open the file to DIO */ >> + fd = open("/tmp", O_TMPFILE | O_RDWR | O_DIRECT, 0664); >> + if (fd < 0) >> + ksft_exit_skip("Unable to allocate file: %s\n", >> strerror(errno)); >> + close(fd); >> + >> + /* Check if huge pages are free */ >> + if (!get_free_hugepages()) >> + ksft_exit_skip("No free hugepage, exiting\n"); >> + >> ksft_set_plan(4); >> /* Get base page size */ ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-08 10:49 ` Donet Tom @ 2024-11-08 14:13 ` Muhammad Usama Anjum 2024-11-10 6:20 ` Andrew Morton 0 siblings, 1 reply; 11+ messages in thread From: Muhammad Usama Anjum @ 2024-11-08 14:13 UTC (permalink / raw) To: Donet Tom, Andrew Morton, Shuah Khan Cc: Usama.Anjum, kernel, linux-mm, linux-kselftest, linux-kernel On 11/8/24 3:49 PM, Donet Tom wrote: > I think below changes are required. > > iff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c > index 60001c142ce9..4b52106b8124 100644 > --- a/tools/testing/selftests/mm/hugetlb_dio.c > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > @@ -44,6 +44,9 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > if (fd < 0) > ksft_exit_fail_perror("Error opening file\n"); > > + /* Get the free huge pages before allocation */ > + free_hpage_b = get_free_hugepages(); > + > /* Allocate a hugetlb page */ > > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > > if (orig_buffer == MAP_FAILED) { Please can you send a fixup patch as you have working test setup? Otherwise I'll take it up and try to test on working setup before posting the fixup patch. Please let me know. > > With this change the tests are passing. > > ./tools/testing/selftests/mm/hugetlb_dio > > TAP version 131..4 > # No. Free pages before allocation : 100 > # No. Free pages after munmap : 100 > ok 1 : Huge pages freed successfully ! > # No. Free pages before allocation : 100 > # No. Free pages after munmap : 100 > ok 2 : Huge pages freed successfully ! > # No. Free pages before allocation : 100 > # No. Free pages after munmap : 100 > ok 3 : Huge pages freed successfully ! > # No. Free pages before allocation : 100 > # No. Free pages after munmap : 100 > ok 4 : Huge pages freed successfully ! > # Totals: pass:4 fail:0 xfail:0 xpass:0 skip:0 error:0 > > Thanks > Donet > > -- BR, Muhammad Usama Anjum ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-08 14:13 ` Muhammad Usama Anjum @ 2024-11-10 6:20 ` Andrew Morton 2024-11-10 6:34 ` Andrew Morton 0 siblings, 1 reply; 11+ messages in thread From: Andrew Morton @ 2024-11-10 6:20 UTC (permalink / raw) To: Muhammad Usama Anjum Cc: Donet Tom, Shuah Khan, kernel, linux-mm, linux-kselftest, linux-kernel On Fri, 8 Nov 2024 19:13:04 +0500 Muhammad Usama Anjum <Usama.Anjum@collabora.com> wrote: > On 11/8/24 3:49 PM, Donet Tom wrote: > > > I think below changes are required. > > > > iff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c > > index 60001c142ce9..4b52106b8124 100644 > > --- a/tools/testing/selftests/mm/hugetlb_dio.c > > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > > @@ -44,6 +44,9 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > > if (fd < 0) > > ksft_exit_fail_perror("Error opening file\n"); > > > > + /* Get the free huge pages before allocation */ > > + free_hpage_b = get_free_hugepages(); > > + > > /* Allocate a hugetlb page */ > > > > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > > > > if (orig_buffer == MAP_FAILED) { > Please can you send a fixup patch as you have working test setup? > Otherwise I'll take it up and try to test on working setup before > posting the fixup patch. Please let me know. I've removed this patch from mm-hotfixes-stable. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start 2024-11-10 6:20 ` Andrew Morton @ 2024-11-10 6:34 ` Andrew Morton 2024-11-10 6:49 ` [PATCH] selftests: hugetlb_dio: Fixup " Donet Tom 0 siblings, 1 reply; 11+ messages in thread From: Andrew Morton @ 2024-11-10 6:34 UTC (permalink / raw) To: Muhammad Usama Anjum, Donet Tom, Shuah Khan, kernel, linux-mm, linux-kselftest, linux-kernel On Sat, 9 Nov 2024 22:20:01 -0800 Andrew Morton <akpm@linux-foundation.org> wrote: > On Fri, 8 Nov 2024 19:13:04 +0500 Muhammad Usama Anjum <Usama.Anjum@collabora.com> wrote: > > > On 11/8/24 3:49 PM, Donet Tom wrote: > > > > > I think below changes are required. > > > > > > iff --git a/tools/testing/selftests/mm/hugetlb_dio.c b/tools/testing/selftests/mm/hugetlb_dio.c > > > index 60001c142ce9..4b52106b8124 100644 > > > --- a/tools/testing/selftests/mm/hugetlb_dio.c > > > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > > > @@ -44,6 +44,9 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > > > if (fd < 0) > > > ksft_exit_fail_perror("Error opening file\n"); > > > > > > + /* Get the free huge pages before allocation */ > > > + free_hpage_b = get_free_hugepages(); > > > + > > > /* Allocate a hugetlb page */ > > > > > > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > > > > > > if (orig_buffer == MAP_FAILED) { > > Please can you send a fixup patch as you have working test setup? > > Otherwise I'll take it up and try to test on working setup before > > posting the fixup patch. Please let me know. > > I've removed this patch from mm-hotfixes-stable. I changed my mind. Please send any fixup against the previous patch. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start 2024-11-10 6:34 ` Andrew Morton @ 2024-11-10 6:49 ` Donet Tom 2024-11-10 6:52 ` Donet Tom 2024-11-10 6:56 ` Donet Tom 0 siblings, 2 replies; 11+ messages in thread From: Donet Tom @ 2024-11-10 6:49 UTC (permalink / raw) To: akpm Cc: Usama.Anjum, donettom, kernel, linux-kernel, linux-kselftest, linux-mm, shuah 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 the same. Reading free hugepages before the test was removed by commit 0268d4579901 ('selftests: hugetlb_dio: check for initial conditions to skip at the start'), causing the test to always fail. This patch adds back reading the free hugepages before starting the test. With this patch, the tests are now passing. Test Results without this patch ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 1 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 2 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 3 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 4 : Huge pages not freed! # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 Test results with this patch. ./tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 1 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 2 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 3 : Huge pages not freed! # No. Free pages before allocation : 0 # No. Free pages after munmap : 100 not ok 4 : Huge pages not freed! # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 Fixes: 0268d4579901 ("selftests: hugetlb_dio: check for initial conditions to skip in the start") Signed-off-by: Donet Tom <donettom@linux.ibm.com> --- 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.c index 60001c142ce9..432d5af15e66 100644 --- a/tools/testing/selftests/mm/hugetlb_dio.c +++ b/tools/testing/selftests/mm/hugetlb_dio.c @@ -44,6 +44,13 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) if (fd < 0) ksft_exit_fail_perror("Error opening file\n"); + /* Get the free huge pages before allocation */ + free_hpage_b = get_free_hugepages(); + if (free_hpage_b == 0) { + close(fd); + ksft_exit_skip("No free hugepage, exiting!\n"); + } + /* Allocate a hugetlb page */ orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); if (orig_buffer == MAP_FAILED) { -- 2.43.5 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start 2024-11-10 6:49 ` [PATCH] selftests: hugetlb_dio: Fixup " Donet Tom @ 2024-11-10 6:52 ` Donet Tom 2024-11-11 6:59 ` Andrew Morton 2024-11-10 6:56 ` Donet Tom 1 sibling, 1 reply; 11+ messages in thread From: Donet Tom @ 2024-11-10 6:52 UTC (permalink / raw) To: akpm; +Cc: Usama.Anjum, kernel, linux-kernel, linux-kselftest, linux-mm, shuah 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 the same. > > Reading free hugepages before the test was removed by commit > 0268d4579901 ('selftests: hugetlb_dio: check for initial conditions > to skip at the start'), causing the test to always fail. > > This patch adds back reading the free hugepages before starting > the test. With this patch, the tests are now passing. > > Test Results without this patch > > ./tools/testing/selftests/mm/hugetlb_dio > TAP version 13 > 1..4 > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 1 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 2 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 3 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 4 : Huge pages not freed! > # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 > > Test results with this patch. > > ./tools/testing/selftests/mm/hugetlb_dio > TAP version 13 > 1..4 > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 1 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 2 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 3 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 4 : Huge pages not freed! > # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 > > Fixes: 0268d4579901 ("selftests: hugetlb_dio: check for initial conditions to skip in the start") > Signed-off-by: Donet Tom <donettom@linux.ibm.com> > --- > 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.c > index 60001c142ce9..432d5af15e66 100644 > --- a/tools/testing/selftests/mm/hugetlb_dio.c > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > @@ -44,6 +44,13 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > if (fd < 0) > ksft_exit_fail_perror("Error opening file\n"); > > + /* Get the free huge pages before allocation */ > + free_hpage_b = get_free_hugepages(); > + if (free_hpage_b == 0) { > + close(fd); > + ksft_exit_skip("No free hugepage, exiting!\n"); > + } > + > /* Allocate a hugetlb page */ > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > if (orig_buffer == MAP_FAILED) { > > Hi Andrew Would you prefer I send this fixup patch as a new series, or is it okay as is? Thanks Donet ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start 2024-11-10 6:52 ` Donet Tom @ 2024-11-11 6:59 ` Andrew Morton 0 siblings, 0 replies; 11+ messages in thread From: Andrew Morton @ 2024-11-11 6:59 UTC (permalink / raw) To: Donet Tom Cc: Usama.Anjum, kernel, linux-kernel, linux-kselftest, linux-mm, shuah On Sun, 10 Nov 2024 12:22:12 +0530 Donet Tom <donettom@linux.ibm.com> wrote: > > Fixes: 0268d4579901 ("selftests: hugetlb_dio: check for initial conditions to skip in the start") > > ... > > Would you prefer I send this fixup patch as a new series, or is it okay as is? All looks good, thanks. I added cc:stable, because 0268d4579901 was cc:stable. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] selftests: hugetlb_dio: Fixup Check for initial conditions to skip in the start 2024-11-10 6:49 ` [PATCH] selftests: hugetlb_dio: Fixup " Donet Tom 2024-11-10 6:52 ` Donet Tom @ 2024-11-10 6:56 ` Donet Tom 1 sibling, 0 replies; 11+ messages in thread From: Donet Tom @ 2024-11-10 6:56 UTC (permalink / raw) To: akpm; +Cc: Usama.Anjum, kernel, linux-kernel, linux-kselftest, linux-mm, shuah 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 the same. > > Reading free hugepages before the test was removed by commit > 0268d4579901 ('selftests: hugetlb_dio: check for initial conditions > to skip at the start'), causing the test to always fail. > > This patch adds back reading the free hugepages before starting > the test. With this patch, the tests are now passing. > > Test Results without this patch > > ./tools/testing/selftests/mm/hugetlb_dio > TAP version 13 > 1..4 > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 1 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 2 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 3 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 4 : Huge pages not freed! > # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 > > Test results with this patch. Sorry the test results with this patch is below. Test results With this patch /tools/testing/selftests/mm/hugetlb_dio TAP version 13 1..4 # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 1 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 2 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 3 : Huge pages freed successfully ! # No. Free pages before allocation : 100 # No. Free pages after munmap : 100 ok 4 : Huge pages freed successfully ! # Totals: pass:4 fail:0 xfail:0 xpass:0 skip:0 error:0 Thank Donet > > ./tools/testing/selftests/mm/hugetlb_dio > TAP version 13 > 1..4 > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 1 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 2 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 3 : Huge pages not freed! > # No. Free pages before allocation : 0 > # No. Free pages after munmap : 100 > not ok 4 : Huge pages not freed! > # Totals: pass:0 fail:4 xfail:0 xpass:0 skip:0 error:0 > > Fixes: 0268d4579901 ("selftests: hugetlb_dio: check for initial conditions to skip in the start") > Signed-off-by: Donet Tom <donettom@linux.ibm.com> > --- > 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.c > index 60001c142ce9..432d5af15e66 100644 > --- a/tools/testing/selftests/mm/hugetlb_dio.c > +++ b/tools/testing/selftests/mm/hugetlb_dio.c > @@ -44,6 +44,13 @@ void run_dio_using_hugetlb(unsigned int start_off, unsigned int end_off) > if (fd < 0) > ksft_exit_fail_perror("Error opening file\n"); > > + /* Get the free huge pages before allocation */ > + free_hpage_b = get_free_hugepages(); > + if (free_hpage_b == 0) { > + close(fd); > + ksft_exit_skip("No free hugepage, exiting!\n"); > + } > + > /* Allocate a hugetlb page */ > orig_buffer = mmap(NULL, h_pagesize, mmap_prot, mmap_flags, -1, 0); > if (orig_buffer == MAP_FAILED) { ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2024-12-05 15:28 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2024-11-01 14:15 [PATCH] selftests: hugetlb_dio: Check for initial conditions to skip in the start Muhammad Usama Anjum 2024-11-01 18:54 ` Andrew Morton 2024-11-08 10:35 ` Donet Tom 2024-11-08 10:49 ` Donet Tom 2024-11-08 14:13 ` Muhammad Usama Anjum 2024-11-10 6:20 ` Andrew Morton 2024-11-10 6:34 ` Andrew Morton 2024-11-10 6:49 ` [PATCH] selftests: hugetlb_dio: Fixup " Donet Tom 2024-11-10 6:52 ` Donet Tom 2024-11-11 6:59 ` Andrew Morton 2024-11-10 6:56 ` Donet Tom
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox