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

[Srirangan Madhavan]

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.

--
Regards,
Srirangan