diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-12-08 21:31:56 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-01-08 09:46:58 +0000 |
commit | 130958cc24ad306684d92a8d965f3772f30e196a (patch) | |
tree | ba2181043478ee4a881b07b1bcc683fe860cc4d2 /bitbake/bin | |
parent | 1393e224b51d07297f615cc53dab4ef3bab72770 (diff) | |
download | poky-130958cc24ad306684d92a8d965f3772f30e196a.tar.gz |
bitbake: bitbake-worker: Use setsid() rather than setpgid()
The bug has a long discussion of this. Basically, in some environments,
the exact details of which aren't understood, a Ctrl+C signal to the
UI is being transmitted to all the process children. Looking at the output
of "ps ax -O tpgid", its clear the main process is still the terminal
owner of these processes.
stty -a on a problematic system shows: "-ignbrk brkint"
and on a working system shows: "-ignbrk -brkint"
The description of brkint would suggest this is the problem, setting up
that terminal environment wasn't able to reproduce the problem though.
It was confirmed that using setsid() caused the problem to be resolved
and is probably the right thing to be doing anyway, so lets do it.
[YOCTO #6949]
(Bitbake rev: 461aa73fff0ab616032d28c4fd0322eb88838be6)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/bin')
-rwxr-xr-x | bitbake/bin/bitbake-worker | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/bitbake/bin/bitbake-worker b/bitbake/bin/bitbake-worker index 371c99a677..8a24161250 100755 --- a/bitbake/bin/bitbake-worker +++ b/bitbake/bin/bitbake-worker | |||
@@ -156,8 +156,11 @@ def fork_off_task(cfg, data, workerdata, fn, task, taskname, appends, taskdepdat | |||
156 | bb.event.worker_fire = worker_child_fire | 156 | bb.event.worker_fire = worker_child_fire |
157 | worker_pipe = pipeout | 157 | worker_pipe = pipeout |
158 | 158 | ||
159 | # Make the child the process group leader | 159 | # Make the child the process group leader and ensure no |
160 | os.setpgid(0, 0) | 160 | # child process will be controlled by the current terminal |
161 | # This ensures signals sent to the controlling terminal like Ctrl+C | ||
162 | # don't stop the child processes. | ||
163 | os.setsid() | ||
161 | # No stdin | 164 | # No stdin |
162 | newsi = os.open(os.devnull, os.O_RDWR) | 165 | newsi = os.open(os.devnull, os.O_RDWR) |
163 | os.dup2(newsi, sys.stdin.fileno()) | 166 | os.dup2(newsi, sys.stdin.fileno()) |