diff options
author | Robert Yang <liezhi.yang@windriver.com> | 2014-11-17 02:27:36 -0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-12-31 10:09:51 +0000 |
commit | 72536d4e0cc3379001b730950afa012f7a96a79b (patch) | |
tree | 324804a984e860cbd47a738db54a09b121108dc9 | |
parent | f12c738d3dc1f0fd105d457385511440024bffab (diff) | |
download | bitbake-72536d4e0cc3379001b730950afa012f7a96a79b.tar.gz |
bitbake-worker: exit normally when SIGHUP
Fixed:
1) Run "bitbake recipe" in the terminal
2) Close the terminal while building
3) $ ps aux | grep bitbake-worker
There will be many processes, and they will keep the resources (e.g.,
memory), and won't exit unless kill or kill -9.
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-x | bin/bitbake-worker | 9 | ||||
-rw-r--r-- | lib/bb/cooker.py | 7 |
2 files changed, 14 insertions, 2 deletions
diff --git a/bin/bitbake-worker b/bin/bitbake-worker index dde2c9c8f..371c99a67 100755 --- a/bin/bitbake-worker +++ b/bin/bitbake-worker @@ -147,6 +147,8 @@ def fork_off_task(cfg, data, workerdata, fn, task, taskname, appends, taskdepdat pipein.close() signal.signal(signal.SIGTERM, sigterm_handler) + # Let SIGHUP exit as SIGTERM + signal.signal(signal.SIGHUP, sigterm_handler) # Save out the PID so that the event can include it the # events @@ -266,9 +268,14 @@ class BitbakeWorker(object): self.build_pipes = {} signal.signal(signal.SIGTERM, self.sigterm_exception) + # Let SIGHUP exit as SIGTERM + signal.signal(signal.SIGHUP, self.sigterm_exception) def sigterm_exception(self, signum, stackframe): - bb.warn("Worker recieved SIGTERM, shutting down...") + if signum == signal.SIGTERM: + bb.warn("Worker recieved SIGTERM, shutting down...") + elif signum == signal.SIGHUP: + bb.warn("Worker recieved SIGHUP, shutting down...") self.handle_finishnow(None) signal.signal(signal.SIGTERM, signal.SIG_DFL) os.kill(os.getpid(), signal.SIGTERM) diff --git a/lib/bb/cooker.py b/lib/bb/cooker.py index c6c69c30e..a900b07a8 100644 --- a/lib/bb/cooker.py +++ b/lib/bb/cooker.py @@ -153,9 +153,14 @@ class BBCooker: self.parser = None signal.signal(signal.SIGTERM, self.sigterm_exception) + # Let SIGHUP exit as SIGTERM + signal.signal(signal.SIGHUP, self.sigterm_exception) def sigterm_exception(self, signum, stackframe): - bb.warn("Cooker recieved SIGTERM, shutting down...") + if signum == signal.SIGTERM: + bb.warn("Cooker recieved SIGTERM, shutting down...") + elif signum == signal.SIGHUP: + bb.warn("Cooker recieved SIGHUP, shutting down...") self.state = state.forceshutdown def setFeatures(self, features): |