process: add support for cgroups

Authored by topimiettinen on Apr 24 2019, 1:03 PM.



Signed-off-by: Topi Miettinen <>

Diff Detail

R111 KSysguard Library
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
topimiettinen created this revision.Apr 24 2019, 1:03 PM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 24 2019, 1:03 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
topimiettinen requested review of this revision.Apr 24 2019, 1:03 PM
apol added a subscriber: apol.Apr 24 2019, 1:53 PM
apol added inline comments.

Just don't return &? I don't understand the comment.



may as well make it non-ref now.

I assume that ref pattern existed as an old way of sscanf-ing directly into the member variables skipping the setBlah call.


The control group (cgroup) where this process was belongs.

That doesn't scan right.

s/was // ?

update per comments from @davidedmundson and @apol

topimiettinen marked 3 inline comments as done.Apr 24 2019, 4:50 PM
davidedmundson accepted this revision.Apr 24 2019, 4:50 PM
This revision is now accepted and ready to land.Apr 24 2019, 4:50 PM

It looks like the CGroup rows are visually not vertically aligned to center like other cells, but instead they are aligned to the top. However, I'd expect that the default item of switch in ProcessModel.cpp:1693 should implement this, like it does for e.g. HeadingName. So I'm a bit puzzled why this happens.

Trim trailing newline for proper vertical alignment

phabricate only one git commit, sorry

Looks much better!

fix copy-pasta

hide cgroup column by default

This revision was automatically updated to reflect the committed changes.