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

[Lennart Poettering]

On Tue, 12.05.15 14:36, Michael Biebl (mbiebl at gmail.com) wrote:

> 2015-05-12 0:03 GMT+02:00 Lennart Poettering <lennart at poettering.net>:
> > On Tue, 12.05.15 00:01, Lennart Poettering (lennart at poettering.net) wrote:
> >
> >> On Mon, 11.05.15 23:50, Michael Biebl (mbiebl at gmail.com) wrote:
> >>
> >> > 2015-05-08 17:43 GMT+02:00 Michael Biebl <mbiebl at gmail.com>:
> >> > > See http://lists.freedesktop.org/archives/systemd-devel/2015-May/031536.html
> >> >
> >> > Lennart, are you ok if I commit this?
> >>
> >> Yes, please!
> >
> > But actually, I agree with Zbigniew, can you please replace all
> > occurences of Killmode=process with KillMode=mixed, with the exception
> > of debug-shell.service. Also, if you'd add a comment there explaining
> > that that file sticks to KillMode=process to not affect backgrounded
> > debugging process, things would even be better!
> 
> Hm, I just tested KillMode=mixed with getty at .service on installations
> which didn't have libpam-systemd enabled.

Ah, humm. 

So, we don't really care about installations without pam_systemd
upstream regarding this. However, logind is actually responsible for
killing user process on logout if that's configured via logind.conf.

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.

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

Lennart

-- 
Lennart Poettering, Red Hat