Commit 597704e2 authored by Robin Murphy's avatar Robin Murphy Committed by Will Deacon
Browse files

perf/arm-cmn: Initialise cmn->cpu earlier 



For all the complexity of handling affinity for CPU hotplug, what we've
apparently managed to overlook is that arm_cmn_init_irqs() has in fact
always been setting the *initial* affinity of all IRQs to CPU 0, not the
CPU we subsequently choose for event scheduling. Oh dear.

Cc: stable@vger.kernel.org
Fixes: 0ba64770 ("perf: Add Arm CMN-600 PMU driver")
Signed-off-by: default avatarRobin Murphy <robin.murphy@arm.com>
Reviewed-by: default avatarIlkka Koskinen <ilkka@os.amperecomputing.com>
Link: https://lore.kernel.org/r/b12fccba6b5b4d2674944f59e4daad91cd63420b.1747069914.git.robin.murphy@arm.com


Signed-off-by: default avatarWill Deacon <will@kernel.org>
parent 097469a2

drivers/perf/arm-cmn.c

+1 −1
Original line number Diff line number Diff line
@@ -2551,6 +2551,7 @@ static int arm_cmn_probe(struct platform_device *pdev) 


	cmn->dev = &pdev->dev;

	cmn->part = (unsigned long)device_get_match_data(cmn->dev);

	cmn->cpu = cpumask_local_spread(0, dev_to_node(cmn->dev));

	platform_set_drvdata(pdev, cmn);



	if (cmn->part == PART_CMN600 && has_acpi_companion(cmn->dev)) {
@@ -2578,7 +2579,6 @@ static int arm_cmn_probe(struct platform_device *pdev) 
	if (err)

		return err;



	cmn->cpu = cpumask_local_spread(0, dev_to_node(cmn->dev));

	cmn->pmu = (struct pmu) {

		.module = THIS_MODULE,

		.parent = cmn->dev,