Re: [PATCH] PCI/DOE: Add debug messages for transaction failures

[sashiko-bot@kernel.org]

> DOE feature discovery reports only -EIO when several distinct mailbox
> status and response-validation failures occur. The abort used to recover
> the mailbox can then clear the status before it is captured, leaving the
> existing failed-to-cache-features message insufficient to identify the
> failing phase.
> 
> Add dynamic-debug messages immediately before the silent error returns.
> Log the transaction phase and raw status or response length, but do not
> change mailbox behavior or expose payload contents.
> 
> On a WD_BLACK SN8100 NVMe device (15b7:5050, firmware 830ZRR0A), whose
> mailbox creation fails on every boot, these messages identified the
> failure: the device accepts the Discovery request but never presents a
> response, and the transaction times out with the Busy bit still set:
> 
>   pci 0000:05:00.0: DOE: [500] response timeout, status: 0x00000001
>   pci 0000:05:00.0: DOE: [500] failed to cache features : -5
>   pci 0000:05:00.0: DOE: [500] failed to create mailbox: -5
> 
> Signed-off-by: Saverio Miroddi <saverio.pub2@gmail.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260721193310.1179300-1-saverio.pub2@gmail.com?part=1