[systemd-devel] [PATCH] service: don't create extra cgroup for control process when reloading SysV service

Lennart Poettering lennart at poettering.net
Mon Mar 24 15:07:46 PDT 2014


On Thu, 13.03.14 09:40, Lukáš Nykrýn (lnykryn at redhat.com) wrote:

> Exactly. Systemd exec /etc/init.d/foo reload in control "subgroup".
> Than the initscript kills the original deamon, starts a new one and
> quits. Systemd sees that the reload process finished and kills
> remaining processes in the control group, thus kills the daemon.
> 
> This patch works quite fine when the initscripts is using pid files,
> systemd correctly updates the information about main pid.

I see the problem now. But it's really not that simple as the patch.. We
need to restart the PID logic if this shall be supported, and rerrange
things so that we don't use the control cgroup anymore..

But I must say, I really don't like a change like this. Kay, any opinion?

Lennart

-- 
Lennart Poettering, Red Hat


More information about the systemd-devel mailing list