selftests/mm: run_vmtests.sh: add hugetlb test category

[PATCH] selftests/mm: run_vmtests.sh: add hugetlb test category

Posted by Breno Leitao 2 years ago

The usage of run_vmtests.sh does not include hugetlb, which is a valid
test category.

Add the 'hugetlb' to the usage of run_vmtests.sh.

Signed-off-by: Breno Leitao <leitao@debian.org>
---
 tools/testing/selftests/mm/run_vmtests.sh | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/testing/selftests/mm/run_vmtests.sh b/tools/testing/selftests/mm/run_vmtests.sh
index 55898d64e2eb..2ee0a1c4740f 100755
--- a/tools/testing/selftests/mm/run_vmtests.sh
+++ b/tools/testing/selftests/mm/run_vmtests.sh
@@ -65,6 +65,8 @@ separated by spaces:
 	test copy-on-write semantics
 - thp
 	test transparent huge pages
+- hugetlb
+	test hugetlbfs huge pages
 - migration
 	invoke move_pages(2) to exercise the migration entry code
 	paths in the kernel
-- 
2.39.3

Re: [PATCH] selftests/mm: run_vmtests.sh: add hugetlb test category

Posted by Joel Savitz 2 years ago

On Mon, Jan 29, 2024 at 6:53 AM Breno Leitao <leitao@debian.org> wrote:
>
> The usage of run_vmtests.sh does not include hugetlb, which is a valid
> test category.
>
> Add the 'hugetlb' to the usage of run_vmtests.sh.
>
> Signed-off-by: Breno Leitao <leitao@debian.org>
> ---
>  tools/testing/selftests/mm/run_vmtests.sh | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/tools/testing/selftests/mm/run_vmtests.sh b/tools/testing/selftests/mm/run_vmtests.sh
> index 55898d64e2eb..2ee0a1c4740f 100755
> --- a/tools/testing/selftests/mm/run_vmtests.sh
> +++ b/tools/testing/selftests/mm/run_vmtests.sh
> @@ -65,6 +65,8 @@ separated by spaces:
>         test copy-on-write semantics
>  - thp
>         test transparent huge pages
> +- hugetlb
> +       test hugetlbfs huge pages
>  - migration
>         invoke move_pages(2) to exercise the migration entry code
>         paths in the kernel
> --
> 2.39.3
>

Reviewed-by: Joel Savitz <jsavitz@redhat.com>

Re: [PATCH] selftests/mm: run_vmtests.sh: add hugetlb test category

Posted by Muhammad Usama Anjum 2 years ago

On 1/29/24 4:52 PM, Breno Leitao wrote:
> The usage of run_vmtests.sh does not include hugetlb, which is a valid
> test category.
> 
> Add the 'hugetlb' to the usage of run_vmtests.sh.
> 
> Signed-off-by: Breno Leitao <leitao@debian.org>
Reviewed-by: Muhammad Usama Anjum <usama.anjum@collabora.com>

> ---
>  tools/testing/selftests/mm/run_vmtests.sh | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tools/testing/selftests/mm/run_vmtests.sh b/tools/testing/selftests/mm/run_vmtests.sh
> index 55898d64e2eb..2ee0a1c4740f 100755
> --- a/tools/testing/selftests/mm/run_vmtests.sh
> +++ b/tools/testing/selftests/mm/run_vmtests.sh
> @@ -65,6 +65,8 @@ separated by spaces:
>  	test copy-on-write semantics
>  - thp
>  	test transparent huge pages
> +- hugetlb
> +	test hugetlbfs huge pages
>  - migration
>  	invoke move_pages(2) to exercise the migration entry code
>  	paths in the kernel

-- 
BR,
Muhammad Usama Anjum