linux-kernel - Re: [PATCH] accel/amdxdna: Treat power-off failure as unrecoverable error

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-prev] [day] [month] [year] [list]

Message-ID: <2dcb4668-2c4d-4c73-9805-1af1338c9e7d@kernel.org>
Date: Thu, 6 Nov 2025 12:31:37 -0600
From: Mario Limonciello <superm1@...nel.org>
To: Lizhi Hou <lizhi.hou@....com>, ogabbay@...nel.org,
 quic_jhugo@...cinc.com, maciej.falkowski@...ux.intel.com,
 dri-devel@...ts.freedesktop.org
Cc: linux-kernel@...r.kernel.org, max.zhen@....com, sonal.santan@....com
Subject: Re: [PATCH] accel/amdxdna: Treat power-off failure as unrecoverable
 error

On 11/6/25 12:19 PM, Lizhi Hou wrote:
> 
> On 11/6/25 10:12, Mario Limonciello wrote:
>> On 11/6/25 12:05 PM, Lizhi Hou wrote:
>>> Failing to set power off indicates an unrecoverable hardware or firmware
>>> error. Update the driver to treat such a failure as a fatal condition
>>> and stop further operations that depend on successful power state
>>> transition.
>>>
>>> This prevents undefined behavior when the hardware remains in an
>>> unexpected state after a failed power-off attempt.
>>>
>>> Signed-off-by: Lizhi Hou <lizhi.hou@....com>
>>
>> Presumably all versions of hardware in the wild can handle receiving a 
>> power off command if they're already powered off?
> 
> Yes for the aie2 platforms. This was verified by xdna-driver pipeline 
> tests.
> 
> 

OK LGTM then.

Reviewed-by: Mario Limonciello (AMD) <superm1@...nel.org>

> Lizhi
> 
>>
>>> ---
>>>   drivers/accel/amdxdna/aie2_smu.c | 10 ++++++++++
>>>   1 file changed, 10 insertions(+)
>>>
>>> diff --git a/drivers/accel/amdxdna/aie2_smu.c b/drivers/accel/ 
>>> amdxdna/aie2_smu.c
>>> index 11c0e9e7b03a..bd94ee96c2bc 100644
>>> --- a/drivers/accel/amdxdna/aie2_smu.c
>>> +++ b/drivers/accel/amdxdna/aie2_smu.c
>>> @@ -147,6 +147,16 @@ int aie2_smu_init(struct amdxdna_dev_hdl *ndev)
>>>   {
>>>       int ret;
>>>   +    /*
>>> +     * Failing to set power off indicates an unrecoverable hardware or
>>> +     * firmware error.
>>> +     */
>>> +    ret = aie2_smu_exec(ndev, AIE2_SMU_POWER_OFF, 0, NULL);
>>> +    if (ret) {
>>> +        XDNA_ERR(ndev->xdna, "Access power failed, ret %d", ret);
>>> +        return ret;
>>> +    }
>>> +
>>>       ret = aie2_smu_exec(ndev, AIE2_SMU_POWER_ON, 0, NULL);
>>>       if (ret) {
>>>           XDNA_ERR(ndev->xdna, "Power on failed, ret %d", ret);
>>