Messages in this thread | | | From | <> | Subject | Re: [GIT PULL] Fix RISC-V's arch-topology reporting | Date | Wed, 7 Sep 2022 16:39:31 +0000 |
| |
On 15/08/2022 23:14, Conor Dooley wrote: > Hey Will/Palmer/Sudeep, > > Catalin suggested [0] dropping the CC: stable for the arm64 patch and > instead making it a specific prereq of the RISC-V patch & making a PR, > so here we are.. I was still up when -rc1 came out so pushed it last > night to get the test coverage, but LKP seems to not have reported a > build success since early on the 13th so not holding my horses! I built > it again for both ARMs and RISC-V myself. > > I tagged it tonight, so it's on conor/linux.git as riscv-topo-on-6.0-rc1 > with the prereq specified. > > Not sure if you want to merge this too Sudeep or if that's up to Greg?
What's the story with this from an arm64 & topology PoV? Palmer merged this into riscv/for-next a couple weeks ago, so just wondering what the craic is on the other fronts.
Thanks, Conor.
> > Thanks, > Conor. > > 0 - https://lore.kernel.org/linux-riscv/Ytac7G1zlq6WW4jt@arm.com/ > > The following changes since commit 568035b01cfb107af8d2e4bd2fb9aea22cf5b868: > > Linux 6.0-rc1 (2022-08-14 15:50:18 -0700) > > are available in the Git repository at: > > https://git.kernel.org/pub/scm/linux/kernel/git/conor/linux.git/ tags/riscv-topo-on-6.0-rc1 > > for you to fetch changes up to fbd92809997a391f28075f1c8b5ee314c225557c: > > riscv: topology: fix default topology reporting (2022-08-15 22:07:34 +0100) > > ---------------------------------------------------------------- > Fix RISC-V's topology reporting > > The goal here is the fix the incorrectly reported arch topology on > RISC-V which seems to have been broken since it was added. > cpu, package and thread IDs are all currently reported as -1, so tools > like lstopo think systems have multiple threads on the same core when > this is not true: > https://github.com/open-mpi/hwloc/issues/536 > > arm64's topology code basically applies to RISC-V too, so it has been > made generic along with the removal of MPIDR related code, which > appears to be redudant code since '3102bc0e6ac7 ("arm64: topology: Stop > using MPIDR for topology information")' replaced the code that actually > interacted with MPIDR with default values. > > ---------------------------------------------------------------- > Conor Dooley (2): > arm64: topology: move store_cpu_topology() to shared code > riscv: topology: fix default topology reporting > > arch/arm64/kernel/topology.c | 40 ---------------------------------------- > arch/riscv/Kconfig | 2 +- > arch/riscv/kernel/smpboot.c | 3 ++- > drivers/base/arch_topology.c | 19 +++++++++++++++++++ > 4 files changed, 22 insertions(+), 42 deletions(-)
| |