[ add Boris and Ard ] Ard, Boris, can you have a look at the touches to early e820/x86 init (insert_resource_late()) and give an ack (or nak). The general problem here is conflicts between e820 memory resources and CXL subsystem memory resources. Smita Koralahalli wrote: > Insert Soft Reserved memory into a dedicated soft_reserve_resource tree > instead of the iomem_resource tree at boot. > > Publishing Soft Reserved ranges into iomem too early causes conflicts with > CXL hotplug and region assembly failure, especially when Soft Reserved > overlaps CXL regions. > > Re-inserting these ranges into iomem will be handled in follow-up patches, > after ensuring CXL window publication ordering is stabilized and when the > dax_hmem is ready to consume them. > > This avoids trimming or deleting resources later and provides a cleaner > handoff between EFI-defined memory and CXL resource management. > > Signed-off-by: Smita Koralahalli <Smita.KoralahalliChannabasappa@xxxxxxx> > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> Smita, if you added changes this should have Co-developed-by. Otherwise plain Signed-off-by is interpreted as only chain of custody. Any other patches that you add my Signed-off-by to should also have Co-developed-by or be From: me. Alternatively if you completely rewritel a patch with your own approach then note the source (with a Link:) and leave off the original SOB. Lastly, in this case it looks unmodified from what I wrote? Then it should be: From: Dan Williams <dan.j.williams@xxxxxxxxx> Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> Signed-off-by: Smita Koralahalli <Smita.KoralahalliChannabasappa@xxxxxxx> ...to show the chain of custody of you forwarding a diff authored completely by someone else.