public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH v2] cgroups/cpuacct_setup: fix task_kill() search for process
@ 2011-04-20 14:00 Jan Stancek
  0 siblings, 0 replies; 3+ messages in thread
From: Jan Stancek @ 2011-04-20 14:00 UTC (permalink / raw)
  To: ltp-list

[-- Attachment #1: Type: text/plain, Size: 374 bytes --]


1. "i* in "grep cpuacct_taski*" seems pointless as
its target is cpuacct_task01

2. using cut fails when ps indents the output as following:
   99 cpuacct_task01
  112 cpuacct_task01
1111 cpuacct_task01

Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
  .../kernel/controllers/cpuacct/cpuacct_setup.sh    |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)



[-- Attachment #2: 0001-cgroups-cpuacct_setup-fix-task_kill-search-for-proce.patch --]
[-- Type: text/x-patch, Size: 470 bytes --]

diff --git a/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh b/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
index 47cd1da..2443a28 100755
--- a/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
+++ b/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
@@ -58,7 +58,7 @@ cleanup ()
 }
 task_kill ()
 {
-	for i in `ps -e | grep cpuacct_taski* | cut -d" " -f1`
+	for i in `ps -e | grep cpuacct_task | awk '{print $1}'`
 	do
 		kill -SIGUSR1 $i
 	done


[-- Attachment #3: Type: text/plain, Size: 438 bytes --]

------------------------------------------------------------------------------
Benefiting from Server Virtualization: Beyond Initial Workload 
Consolidation -- Increasing the use of server virtualization is a top
priority.Virtualization can reduce costs, simplify management, and improve 
application availability and disaster protection. Learn more about boosting 
the value of server virtualization. http://p.sf.net/sfu/vmware-sfdev2dev

[-- Attachment #4: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

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

* [LTP] [PATCH v2] cgroups/cpuacct_setup: fix task_kill() search for process
@ 2011-06-07 14:19 Jan Stancek
  2011-07-11 13:49 ` Cyril Hrubis
  0 siblings, 1 reply; 3+ messages in thread
From: Jan Stancek @ 2011-06-07 14:19 UTC (permalink / raw)
  To: ltp-list

[-- Attachment #1: Type: text/plain, Size: 374 bytes --]


1. "i* in "grep cpuacct_taski*" seems pointless as
its target is cpuacct_task01

2. using cut fails when ps indents the output as following:
   99 cpuacct_task01
  112 cpuacct_task01
1111 cpuacct_task01

Signed-off-by: Jan Stancek <jstancek@redhat.com>
---
  .../kernel/controllers/cpuacct/cpuacct_setup.sh    |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)



[-- Attachment #2: 0001-cgroups-cpuacct_setup-fix-task_kill-search-for-proce.patch --]
[-- Type: text/x-patch, Size: 470 bytes --]

diff --git a/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh b/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
index 47cd1da..2443a28 100755
--- a/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
+++ b/testcases/kernel/controllers/cpuacct/cpuacct_setup.sh
@@ -58,7 +58,7 @@ cleanup ()
 }
 task_kill ()
 {
-	for i in `ps -e | grep cpuacct_taski* | cut -d" " -f1`
+	for i in `ps -e | grep cpuacct_task | awk '{print $1}'`
 	do
 		kill -SIGUSR1 $i
 	done


[-- Attachment #3: Type: text/plain, Size: 318 bytes --]

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev

[-- Attachment #4: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

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

* Re: [LTP] [PATCH v2] cgroups/cpuacct_setup: fix task_kill() search for process
  2011-06-07 14:19 Jan Stancek
@ 2011-07-11 13:49 ` Cyril Hrubis
  0 siblings, 0 replies; 3+ messages in thread
From: Cyril Hrubis @ 2011-07-11 13:49 UTC (permalink / raw)
  To: Jan Stancek; +Cc: ltp-list

Hi!
> 
> 1. "i* in "grep cpuacct_taski*" seems pointless as
> its target is cpuacct_task01
> 
> 2. using cut fails when ps indents the output as following:
>   99 cpuacct_task01
>  112 cpuacct_task01
> 1111 cpuacct_task01
> 
> Signed-off-by: Jan Stancek <jstancek@redhat.com>

Commited, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security 
threats, fraudulent activity, and more. Splunk takes this data and makes 
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

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

end of thread, other threads:[~2011-07-11 13:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-20 14:00 [LTP] [PATCH v2] cgroups/cpuacct_setup: fix task_kill() search for process Jan Stancek
  -- strict thread matches above, loose matches on Subject: below --
2011-06-07 14:19 Jan Stancek
2011-07-11 13:49 ` Cyril Hrubis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox