Re: [RFC PATCH 1/4] cxl/mem: Add support to cleanly continue after attach
From: Dan Williams
Date: Sat Apr 11 2026 - 18:34:06 EST
Cheatham, Benjamin wrote:
> On 4/3/2026 4:00 PM, Dan Williams wrote:
> > For drivers that want to fallback to PCI-only operation, immediately
> > cleanup on attach failure. Otherwise vestigial topology objects are left
> > until driver unload.
> >
>
> The usage of "attach" here is somewhat ambiguous. Does this mean the attach callback,
> or cxl_mem attach?
Fair point.
> Also, I know that providing the attach callback is probably what
> indicates a driver wants to fallback to PCIe operation, but it's not
> evident based on the description or contents of the patch.
I rushed this... it was a hectic week when I sent this.
> 2) Isn't the memdev already cleaned up if cxl_mem probe fails and
> attach is provided? What's the extra cleanup happening here?
Exactly right. cxl_mem_probe() failure does the job.
Sorry for the noise.