[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220321143021.GB11145@willie-the-truck>
Date: Mon, 21 Mar 2022 14:30:22 +0000
From: Will Deacon <will@...nel.org>
To: Sudeep Holla <sudeep.holla@....com>
Cc: Darren Hart <darren@...amperecomputing.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Dietmar Eggemann <dietmar.eggemann@....com>,
Vincent Guittot <vincent.guittot@...aro.org>,
LKML <linux-kernel@...r.kernel.org>,
Linux Arm <linux-arm-kernel@...ts.infradead.org>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Catalin Marinas <Catalin.Marinas@....com>,
Peter Zijlstra <peterz@...radead.org>,
Valentin Schneider <Valentin.Schneider@....com>,
"D . Scott Phillips" <scott@...amperecomputing.com>,
Ilkka Koskinen <ilkka@...amperecomputing.com>,
stable@...r.kernel.org, Barry Song <21cnbao@...il.com>
Subject: Re: [PATCH v3] topology: make core_mask include at least
cluster_siblings
On Wed, Mar 16, 2022 at 03:55:41PM +0000, Sudeep Holla wrote:
> On Wed, Mar 16, 2022 at 08:20:32AM -0700, Darren Hart wrote:
> > On Wed, Mar 16, 2022 at 03:48:50PM +0100, Dietmar Eggemann wrote:
>
> [...]
>
> > >
> > > Yeah, I can see your point. It's the smaller hack. My solution just
> > > prevents us to manipulate the coregroup mask only to get the MC layer
> > > degenerated by the core topology code. But people might say that's a
> > > clever thing to do here. So I'm fine with your original solution as well.
> > >
> > > [...]
> >
> > Thanks Dietmar,
> >
> > Sudeep, do we have sufficient consensus to pull in this patch?
>
> Indeed. I have already Acked, and sure after all these discussions we have
> concluded that this is the best we can do though not matches everyone's taste.
>
> Greg or Will(not sure why he had asked since v3 doesn't touch arm64),
> Can one of you pick this patch ?
Right, this doesn't touch arm64 any more so I don't think I'm the right
person to queue it.
Will
Powered by blists - more mailing lists