Re: [PATCH v11 0/5] Add utilization clamping support (CGroups API)

From: Patrick Bellasi
Date: Tue Jul 16 2019 - 10:03:26 EST

On 15-Jul 18:51, Michal Koutný wrote:
> Hello Patrick.

Hi Michal,

> I took a look at your series and I've posted some notes to your patches.

thanks for your review!

> One applies more to the series overall -- I see there is enum uclamp_id
> defined but at many places (local variables, function args) int or
> unsigned int is used. Besides the inconsistency, I think it'd be nice to
> use the enum at these places.

Right, I think in some of the original versions I had few code paths
where it was not possible to use enum values. That seems no more the case.

Since this change is likely affecting also core bits already merged in
5.3, in v12 I'm going to add a bulk rename patch at the end of the
series, so that we can keep a better tracking of this change.

> (Also, I may suggest CCing ML cgroups@xxxxxxxxxxxxxxx where more eyes
> may be available to the cgroup part of your series.)

Good point, I'll add that for the upcoming v12 posting.


#include <best/regards.h>

Patrick Bellasi