[systemd-devel] [PATCH] unit: Set KillMode=mixed for services which use sulogin

[Lennart Poettering]

On Wed, 13.05.15 16:18, Michael Biebl (mbiebl at gmail.com) wrote:

> 2015-05-13 15:45 GMT+02:00 Lennart Poettering <lennart at poettering.net>:
> > Well, I am pretty sure that in this case, it should be sulogin that
> > propagates the shutdown request to the shell it spawned, but we should
> > not do it otherwise.
> >
> > Note that by default we don't even clean up processes of unprivileged
> > users on logout. You have to turn this on via KillUserProcesses=
> > explicitly. And if we don't do this for unprivileged users, we
> > certainly shouldn't do it for debug shells either....
> 
> Well, it's not the debug shell, it's emergency/rescue mode.
> You typically end up there, because you have a fatal (config) error.

Well, if you in either mode, you ran into problems, and you have to
debug them now. I am pretty sure we should make more people unhappy if
we killed everything they forked off from these recovery shells, then
we'd make happy by cleaning up after them...

KillMode=mixed appears OK as a temporary fix to work-around a broken
sulogin version, but I dont think it makes a good default otherwise.

Lennart

-- 
Lennart Poettering, Red Hat