[PATCH v3 09/26] perf: arm_spe_pmu: Convert to new IRQ affinity retrieval API

Marc Zyngier posted 26 patches 4 months, 2 weeks ago
There is a newer version of this series
[PATCH v3 09/26] perf: arm_spe_pmu: Convert to new IRQ affinity retrieval API
Posted by Marc Zyngier 4 months, 2 weeks ago
Now that the relevant interrupt controllers are equipped with
a callback returning the affinity of per-CPU interrupts, switch
the ARM SPE driver over to this new method.

Signed-off-by: Marc Zyngier <maz@kernel.org>
---
 drivers/perf/arm_spe_pmu.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/perf/arm_spe_pmu.c b/drivers/perf/arm_spe_pmu.c
index 369e77ad5f13f..a1c86a1f21427 100644
--- a/drivers/perf/arm_spe_pmu.c
+++ b/drivers/perf/arm_spe_pmu.c
@@ -1211,8 +1211,10 @@ static void arm_spe_pmu_dev_teardown(struct arm_spe_pmu *spe_pmu)
 static int arm_spe_pmu_irq_probe(struct arm_spe_pmu *spe_pmu)
 {
 	struct platform_device *pdev = spe_pmu->pdev;
-	int irq = platform_get_irq(pdev, 0);
+	const struct cpumask *affinity;
+	int irq;
 
+	irq = platform_get_irq_affinity(pdev, 0, &affinity);
 	if (irq < 0)
 		return -ENXIO;
 
@@ -1221,10 +1223,7 @@ static int arm_spe_pmu_irq_probe(struct arm_spe_pmu *spe_pmu)
 		return -EINVAL;
 	}
 
-	if (irq_get_percpu_devid_partition(irq, &spe_pmu->supported_cpus)) {
-		dev_err(&pdev->dev, "failed to get PPI partition (%d)\n", irq);
-		return -EINVAL;
-	}
+	cpumask_copy(&spe_pmu->supported_cpus, affinity);
 
 	spe_pmu->irq = irq;
 	return 0;
-- 
2.47.3
Re: [PATCH v3 09/26] perf: arm_spe_pmu: Convert to new IRQ affinity retrieval API
Posted by Jinjie Ruan 4 months ago

On 2025/9/22 16:28, Marc Zyngier wrote:
> Now that the relevant interrupt controllers are equipped with
> a callback returning the affinity of per-CPU interrupts, switch
> the ARM SPE driver over to this new method.
> 
> Signed-off-by: Marc Zyngier <maz@kernel.org>
> ---
>  drivers/perf/arm_spe_pmu.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/perf/arm_spe_pmu.c b/drivers/perf/arm_spe_pmu.c
> index 369e77ad5f13f..a1c86a1f21427 100644
> --- a/drivers/perf/arm_spe_pmu.c
> +++ b/drivers/perf/arm_spe_pmu.c
> @@ -1211,8 +1211,10 @@ static void arm_spe_pmu_dev_teardown(struct arm_spe_pmu *spe_pmu)
>  static int arm_spe_pmu_irq_probe(struct arm_spe_pmu *spe_pmu)
>  {
>  	struct platform_device *pdev = spe_pmu->pdev;
> -	int irq = platform_get_irq(pdev, 0);
> +	const struct cpumask *affinity;
> +	int irq;
>  
> +	irq = platform_get_irq_affinity(pdev, 0, &affinity);
>  	if (irq < 0)
>  		return -ENXIO;
>  
> @@ -1221,10 +1223,7 @@ static int arm_spe_pmu_irq_probe(struct arm_spe_pmu *spe_pmu)
>  		return -EINVAL;
>  	}
>  
> -	if (irq_get_percpu_devid_partition(irq, &spe_pmu->supported_cpus)) {
> -		dev_err(&pdev->dev, "failed to get PPI partition (%d)\n", irq);
> -		return -EINVAL;
> -	}
> +	cpumask_copy(&spe_pmu->supported_cpus, affinity);

Reviewed-by: Jinjie Ruan <ruanjinjie@huawei.com>

>  
>  	spe_pmu->irq = irq;
>  	return 0;