Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 28 Jul 2021 19:25:21 +0200
From:   Michal =?iso-8859-1?Q?Koutn=FD?= <mkoutny@suse.com>
To:     Waiman Long <llong@redhat.com>
Cc:     Tejun Heo <tj@kernel.org>, Zefan Li <lizefan.x@bytedance.com>,
        Johannes Weiner <hannes@cmpxchg.org>,
        Jonathan Corbet <corbet@lwn.net>,
        Shuah Khan <shuah@kernel.org>, cgroups@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
        linux-kselftest@vger.kernel.org,
        Andrew Morton <akpm@linux-foundation.org>,
        Roman Gushchin <guro@fb.com>, Phil Auld <pauld@redhat.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Juri Lelli <juri.lelli@redhat.com>,
        Frederic Weisbecker <frederic@kernel.org>,
        Marcelo Tosatti <mtosatti@redhat.com>
Subject: Re: [PATCH v3 6/9] cgroup/cpuset: Add a new isolated cpus.partition
 type
Message-ID: <20210728172521.GE7584@blackbody.suse.cz>
References: <20210720141834.10624-1-longman@redhat.com>
 <20210720141834.10624-7-longman@redhat.com>
 <20210728160900.GA8905@blackbody.suse.cz>
 <f1afbd9e-d16b-c972-c3c0-022a05cec2a6@redhat.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="k4f25fnPtRuIRUb3"
Content-Disposition: inline
In-Reply-To: <f1afbd9e-d16b-c972-c3c0-022a05cec2a6@redhat.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk


--k4f25fnPtRuIRUb3
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 28, 2021 at 12:27:58PM -0400, Waiman Long <llong@redhat.com> wr=
ote:
> PRS_ERROR cannot be passed to update_prstate(). For this patchset, PRS_ER=
ROR
> can only be set by changes in hotplug. The current design will maintain t=
he
> 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=FD <mkoutny@suse.com>


--k4f25fnPtRuIRUb3
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEEoQaUCWq8F2Id1tNia1+riC5qSgFAmEBkvoACgkQia1+riC5
qSjxqg/8DOuQ6BV0vUCrSyyP0k+Wyx7QwrY7UHosDiAVrUYlzfd0NEhckQf7fDr4
faf5/E/Y0t99oRaBYnPqtaqUjtiUUB4e7L/uTxrfGaeOT2SyAHjq5QHtFcdSOHgz
OfinfqEGFuMQLo2vbWtuKaPs6avaPT6443FT/WpG29FHkB7wRPuBAS0f3USLQ/F4
/GqawCdjKCZ4pP59LQpb5jSELNtQKaxe6MpZJ0tcj9JpFgnR6XZcwmxe0FP2hHtY
JzcnXUfMW3CU4yynukd5rEwrCiVS1T5w9XLh0d/MrvbamMEfwBMY8hoVH3md32X4
oTVdlW1IrKkm+VXoW/71WTzKTlkXof/MxMe52QigiOj25g+W/R2CTAoaCKVY5Ou3
RgQNljmvKmN7kh14Zc469PLC8R8t+FYVdMM4xmK+HjbFOBAaymWx6eUHejfQNba0
sgulNpYQjzeR+OBqqt7NI+KLu9s1yag6ZY+N2bUtOWxsdMjuU4vY0yWuHw+SnrTF
itkJg/NIkUmFccnq1ToOAxHyC5vA6ux8rUDBPnJduXr1czXZKi8zoy31h4uq6CAF
iWqDZdmi9dOB9A3rbjOm8BP0vHrs7s3tSLc0c4tu6VncM0mosp6dDGC8RCam3YCG
mnFetTKvSLbNBN/6k3f6nKLX6AGzNHs45EiSi7ut17NMVk5/5EU=
=i07x
-----END PGP SIGNATURE-----

--k4f25fnPtRuIRUb3--