linux-kernel - Re: [PATCH v3 1/5] perf/arm_cspmu: Add arm_cspmu_acpi_dev

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <94832766-e408-4485-8397-e48bc23f615c@arm.com>
Date: Tue, 30 Sep 2025 14:05:10 +0100
From: Suzuki K Poulose <suzuki.poulose@....com>
To: Besar Wicaksono <bwicaksono@...dia.com>, "will@...nel.org"
 <will@...nel.org>, "robin.murphy@....com" <robin.murphy@....com>,
 "ilkka@...amperecomputing.com" <ilkka@...amperecomputing.com>
Cc: "linux-arm-kernel@...ts.infradead.org"
 <linux-arm-kernel@...ts.infradead.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
 "mark.rutland@....com" <mark.rutland@....com>,
 Thierry Reding <treding@...dia.com>, Jon Hunter <jonathanh@...dia.com>,
 Vikram Sethi <vsethi@...dia.com>, Rich Wiley <rwiley@...dia.com>,
 Shanker Donthineni <sdonthineni@...dia.com>
Subject: Re: [PATCH v3 1/5] perf/arm_cspmu: Add arm_cspmu_acpi_dev_get

On 30/09/2025 14:00, Besar Wicaksono wrote:
> 
> 
>> -----Original Message-----
>> From: Suzuki K Poulose <suzuki.poulose@....com>
>> Sent: Tuesday, September 30, 2025 3:38 AM
>> To: Besar Wicaksono <bwicaksono@...dia.com>; will@...nel.org;
>> robin.murphy@....com; ilkka@...amperecomputing.com
>> Cc: linux-arm-kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; linux-
>> tegra@...r.kernel.org; mark.rutland@....com; Thierry Reding
>> <treding@...dia.com>; Jon Hunter <jonathanh@...dia.com>; Vikram Sethi
>> <vsethi@...dia.com>; Rich Wiley <rwiley@...dia.com>; Shanker Donthineni
>> <sdonthineni@...dia.com>
>> Subject: Re: [PATCH v3 1/5] perf/arm_cspmu: Add arm_cspmu_acpi_dev_get
>>
>> External email: Use caution opening links or attachments
>>
>>
>> On 30/09/2025 01:26, Besar Wicaksono wrote:
>> > Add interface to get ACPI device associated with the
>> > PMU. This ACPI device may contain additional properties
>> > not covered by the standard properties.
>> >
>> > Signed-off-by: Besar Wicaksono <bwicaksono@...dia.com>
>>
>> Repeat: Where is the user ?
> 
> I thought you were okay if the use case is on future (different) series.
> 
> So, this is a heads up on what I will have in nvidia_cspmu when adding the
> new PMU support. Do you prefer it to be part of this series?

No, I recommended the opposite. Apologies, if that wasn't clear.
Add a helper only when there is a user.

Suzuki


> 
> +static u32 nv_cspmu_get_inst_id(const struct arm_cspmu *cspmu)
> 
> +{
> 
> +     u32 inst_id;
> 
> +     struct fwnode_handle *fwnode;
> 
> +     struct acpi_device *adev;
> 
> +
> 
> +     inst_id = 0;
> 
> +
> 
> +     adev = arm_cspmu_acpi_dev_get(cspmu);
> 
> +     if (!adev)
> 
> +           return 0;
> 
> +
> 
> +     fwnode = acpi_fwnode_handle(adev);
> 
> +     if (fwnode && fwnode_property_read_u32(fwnode, "instance_id", 
> &inst_id))
> 
> +           kstrtou32(acpi_device_uid(adev), 0, &inst_id);
> 
> +
> 
> +     acpi_dev_put(adev);
> 
> +     return inst_id;
> 
> +}
> 
> Thanks,
> Besar
> 
>