[systemd-devel] [PATCH] Set cgroup path to uid for user sessions.

[Kok, Auke-jan H]

On Fri, Mar 22, 2013 at 4:11 PM, Lennart Poettering
<lennart at poettering.net> wrote:
> On Tue, 19.03.13 21:34, Auke Kok (auke-jan.h.kok at intel.com) wrote:
>
>> Without this patch, I'm seeing cgroup paths for user sessions with
>> both the user name, and the uid created, which can't be intended.
>>
>> Spotted through user-session-units use.
>
> Hmm, I am missing something here: where do we generate a cgroup path
> with the uid currently? It's all use rname for the cgroup name right,
> now, no? What am I missing?

the user at .service (and thus user-session at .service from user-session-units) have:

ControlGroup=%R/user/%I/shared cpu:/ ...
ControlGroupModify=yes

and we've had to symlink the instances as user@<uid>.service since v185 or so..

Now, just from a sanity perspective I really think we should be using
uid's here and not usernames, just like /run/user/<uid>...


Auke