Re: [PATCH 2/4] cr_tests/ptree1: count tasks accurately

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 26, 2010 at 01:51:52PM -0600, Nathan Lynch wrote:
> "ps aux |grep $TEST_CMD | wc -l" tends to also count the grep process.
> The test script already assumes pidof is available, so just use that.
> 
> Signed-off-by: Nathan Lynch <ntl@xxxxxxxxx>

Acked-by: Matt Helsley <matthltc@xxxxxxxxxx>

> ---
>  process-tree/run-ptree1.sh |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/process-tree/run-ptree1.sh b/process-tree/run-ptree1.sh
> index 6d56a66..d0cefff 100755
> --- a/process-tree/run-ptree1.sh
> +++ b/process-tree/run-ptree1.sh
> @@ -174,7 +174,7 @@ while [ $cnt -lt 15 ]; do
>  	freezerdir=$freezermountpoint/$pid
>  	freeze_pid $pid
> 
> -	num_pids1=`ps aux |grep $TEST_CMD | wc -l`
> +	num_pids1=`pidof $TEST_CMD | wc -w`
> 
>  	create_fs_snapshot
> 
> @@ -196,7 +196,7 @@ while [ $cnt -lt 15 ]; do
> 
>  	sleep 3;
> 
> -	num_pids2=`ps aux |grep $TEST_CMD | wc -l`
> +	num_pids2=`pidof $TEST_CMD | wc -w`
>  	ps aux |grep $TEST_CMD >> $SCRIPT_LOG
>  	$ECHO "\t- num_pids1 $num_pids1, num_pids2 $num_pids2";
> 
> -- 
> 1.6.0.6
> 
> _______________________________________________
> Containers mailing list
> Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/containers

[Index of Archives]     [Cgroups]     [Netdev]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux