Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1667575 > unrolled thread
| Started by | Orson Zhai <orson.zhai@linaro.org> |
|---|---|
| First post | 2017-06-16 11:30 +0200 |
| Last post | 2017-06-19 21:10 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Fwd: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict Orson Zhai <orson.zhai@linaro.org> - 2017-06-16 11:30 +0200
Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict Sumit Semwal <sumit.semwal@linaro.org> - 2017-06-16 17:00 +0200
Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict Orson Zhai <orson.zhai@linaro.org> - 2017-06-16 21:50 +0200
Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict Kees Cook <keescook@chromium.org> - 2017-06-19 21:10 +0200
| From | Orson Zhai <orson.zhai@linaro.org> |
|---|---|
| Date | 2017-06-16 11:30 +0200 |
| Subject | Fwd: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict |
| Message-ID | <tSVJL-6Aa-3@gated-at.bofh.it> |
Sysctl test will fail in some items if the value of /proc/sys/kernel
/sysctrl_writes_strict is 0 as the default value in kernel older than v4.5.
Make this test more robust and compatible with older kernels by checking and
update sysctrl_writes_strict value and restore it when test is done.
Signed-off-by: Orson Zhai <orson.zhai@linaro.org>
---
tools/testing/selftests/sysctl/common_tests | 14 ++++++++++++++
tools/testing/selftests/sysctl/run_numerictests | 3 +++
tools/testing/selftests/sysctl/run_stringtests | 3 +++
3 files changed, 20 insertions(+)
diff --git a/tools/testing/selftests/sysctl/common_tests
b/tools/testing/selftests/sysctl/common_tests
index 17d534b1b7b4..f5c5c51d16f3 100644
--- a/tools/testing/selftests/sysctl/common_tests
+++ b/tools/testing/selftests/sysctl/common_tests
@@ -63,6 +63,20 @@ else
echo "ok"
fi
+echo -n "Checking writes strict setting ... "
+WRITES_STRICT="${SYSCTL}/kernel/sysctl_writes_strict"
+if [ ! -e ${WRITES_STRICT} ]; then
+ echo "FAIL, but skip in case of old kernel" >&2
+else
+ val=$(cat ${WRITES_STRICT})
+ if [ "$val" = "1" ]; then
+ echo "ok"
+ else
+ echo "FAIL, strict value is 0 but force to 1 to continue" >&2
+ echo "1" > ${WRITES_STRICT}
+ fi
+fi
+
# Now that we've validated the sanity of "set_test" and "set_orig",
# we can use those functions to set starting states before running
# specific behavioral tests.
diff --git a/tools/testing/selftests/sysctl/run_numerictests
b/tools/testing/selftests/sysctl/run_numerictests
index 8510f93f2d14..c0a98fd82c5c 100755
--- a/tools/testing/selftests/sysctl/run_numerictests
+++ b/tools/testing/selftests/sysctl/run_numerictests
@@ -7,4 +7,7 @@ TEST_STR=$(( $ORIG + 1 ))
. ./common_tests
+if [ ! -z ${val} ]; then
+ echo ${val} > ${WRITES_STRICT}
+fi
exit $rc
diff --git a/tools/testing/selftests/sysctl/run_stringtests
b/tools/testing/selftests/sysctl/run_stringtests
index 90a9293d520c..ae98d66a9ec6 100755
--- a/tools/testing/selftests/sysctl/run_stringtests
+++ b/tools/testing/selftests/sysctl/run_stringtests
@@ -74,4 +74,7 @@ else
echo "ok"
fi
+if [ ! -z ${val} ]; then
+ echo ${val} > ${WRITES_STRICT}
+fi
exit $rc
--
2.12.2
[toc] | [next] | [standalone]
| From | Sumit Semwal <sumit.semwal@linaro.org> |
|---|---|
| Date | 2017-06-16 17:00 +0200 |
| Message-ID | <tT0T7-1p7-17@gated-at.bofh.it> |
| In reply to | #1667575 |
Hi Orson,
Thanks for the patch.
On 16 June 2017 at 14:58, Orson Zhai <orson.zhai@linaro.org> wrote:
> Sysctl test will fail in some items if the value of /proc/sys/kernel
> /sysctrl_writes_strict is 0 as the default value in kernel older than v4.5.
>
> Make this test more robust and compatible with older kernels by checking and
> update sysctrl_writes_strict value and restore it when test is done.
>
> Signed-off-by: Orson Zhai <orson.zhai@linaro.org>
Please feel free to add my
Reviewed-by: Sumit Semwal <sumit.semwal@linaro.org>
Tested-by: Sumit Semwal <sumit.semwal@linaro.org>
[sumits: tested LTS-4.4 with hikey (arm64) ]
> ---
> tools/testing/selftests/sysctl/common_tests | 14 ++++++++++++++
> tools/testing/selftests/sysctl/run_numerictests | 3 +++
> tools/testing/selftests/sysctl/run_stringtests | 3 +++
> 3 files changed, 20 insertions(+)
>
> diff --git a/tools/testing/selftests/sysctl/common_tests
> b/tools/testing/selftests/sysctl/common_tests
> index 17d534b1b7b4..f5c5c51d16f3 100644
> --- a/tools/testing/selftests/sysctl/common_tests
> +++ b/tools/testing/selftests/sysctl/common_tests
> @@ -63,6 +63,20 @@ else
> echo "ok"
> fi
>
> +echo -n "Checking writes strict setting ... "
> +WRITES_STRICT="${SYSCTL}/kernel/sysctl_writes_strict"
> +if [ ! -e ${WRITES_STRICT} ]; then
> + echo "FAIL, but skip in case of old kernel" >&2
> +else
> + val=$(cat ${WRITES_STRICT})
> + if [ "$val" = "1" ]; then
> + echo "ok"
> + else
> + echo "FAIL, strict value is 0 but force to 1 to continue" >&2
> + echo "1" > ${WRITES_STRICT}
> + fi
> +fi
> +
> # Now that we've validated the sanity of "set_test" and "set_orig",
> # we can use those functions to set starting states before running
> # specific behavioral tests.
> diff --git a/tools/testing/selftests/sysctl/run_numerictests
> b/tools/testing/selftests/sysctl/run_numerictests
> index 8510f93f2d14..c0a98fd82c5c 100755
> --- a/tools/testing/selftests/sysctl/run_numerictests
> +++ b/tools/testing/selftests/sysctl/run_numerictests
> @@ -7,4 +7,7 @@ TEST_STR=$(( $ORIG + 1 ))
>
> . ./common_tests
>
> +if [ ! -z ${val} ]; then
> + echo ${val} > ${WRITES_STRICT}
> +fi
> exit $rc
> diff --git a/tools/testing/selftests/sysctl/run_stringtests
> b/tools/testing/selftests/sysctl/run_stringtests
> index 90a9293d520c..ae98d66a9ec6 100755
> --- a/tools/testing/selftests/sysctl/run_stringtests
> +++ b/tools/testing/selftests/sysctl/run_stringtests
> @@ -74,4 +74,7 @@ else
> echo "ok"
> fi
>
> +if [ ! -z ${val} ]; then
> + echo ${val} > ${WRITES_STRICT}
> +fi
> exit $rc
> --
> 2.12.2
Best,
Sumit.
[toc] | [prev] | [next] | [standalone]
| From | Orson Zhai <orson.zhai@linaro.org> |
|---|---|
| Date | 2017-06-16 21:50 +0200 |
| Message-ID | <tT5pM-4AB-13@gated-at.bofh.it> |
| In reply to | #1667820 |
On 16 June 2017 at 22:49, Sumit Semwal <sumit.semwal@linaro.org> wrote:
> Hi Orson,
>
> Thanks for the patch.
>
> On 16 June 2017 at 14:58, Orson Zhai <orson.zhai@linaro.org> wrote:
>> Sysctl test will fail in some items if the value of /proc/sys/kernel
>> /sysctrl_writes_strict is 0 as the default value in kernel older than v4.5.
>>
>> Make this test more robust and compatible with older kernels by checking and
>> update sysctrl_writes_strict value and restore it when test is done.
>>
>> Signed-off-by: Orson Zhai <orson.zhai@linaro.org>
>
> Please feel free to add my
> Reviewed-by: Sumit Semwal <sumit.semwal@linaro.org>
> Tested-by: Sumit Semwal <sumit.semwal@linaro.org>
> [sumits: tested LTS-4.4 with hikey (arm64) ]
Sure. I will add them to my patch V2.
Thanks,
Orson
>
>> ---
>> tools/testing/selftests/sysctl/common_tests | 14 ++++++++++++++
>> tools/testing/selftests/sysctl/run_numerictests | 3 +++
>> tools/testing/selftests/sysctl/run_stringtests | 3 +++
>> 3 files changed, 20 insertions(+)
>>
>> diff --git a/tools/testing/selftests/sysctl/common_tests
>> b/tools/testing/selftests/sysctl/common_tests
>> index 17d534b1b7b4..f5c5c51d16f3 100644
>> --- a/tools/testing/selftests/sysctl/common_tests
>> +++ b/tools/testing/selftests/sysctl/common_tests
>> @@ -63,6 +63,20 @@ else
>> echo "ok"
>> fi
>>
>> +echo -n "Checking writes strict setting ... "
>> +WRITES_STRICT="${SYSCTL}/kernel/sysctl_writes_strict"
>> +if [ ! -e ${WRITES_STRICT} ]; then
>> + echo "FAIL, but skip in case of old kernel" >&2
>> +else
>> + val=$(cat ${WRITES_STRICT})
>> + if [ "$val" = "1" ]; then
>> + echo "ok"
>> + else
>> + echo "FAIL, strict value is 0 but force to 1 to continue" >&2
>> + echo "1" > ${WRITES_STRICT}
>> + fi
>> +fi
>> +
>> # Now that we've validated the sanity of "set_test" and "set_orig",
>> # we can use those functions to set starting states before running
>> # specific behavioral tests.
>> diff --git a/tools/testing/selftests/sysctl/run_numerictests
>> b/tools/testing/selftests/sysctl/run_numerictests
>> index 8510f93f2d14..c0a98fd82c5c 100755
>> --- a/tools/testing/selftests/sysctl/run_numerictests
>> +++ b/tools/testing/selftests/sysctl/run_numerictests
>> @@ -7,4 +7,7 @@ TEST_STR=$(( $ORIG + 1 ))
>>
>> . ./common_tests
>>
>> +if [ ! -z ${val} ]; then
>> + echo ${val} > ${WRITES_STRICT}
>> +fi
>> exit $rc
>> diff --git a/tools/testing/selftests/sysctl/run_stringtests
>> b/tools/testing/selftests/sysctl/run_stringtests
>> index 90a9293d520c..ae98d66a9ec6 100755
>> --- a/tools/testing/selftests/sysctl/run_stringtests
>> +++ b/tools/testing/selftests/sysctl/run_stringtests
>> @@ -74,4 +74,7 @@ else
>> echo "ok"
>> fi
>>
>> +if [ ! -z ${val} ]; then
>> + echo ${val} > ${WRITES_STRICT}
>> +fi
>> exit $rc
>> --
>> 2.12.2
>
>
> Best,
> Sumit.
[toc] | [prev] | [next] | [standalone]
| From | Kees Cook <keescook@chromium.org> |
|---|---|
| Date | 2017-06-19 21:10 +0200 |
| Message-ID | <tUadL-6Iy-99@gated-at.bofh.it> |
| In reply to | #1667575 |
On Fri, Jun 16, 2017 at 2:28 AM, Orson Zhai <orson.zhai@linaro.org> wrote:
> Sysctl test will fail in some items if the value of /proc/sys/kernel
> /sysctrl_writes_strict is 0 as the default value in kernel older than v4.5.
>
> Make this test more robust and compatible with older kernels by checking and
> update sysctrl_writes_strict value and restore it when test is done.
>
> Signed-off-by: Orson Zhai <orson.zhai@linaro.org>
> ---
> tools/testing/selftests/sysctl/common_tests | 14 ++++++++++++++
> tools/testing/selftests/sysctl/run_numerictests | 3 +++
> tools/testing/selftests/sysctl/run_stringtests | 3 +++
> 3 files changed, 20 insertions(+)
>
> diff --git a/tools/testing/selftests/sysctl/common_tests
> b/tools/testing/selftests/sysctl/common_tests
> index 17d534b1b7b4..f5c5c51d16f3 100644
> --- a/tools/testing/selftests/sysctl/common_tests
> +++ b/tools/testing/selftests/sysctl/common_tests
> @@ -63,6 +63,20 @@ else
> echo "ok"
> fi
>
> +echo -n "Checking writes strict setting ... "
> +WRITES_STRICT="${SYSCTL}/kernel/sysctl_writes_strict"
> +if [ ! -e ${WRITES_STRICT} ]; then
> + echo "FAIL, but skip in case of old kernel" >&2
> +else
> + val=$(cat ${WRITES_STRICT})
Can you give this a more meaningful name instead of "val". Perhaps "old_strict"?
> + if [ "$val" = "1" ]; then
> + echo "ok"
> + else
> + echo "FAIL, strict value is 0 but force to 1 to continue" >&2
> + echo "1" > ${WRITES_STRICT}
> + fi
> +fi
> +
> # Now that we've validated the sanity of "set_test" and "set_orig",
> # we can use those functions to set starting states before running
> # specific behavioral tests.
> diff --git a/tools/testing/selftests/sysctl/run_numerictests
> b/tools/testing/selftests/sysctl/run_numerictests
> index 8510f93f2d14..c0a98fd82c5c 100755
> --- a/tools/testing/selftests/sysctl/run_numerictests
> +++ b/tools/testing/selftests/sysctl/run_numerictests
> @@ -7,4 +7,7 @@ TEST_STR=$(( $ORIG + 1 ))
>
> . ./common_tests
>
> +if [ ! -z ${val} ]; then
> + echo ${val} > ${WRITES_STRICT}
> +fi
> exit $rc
> diff --git a/tools/testing/selftests/sysctl/run_stringtests
> b/tools/testing/selftests/sysctl/run_stringtests
> index 90a9293d520c..ae98d66a9ec6 100755
> --- a/tools/testing/selftests/sysctl/run_stringtests
> +++ b/tools/testing/selftests/sysctl/run_stringtests
> @@ -74,4 +74,7 @@ else
> echo "ok"
> fi
>
> +if [ ! -z ${val} ]; then
> + echo ${val} > ${WRITES_STRICT}
> +fi
> exit $rc
And instead of open-coding these, can you instead add an "exit_test"
function to common_tests, which will do this and the exit $rc?
-Kees
--
Kees Cook
Pixel Security
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web