All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH] cpuctl_fj: fix syntax error of awk
@ 2009-11-16  2:07 Shi Weihua
  2009-11-16  7:35 ` Garrett Cooper
  0 siblings, 1 reply; 2+ messages in thread
From: Shi Weihua @ 2009-11-16  2:07 UTC (permalink / raw)
  To: ltp-list; +Cc: jpalecek

cpuctl_test_fj's case19-22 failed on my x86_64 box.
----------------------------------------------
awk: print $9
awk: ^ syntax error
pid 8077 cpu_usage
cpuctl_test_fj    1  TFAIL  :  case19    FAIL
awk: print $9
awk: ^ syntax error
pid 8104 cpu_usage
cpuctl_test_fj    1  TFAIL  :  case20    FAIL
awk: print $9
awk: ^ syntax error
pid 8131 cpu_usage
cpuctl_test_fj    1  TFAIL  :  case21    FAIL
awk: print $9
awk: ^ syntax error
pid 8193 cpu_usage
cpuctl_test_fj    1  TFAIL  :  case22    FAIL
----------------------------------------------
I fixed the syntax error of awk in the following patch.

Signed-off-by: Shi Weihua <shiwh@cn.fujitsu.com>
---
--- ltp-full-20091031.orig/testcases/kernel/controllers/cpuctl_fj/run_cpuctl_test_fj.sh	2009-11-02 04:07:14.000000000 -0500
+++ ltp-full-20091031/testcases/kernel/controllers/cpuctl_fj/run_cpuctl_test_fj.sh	2009-11-16 04:51:22.000000000 -0500
@@ -85,7 +85,7 @@ creat_process()
 
 get_cpu_usage()
 {
-	top -b -n 1 -p $1 | tail -2 | head -1 | awk 'print $9' | awk -F "." '{print $1}'
+	top -b -n 1 -p $1 | tail -2 | head -1 | awk '{print $9}' | awk -F "." '{print $1}'
 }
 
 kill_all_pid()


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [LTP] [PATCH] cpuctl_fj: fix syntax error of awk
  2009-11-16  2:07 [LTP] [PATCH] cpuctl_fj: fix syntax error of awk Shi Weihua
@ 2009-11-16  7:35 ` Garrett Cooper
  0 siblings, 0 replies; 2+ messages in thread
From: Garrett Cooper @ 2009-11-16  7:35 UTC (permalink / raw)
  To: Shi Weihua; +Cc: ltp-list, jpalecek

On Sun, Nov 15, 2009 at 6:07 PM, Shi Weihua <shiwh@cn.fujitsu.com> wrote:
> cpuctl_test_fj's case19-22 failed on my x86_64 box.
> ----------------------------------------------
> awk: print $9
> awk: ^ syntax error
> pid 8077 cpu_usage
> cpuctl_test_fj    1  TFAIL  :  case19    FAIL
> awk: print $9
> awk: ^ syntax error
> pid 8104 cpu_usage
> cpuctl_test_fj    1  TFAIL  :  case20    FAIL
> awk: print $9
> awk: ^ syntax error
> pid 8131 cpu_usage
> cpuctl_test_fj    1  TFAIL  :  case21    FAIL
> awk: print $9
> awk: ^ syntax error
> pid 8193 cpu_usage
> cpuctl_test_fj    1  TFAIL  :  case22    FAIL
> ----------------------------------------------
> I fixed the syntax error of awk in the following patch.
>
> Signed-off-by: Shi Weihua <shiwh@cn.fujitsu.com>
> ---
> --- ltp-full-20091031.orig/testcases/kernel/controllers/cpuctl_fj/run_cpuctl_test_fj.sh 2009-11-02 04:07:14.000000000 -0500
> +++ ltp-full-20091031/testcases/kernel/controllers/cpuctl_fj/run_cpuctl_test_fj.sh      2009-11-16 04:51:22.000000000 -0500
> @@ -85,7 +85,7 @@ creat_process()
>
>  get_cpu_usage()
>  {
> -       top -b -n 1 -p $1 | tail -2 | head -1 | awk 'print $9' | awk -F "." '{print $1}'
> +       top -b -n 1 -p $1 | tail -2 | head -1 | awk '{print $9}' | awk -F "." '{print $1}'
>  }
>
>  kill_all_pid()

That code's just ugly anyhow. We shouldn't have to call top, when we
can call ps to do the same thing...

ps -eo 'pid,pcpu' | awk '$1 == "'$1'" { sub(/(\.[[:digit:]])*$/, "",
$2); print $2 }'

Example:

ps -eo 'pid,pcpu' | awk '$1 == "1" { sub(/(\.[[:digit:]])*$/, "", $2);
print $2 }'
0
ps -eo 'pid,pcpu' | awk '$1 == "'$(pidof xfce4-session)'" {
sub(/(\.[[:digit:]])*$/, "", $2); print $2 }'
0
ps -eo 'pid,pcpu' | awk '$1 == "'$(pidof vfork)'" {
sub(/(\.[[:digit:]])*$/, "", $2); print $2 }'
99

That's been committed to CVS.

Thanks for the report!
-Garrett

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-11-16  7:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-16  2:07 [LTP] [PATCH] cpuctl_fj: fix syntax error of awk Shi Weihua
2009-11-16  7:35 ` Garrett Cooper

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.