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

From: Michal Koutný
Date: Mon Jul 15 2019 - 12:51:21 EST


Hello Patrick.

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

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.

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

Michal