Re: [PATCH v10 00/12] PCI/CXL: Add CXL reset support for Type 2 devices

[Dave Jiang]

On 9/1/26 10:11 PM, Srirangan Madhavan wrote:
> On 8/25/26 2:13 PM, Dave Jiang wrote:
>>> Userspace can select "cxl_reset" in reset_method and then trigger reset
>>> through the existing reset attribute.
>> Srirangan,
>> Do we need to consider concurrent mailbox commands while reset is ongoing? Although rather than blocking them, maybe we just need to harden the mailbox path in CXL core so that when ~0s are read we return error and give up? and maybe a reset_prepare() to quiesce things. Just some random thoughts.
>>
>> DJ
> 
> I agree that the PCI device lock does not serialize against the mailbox mutex, so a mailbox command could still be in progress during reset.
> 
> Since cxl_reset also needs to work without cxl_pci bound, reset_prepare() and reset_done() seem like a good place to add this.
> 
> I would prefer to do this as a follow-up along with hardening for ~0s read, if that is okay. I can send out a follow up RFC to discuss the specifics.
> 

That is fine with me.