Re: [PATCH v4] topology: make core_mask include at least cluster_siblings

From: Darren Hart
Date: Tue Apr 05 2022 - 21:56:52 EST


On Mon, Apr 04, 2022 at 07:29:20PM -0700, Darren Hart wrote:
...
> A bit more context on the state of review:
>
> Several folks reviewed, but I didn't add their Reviewed-by since I added the
> IS_ENABLED(CONFIG_SCHED_CLUSTER) test since they reviewed it last. This change
> preserves the stated intent of the change when CONFIG_SCHED_CLUSTER is disabled.
>
> Barry Song - Suggested this approach
> Vincent Guittot - informal review with reservations
> Sudeep Holla - Acked-by
> Dietmar Eggemann - informal review (added to Cc, apologies for the omission Dietmar)

Dietmar responded with his:

Reviewed-by: Dietmar Eggemann <dietmar.eggemann@xxxxxxx>

--
Darren Hart
Ampere Computing / OS and Kernel