drivers/firmware: psci_checker: stash and use topology_core_cpumask for hotplug tests
authorSudeep Holla <sudeep.holla@arm.com>
Wed, 25 Jul 2018 10:48:09 +0000 (11:48 +0100)
committerOlof Johansson <olof@lixom.net>
Thu, 26 Jul 2018 07:16:58 +0000 (00:16 -0700)
commit7401056de5f8d4eabe71a4c4aa80d0e278856e07
tree1b84017857a837f0251baa497729ce71320b3988
parent1ca8c0a763418d7f15f5d1d13c3005b6d3a083db
drivers/firmware: psci_checker: stash and use topology_core_cpumask for hotplug tests

Commit 7f9545aa1a91 ("arm64: smp: remove cpu and numa topology
information when hotplugging out CPU") updates the cpu topology when
the CPU is hotplugged out. However the PSCI checker code uses the
topology_core_cpumask pointers for some of the cpu hotplug testing.
Since the pointer to the core_cpumask of the first CPU in the group
is used, which when that CPU itself is hotpugged out is just set to
itself, the testing terminates after that particular CPU is tested out.
But the intention of this tests is to cover all the CPU in the group.

In order to support that, we need to stash the topology_core_cpumask
before the start of the test and use that value instead of pointer to
a cpumask which will be updated on CPU hotplug.

Fixes: 7f9545aa1a91a9a4 ("arm64: smp: remove cpu and numa topology
information when hotplugging out CPU")
Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Tested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Mark Rutland <mark.rutland@arm.com>
Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Signed-off-by: Olof Johansson <olof@lixom.net>
drivers/firmware/psci_checker.c