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

Michael Biebl mbiebl at gmail.com
Wed May 13 06:32:51 PDT 2015


2015-05-13 15:19 GMT+02:00 Lennart Poettering <lennart at poettering.net>:
> sulogin generally does not set up a PAM session, and we indeed should
> allow processes like screen staying around in such a context. Hence
> KillMode=process is actually the right choice for all these services,
> indeed.

Do you really think it makes sense to start screen from emergency/rescue mode?
Imho those are the cases where you don't actually want stuff to stay
around after you log out.

> Hence I figure the status quo for all of this is pretty OK already...

Well, I was intending to commit my original patch, which only uses
KillMode=mixed for services which use sulogin, i.e.
emergency.service, rescue.service and console-shell.service.

See the original bug that triggered this patch [1]. We don't really
want a stray bash process to stay around which potentially fights with
sulogin over the input.

Michael

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=784238

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?


More information about the systemd-devel mailing list