Re: [PATCH v2 00/10] CXL: FMAPI DCD Management Commands 0x5600-0x5605

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

On Fri, 6 Jun 2025 00:28:42 +0000
Anisa Su <anisa.su887@gmail.com> wrote:

> On Mon, Jun 02, 2025 at 05:46:57PM +0000, Anisa Su wrote:
> > On Fri, May 30, 2025 at 03:26:55PM +0100, Jonathan Cameron wrote:  
> > > On Tue, 20 May 2025 14:39:47 +0100
> > > Jonathan Cameron via <qemu-devel@nongnu.org> wrote:
> > >   
> > > > On Thu,  8 May 2025 00:00:56 +0000
> > > > anisa.su887@gmail.com wrote:
> > > >   
> > > > > From: Anisa Su <anisa.su@samsung.com>
> > > > >   
> snip 
> > > Anisa,
> > > 
> > > Ideally I'd like to get the majority of this upstream once you post
> > > a v3.  To do that I'd like to break the dependence on mctp_i2c in patch 2.
> > > These commands are equally valid via a switch-cci (and that path to the
> > > fm owned ld_mctpcci is already upstream).
> > > 
> > > If you could send it as two series. One of which sits on upstream and
> > > doesn't include the i2c_mctp part and the other of which is applied once
> > > that support is available in my tree that would be great. I'll squash
> > > the i2c_mctp part into the patch that adds support for that in the
> > > first place.
> > > 
> > > I'll deal with rebasing needed for other command introductions etc.
> > >   
> Hi Jonathan,
> 
> I applied the v3 patches (which are based on ToT master) on your latest
> published branch cxl-2025-03-20 + Fan's patch fixing dc extent tracking
> and then edited the first patch to add the i2c_mctp stuff and rebased
> the remaining patches on top.
> 
> Would it be easier for you if I send you those?
I'm lost in the description I'm afraid.  Maybe just have a go at it and we'll
see?

Jonathan

> 
> Let me know, thanks!
> Anisa
> 
> > > There are just enough open questions in this series that I haven't done
> > > this rework directly (though I did start doing it - hence the missing
> > > include note in one of the patches).
> > > 
> > > Thanks,
> > >   
> > Sounds good, will do :)
> > 
> > Thanks,
> > Anisa  
> > > Jonathan  
> > > > 
> > > > Jonathan
> > > > 
> > > > 
> > > >   
> snip
> > > > > Anisa Su (10):
> > > > >   cxl-mailbox-utils: Move opcodes enum to new header file
> > > > >   cxl-mailbox-utils: 0x5600 - FMAPI Get DCD Info
> > > > >   cxl/type3: Add dsmas_flags to CXLDCRegion struct
> > > > >   cxl-mailbox-utils: 0x5601 - FMAPI Get Host Region Config
> > > > >   cxl_events.h: Move definition for dynamic_capacity_uuid and enum for
> > > > >     DC event types
> > > > >   hw/cxl_type3: Add DC Region bitmap lock
> > > > >   cxl-mailbox-utils: 0x5602 - FMAPI Set DC Region Config
> > > > >   cxl-mailbox-utils: 0x5603 - FMAPI Get DC Region Extent Lists
> > > > >   cxl-mailbox-utils: 0x5604 - FMAPI Initiate DC Add
> > > > >   cxl-mailbox-utils: 0x5605 - FMAPI Initiate DC Release
> > > > > 
> > > > >  hw/cxl/cxl-mailbox-utils.c   | 649 +++++++++++++++++++++++++++++++----
> > > > >  hw/cxl/i2c_mctp_cxl.c        |   6 +-
> > > > >  hw/mem/cxl_type3.c           |  41 ++-
> > > > >  include/hw/cxl/cxl_device.h  |  24 ++
> > > > >  include/hw/cxl/cxl_events.h  |  15 +
> > > > >  include/hw/cxl/cxl_mailbox.h |   6 +
> > > > >  include/hw/cxl/cxl_opcodes.h |  72 ++++
> > > > >  7 files changed, 724 insertions(+), 89 deletions(-)
> > > > >  create mode 100644 include/hw/cxl/cxl_opcodes.h
> > > > >     
> > > > 
> > > >   
> > >