From: Richard Palethorpe <rpalethorpe@suse.de>
To: Joerg Vehlow <lkml@jv-coder.de>
Cc: Joerg Vehlow <joerg.vehlow@aox-tech.de>, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 2/3] cpuset_regression_test: Drop min cpu requirement
Date: Mon, 15 Nov 2021 09:23:56 +0000 [thread overview]
Message-ID: <87sfvxdb64.fsf@suse.de> (raw)
In-Reply-To: <20210623071543.171021-3-lkml@jv-coder.de>
Joerg Vehlow <lkml@jv-coder.de> writes:
> From: Joerg Vehlow <joerg.vehlow@aox-tech.de>
>
> The bug can also be reproduced with only one cpu.
>
> Signed-off-by: Joerg Vehlow <joerg.vehlow@aox-tech.de>
Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com>
> ---
> .../controllers/cpuset/cpuset_regression_test.sh | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh b/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
> index 1dda19704..369fbedae 100755
> --- a/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
> +++ b/testcases/kernel/controllers/cpuset/cpuset_regression_test.sh
> @@ -27,13 +27,6 @@ old_cpu_exclusive_value=1
>
> setup()
> {
> - local cpu_num
> -
> - cpu_num=$(tst_getconf _NPROCESSORS_ONLN)
> - if [ $cpu_num -lt 2 ]; then
> - tst_brk TCONF "We need 2 cpus at least to have test"
> - fi
> -
> if ! is_cgroup_subsystem_available_and_enabled "cpuset"; then
> tst_brk TCONF "Either kernel does not support cpuset controller or feature not enabled"
> fi
> @@ -101,12 +94,12 @@ test()
> fi
>
> # This may trigger the kernel crash
> - echo 0-1 > ${root_cpuset_dir}/testdir/${cpus}
> - [ $? -ne 0 ] && tst_brk TFAIL "'echo 0-1 > ${root_cpuset_dir}/testdir/${cpus}' failed"
> + echo 0 > ${root_cpuset_dir}/testdir/${cpus}
> + [ $? -ne 0 ] && tst_brk TFAIL "'echo 0 > ${root_cpuset_dir}/testdir/${cpus}' failed"
>
> cpus_value=$(cat ${root_cpuset_dir}/testdir/${cpus})
> - if [ "${cpus_value}" != "0-1" ]; then
> - tst_brk TFAIL "${cpus} is '${cpus_value}', expected '0-1'"
> + if [ "${cpus_value}" != "0" ]; then
> + tst_brk TFAIL "${cpus} is '${cpus_value}', expected '0'"
> fi
>
> tst_res TPASS "Bug is not reproducible"
--
Thank you,
Richard.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2021-11-15 9:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-23 7:15 [LTP] [PATCH 0/3] cpuset_regression_test: convert and improve Joerg Vehlow
2021-06-23 7:15 ` [LTP] [PATCH 1/3] cpuset_regression_test: Convert to new api Joerg Vehlow
2021-11-15 9:23 ` Richard Palethorpe
2021-11-15 10:36 ` Richard Palethorpe
2021-06-23 7:15 ` [LTP] [PATCH 2/3] cpuset_regression_test: Drop min cpu requirement Joerg Vehlow
2021-11-15 9:23 ` Richard Palethorpe [this message]
2021-06-23 7:15 ` [LTP] [PATCH 3/3] cpuset_regression_test: Allow running, if groups exist Joerg Vehlow
2021-11-15 9:24 ` Richard Palethorpe
2021-06-23 11:11 ` [LTP] [PATCH 0/3] cpuset_regression_test: convert and improve Richard Palethorpe
2021-06-23 11:20 ` Joerg Vehlow
2021-11-15 9:19 ` Richard Palethorpe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sfvxdb64.fsf@suse.de \
--to=rpalethorpe@suse.de \
--cc=joerg.vehlow@aox-tech.de \
--cc=lkml@jv-coder.de \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.