Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2

From: Pavel Machek
Date: Mon Jul 30 2018 - 13:39:46 EST


On Mon 2018-07-30 11:40:35, Johannes Weiner wrote:
> On Sat, Jul 28, 2018 at 12:01:23AM +0200, Pavel Machek wrote:
> > > How do you use this feature?
> > >
> > > A kernel with CONFIG_PSI=y will create a /proc/pressure directory with
> > > 3 files: cpu, memory, and io. If using cgroup2, cgroups will also
> >
> > Could we get the config named CONFIG_PRESSURE to match /proc/pressure?
> > "PSI" is little too terse...
>
> I'd rather have the internal config symbol match the naming scheme in
> the code, where psi is a shorter, unique token as copmared to e.g.
> pressure, press, prsr, etc.

I'd do "pressure", really. Yes, psi is shorter, but I'd say that
length is not really important there.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature