----- Original Message -----
> From: "Zeng Linggang" <zenglg...@cn.fujitsu.com>
> To: "Cyril Hrubis" <chru...@suse.cz>
> Cc: "ltp-list" <ltp-list@lists.sourceforge.net>, "naresh kamboju"
> <naresh.kamb...@linaro.org>, "milosz wasilewski"
> <milosz.wasilew...@linaro.org>
> Sent: Monday, 27 July, 2015 10:18:56 AM
> Subject: Re: [LTP] [PATCH v2] cpuhotplug05.sh: Restore part of the code and
> add some comments
>
> Hello,
>
> Ping.
Hi,
I tried both old and new systat to see the behaviour. The check looks good to
me, pushed.
Regards,
Jan
>
> Thanks,
> Zeng
> On Tue, 2015-07-07 at 09:55 +0800, Zeng Linggang wrote:
> > In commit 841c0eebc50068d6595b97b50ae0ee8e8f57c8ca:
> > We changed "sar 1 0" to "sar 1", but it make the test not work well
> > before 'sysstat-8.1.4(include sar)'. The former code has used
> > "ps -C sar" to check, so restore them and add some comments to remind
> > users.
> >
> > Signed-off-by: Zeng Linggang <zenglg...@cn.fujitsu.com>
> > ---
> > testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug05.sh | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git
> > a/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug05.sh
> > b/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug05.sh
> > index 4491d3a..60382c8 100755
> > --- a/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug05.sh
> > +++ b/testcases/kernel/hotplug/cpu_hotplug/functional/cpuhotplug05.sh
> > @@ -81,11 +81,13 @@ TST_CLEANUP=do_clean
> > until [ $LOOP_COUNT -gt $HOTPLUG05_LOOPS ]; do
> >
> > # Start up SAR and give it a couple cycles to run
> > - sar 1 &>/dev/null &
> > + sar 1 0 &>/dev/null &
> > sleep 2
> > + # "sar 1 0" is supported before 'sysstat-8.1.4(include sar)',
> > + # after that use "sar 1" instead of. Use 'ps -C sar' to check.
> > if ps -C sar &>/dev/null; then
> > pkill sar
> > - sar -P ALL 1 > $TMP/log_$$ &
> > + sar -P ALL 1 0 > $TMP/log_$$ &
> > else
> > sar -P ALL 1 > $TMP/log_$$ &
> > fi
>
>
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
