[oe-commits] Stefan Stanacar : oeqa/utils/qemurunner: use the right kill call so we don 't send SIGTERM to bitbake-worker
git at git.openembedded.org
git at git.openembedded.org
Sun Mar 16 23:01:30 UTC 2014
Module: openembedded-core.git
Branch: master-next
Commit: f4bf21b40ec6fa07c4a7bb450e3cfae12128dc2c
URL: http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=f4bf21b40ec6fa07c4a7bb450e3cfae12128dc2c
Author: Stefan Stanacar <stefanx.stanacar at intel.com>
Date: Tue Mar 11 14:11:08 2014 +0200
oeqa/utils/qemurunner: use the right kill call so we don't send SIGTERM to bitbake-worker
After recent bitbake-worker changes [1] this was killing the bitbake-worker,
which isn't what we want and kill(-pid) was the wrong call anyway.
runqemu.pid is the right PGID as the process was started with preexec_fn=setpgrp
(so no need to do os.getpgid(self.runqemu.pid))
[1] http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=15688798520896690561824b2fdc227c8a365c82
Signed-off-by: Stefan Stanacar <stefanx.stanacar at intel.com>
Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
---
meta/lib/oeqa/utils/qemurunner.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py
index 5366a63..337b587 100644
--- a/meta/lib/oeqa/utils/qemurunner.py
+++ b/meta/lib/oeqa/utils/qemurunner.py
@@ -163,13 +163,13 @@ class QemuRunner:
if self.runqemu:
bb.note("Sending SIGTERM to runqemu")
- os.kill(-self.runqemu.pid,signal.SIGTERM)
+ os.killpg(self.runqemu.pid, signal.SIGTERM)
endtime = time.time() + self.runqemutime
while self.runqemu.poll() is None and time.time() < endtime:
time.sleep(1)
if self.runqemu.poll() is None:
bb.note("Sending SIGKILL to runqemu")
- os.kill(-self.runqemu.pid,signal.SIGKILL)
+ os.killpg(self.runqemu.pid, signal.SIGKILL)
self.runqemu = None
if self.server_socket:
self.server_socket.close()
More information about the Openembedded-commits
mailing list