Re: [PATCH v3 6/9] cgroup/cpuset: Add a new isolated cpus.partition type

From: Michal Koutný
Date: Wed Jul 28 2021 - 13:25:26 EST


On Wed, Jul 28, 2021 at 12:27:58PM -0400, Waiman Long <llong@xxxxxxxxxx> wrote:
> PRS_ERROR cannot be passed to update_prstate(). For this patchset, PRS_ERROR
> can only be set by changes in hotplug. The current design will maintain the
> set flag (CS_SCHED_LOAD_BALANCE) and use it to decide to switch back to
> PRS_ENABLED or PRS_ISOLATED when the cpus are available again.

I see it now, thanks. (I still find a bit weird that the "isolated"
partition will be shown as "root invalid" when it's lacking cpus
(instead of "isolated invalid" and returning to "isolated") but I can
understand the approach of having just one "root invalid" for all.)

This patch can have
Reviewed-by: Michal Koutný <mkoutny@xxxxxxxx>

Attachment: signature.asc
Description: Digital signature