selftests/mm: dont run ksm_functional_tests twice

Message ID 20231129221140.614713-1-npache@redhat.com
State New
Headers
Series selftests/mm: dont run ksm_functional_tests twice |

Commit Message

Nico Pache Nov. 29, 2023, 10:11 p.m. UTC
  ksm functional test is already being run.
Remove the duplicate call to ./ksm_functional_tests.

Fixes: 93fb70aa5904 ("selftests/vm: add KSM unmerge tests")
Signed-off-by: Nico Pache <npache@redhat.com>
Cc: David Hildenbrand <david@redhat.com>
---
 tools/testing/selftests/mm/run_vmtests.sh | 2 --
 1 file changed, 2 deletions(-)
  

Comments

Joel Savitz Nov. 29, 2023, 11:44 p.m. UTC | #1
On Wed, Nov 29, 2023 at 5:11 PM Nico Pache <npache@redhat.com> wrote:
>
> ksm functional test is already being run.
> Remove the duplicate call to ./ksm_functional_tests.
>
> Fixes: 93fb70aa5904 ("selftests/vm: add KSM unmerge tests")
> Signed-off-by: Nico Pache <npache@redhat.com>
> Cc: David Hildenbrand <david@redhat.com>
> ---
>  tools/testing/selftests/mm/run_vmtests.sh | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/tools/testing/selftests/mm/run_vmtests.sh b/tools/testing/selftests/mm/run_vmtests.sh
> index 00757445278e..c0212258b852 100755
> --- a/tools/testing/selftests/mm/run_vmtests.sh
> +++ b/tools/testing/selftests/mm/run_vmtests.sh
> @@ -334,8 +334,6 @@ CATEGORY="ksm_numa" run_test ./ksm_tests -N -m 0
>
>  CATEGORY="ksm" run_test ./ksm_functional_tests
>
> -run_test ./ksm_functional_tests
> -
>  # protection_keys tests
>  if [ -x ./protection_keys_32 ]
>  then
> --
> 2.41.0
>

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

Patch

diff --git a/tools/testing/selftests/mm/run_vmtests.sh b/tools/testing/selftests/mm/run_vmtests.sh
index 00757445278e..c0212258b852 100755
--- a/tools/testing/selftests/mm/run_vmtests.sh
+++ b/tools/testing/selftests/mm/run_vmtests.sh
@@ -334,8 +334,6 @@  CATEGORY="ksm_numa" run_test ./ksm_tests -N -m 0
 
 CATEGORY="ksm" run_test ./ksm_functional_tests
 
-run_test ./ksm_functional_tests
-
 # protection_keys tests
 if [ -x ./protection_keys_32 ]
 then