Re: [PATCH v21 01/23] cxl/mem: refactor memdev allocation

[Jonathan Cameron <jonathan.cameron@huawei.com>]

On Wed, 19 Nov 2025 19:22:14 +0000
alejandro.lucero-palau@amd.com wrote:

> From: Alejandro Lucero <alucerop@amd.com>
> 
> In preparation for always-synchronous memdev attach, refactor memdev
> allocation and fix release bug in devm_cxl_add_memdev() when error after
> a successful allocation.
> 
> The diff is busy as this moves cxl_memdev_alloc() down below the definition
> of cxl_memdev_fops and introduces devm_cxl_memdev_add_or_reset() to
> preclude needing to export more symbols from the cxl_core.
> 
> Fixes: 1c3333a28d45 ("cxl/mem: Do not rely on device_add() side effects for dev_set_name() failures")
> 

No line break here. Fixes is part of the tag block and some tools
get grumpy if that isn't contiguous.  That includes a bot that runs
on linux-next.

> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
> Signed-off-by: Alejandro Lucero <alucerop@amd.com>

This SOB chain is wrong.  What was Dan's role in this?  As first SOB with no
Co-developed tag he would normally also be the author (From above)

I'm out of time for today so will leave review for another time. Just flagging
that without these tag chains being correct Dave can't pick this up even
if everything else is good.