Re: [PATCH 5.19 332/365] riscv: dts: sifive: Add fu540 topology information

From: Greg KH
Date: Tue Aug 23 2022 - 10:04:19 EST


On Tue, Aug 23, 2022 at 09:49:35AM +0000, Conor.Dooley@xxxxxxxxxxxxx wrote:
> On 23/08/2022 09:03, Greg Kroah-Hartman wrote:
> > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >
> > From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> >
> > [ Upstream commit af8f260abc608c06e4466a282b53f1e2dc09f042 ]
> >
> > The fu540 has no cpu-map node, so tools like hwloc cannot correctly
> > parse the topology. Add the node using the existing node labels.
> >
> > Reported-by: Brice Goglin <Brice.Goglin@xxxxxxxx>
> > Link: https://github.com/open-mpi/hwloc/issues/536
> > Signed-off-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> > Link: https://lore.kernel.org/r/20220705190435.1790466-3-mail@xxxxxxxxxxx
> > Signed-off-by: Palmer Dabbelt <palmer@xxxxxxxxxxxx>
> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
>
> Hey Greg,
> I pointed out on the AUTOSEL'd version of these patches that
> adding the optional dt property papers over the problem rather than
> really fixing it & Sudeep suggested the time that these patches were
> not stable worthy, hence the lack of a CC: stable.
>
> The following has been merged into riscv/for-next & is pending for
> arm64/driver core as an actual fix for RISC-V's default topology
> reporting:
> https://lore.kernel.org/linux-riscv/4849490e-b362-c13a-c2e4-82acc3268a3f@xxxxxxxxxxxxx/#t
>
> As I said to Sasha, I defer to your (plural) better judgement here,
> but just so that you're aware of the context.

Thanks for letting me know, I've now dropped it from all stable release
queues.

greg k-h