2022-06-01 21:03:43

by Kalpana Shetty

Subject: [PATCH] selftests/vm: Add protection_keys tests to run_vmtests

Signed-off-by: Kalpana Shetty <[email protected]>
---
tools/testing/selftests/vm/run_vmtests.sh | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/tools/testing/selftests/vm/run_vmtests.sh b/tools/testing/selftests/vm/run_vmtests.sh
index 41fce8bea929..54a0c28f810c 100755
--- a/tools/testing/selftests/vm/run_vmtests.sh
+++ b/tools/testing/selftests/vm/run_vmtests.sh
@@ -179,4 +179,11 @@ run_test ./ksm_tests -N -m 1
# KSM test with 2 NUMA nodes and merge_across_nodes = 0
run_test ./ksm_tests -N -m 0

+# protection_keys tests
+if [ $VADDR64 -eq 0 ]; then
+ run_test ./protection_keys_32
+else
+ run_test ./protection_keys_64
+fi
+
exit $exitcode
--
2.25.1

2022-06-09 21:07:20

by Shuah Khan

Subject: Re: [PATCH] selftests/vm: Add protection_keys tests to run_vmtests

On 5/31/22 4:25 AM, Kalpana Shetty wrote:
> Signed-off-by: Kalpana Shetty <[email protected]>

Missing commit log - please add a detailed commit log explaining why
this change is necessary.

> ---
> tools/testing/selftests/vm/run_vmtests.sh | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/tools/testing/selftests/vm/run_vmtests.sh b/tools/testing/selftests/vm/run_vmtests.sh
> index 41fce8bea929..54a0c28f810c 100755
> --- a/tools/testing/selftests/vm/run_vmtests.sh
> +++ b/tools/testing/selftests/vm/run_vmtests.sh
> @@ -179,4 +179,11 @@ run_test ./ksm_tests -N -m 1
> # KSM test with 2 NUMA nodes and merge_across_nodes = 0
> run_test ./ksm_tests -N -m 0
>
> +# protection_keys tests
> +if [ $VADDR64 -eq 0 ]; then
> + run_test ./protection_keys_32
> +else
> + run_test ./protection_keys_64
> +fi
> +
> exit $exitcode
>

thanks,
-- Shuah

2022-06-10 10:05:33

by Kalpana Shetty

Subject: Re: [PATCH] selftests/vm: Add protection_keys tests to run_vmtests

On 6/10/2022 1:51 AM, Shuah Khan wrote:
> On 5/31/22 4:25 AM, Kalpana Shetty wrote:
>> Signed-off-by: Kalpana Shetty <[email protected]>
>
> Missing commit log - please add a detailed commit log explaining why
> this change is necessary.

Sent V2 patch with patch description.

Thanks,

Kalpana

>
>> ---
>>   tools/testing/selftests/vm/run_vmtests.sh | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/tools/testing/selftests/vm/run_vmtests.sh
>> b/tools/testing/selftests/vm/run_vmtests.sh
>> index 41fce8bea929..54a0c28f810c 100755
>> --- a/tools/testing/selftests/vm/run_vmtests.sh
>> +++ b/tools/testing/selftests/vm/run_vmtests.sh
>> @@ -179,4 +179,11 @@ run_test ./ksm_tests -N -m 1
>>   # KSM test with 2 NUMA nodes and merge_across_nodes = 0
>>   run_test ./ksm_tests -N -m 0
>>   +# protection_keys tests
>> +if [ $VADDR64 -eq 0 ]; then
>> +    run_test ./protection_keys_32
>> +else
>> +    run_test ./protection_keys_64
>> +fi
>> +
>>   exit $exitcode
>>
>
> thanks,
> -- Shuah