Re: [PATCH 0/10] psi: pressure stall information for CPU, memory, and IO v2
From: Randy Dunlap
Date: Mon Jul 30 2018 - 13:54:21 EST
On 07/30/2018 10:51 AM, Tejun Heo wrote:
> Hello,
>
> On Mon, Jul 30, 2018 at 07:39:40PM +0200, Pavel Machek wrote:
>>> 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.
>
> This is an extreme bikeshedding without any relevance. You can make
> suggestions but please lay it to the rest. There isn't any general
> consensus against the current name and you're just trying to push your
> favorite name without proper justifications after contributing nothing
> to the project. Please stop.
>
> Thanks.
I'd say he's trying to make something that is readable and easier to
understand for users.
Thanks.
--
~Randy