From: Ed Bartosh <ed.bartosh@linux.intel.com>
To: toaster@yoctoproject.org
Subject: [PATCH 0/4] improved process management
Date: Tue, 23 Aug 2016 14:44:40 +0300 [thread overview]
Message-ID: <cover.1471951968.git.ed.bartosh@linux.intel.com> (raw)
Hi,
This is a set of improvements for process management in toaster script.
It includes a bit of cleanup (first 2 patches) and quite serious changes in
the way toaster script starts and stops runbuild and runserver processes:
- usage of process groups (PGID) instead of PID
- usage of pid namespaces http://man7.org/linux/man-pages/man7/pid_namespaces.7.html
Both changes should hopefully make toaster to stop running processes more reliably.
One side effect and a partial reason of this work:
it should fix #7973: stopping toaster kills ALL django apps running on the machine.
Testing instructions:
- run toaster
- run build in Toaster UI
- wait until you see a lot of build-related processes in 'ps x' output
- stop toaster. expected outcome: no toaster or build-related processes in 'ps x' output
The following changes since commit 34ccad16d40ec0685f405b8d7116250fb58e323d:
buildinfohelper: discover kernel artifacts correctly (2016-08-23 14:29:32 +0300)
are available in the git repository at:
git://git.yoctoproject.org/poky-contrib ed/toaster/kill-toaster-pid-namespace-7973
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/kill-toaster-pid-namespace-7973
Ed Bartosh (4):
toaster: don't kill toaster on start
toaster: remove handling of .toasterui.pid
toaster: use process group id to kill processes
toaster: run builds in pid namespace
bitbake/bin/toaster | 48 ++++++++++++++++--------------------------------
1 file changed, 16 insertions(+), 32 deletions(-)
--
Regards,
Ed
next reply other threads:[~2016-08-23 11:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-23 11:44 Ed Bartosh [this message]
2016-08-23 11:44 ` [PATCH 1/4] toaster: don't kill toaster on start Ed Bartosh
2016-08-23 11:44 ` [PATCH 2/4] toaster: remove handling of .toasterui.pid Ed Bartosh
2016-08-23 11:44 ` [PATCH 3/4] toaster: use process group id to kill processes Ed Bartosh
2016-08-23 11:44 ` [PATCH 4/4] toaster: run builds in pid namespace Ed Bartosh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1471951968.git.ed.bartosh@linux.intel.com \
--to=ed.bartosh@linux.intel.com \
--cc=toaster@yoctoproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.