Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-04-05 Thread Jonathan Cameron via
... > > > > > > > > > > Can we switch this to mem_size and drop the persistent comment? It is > > > > > my > > > > > understanding that HDM is independent of persistence. > > > > > > > > Discussed in the other branch of this thread. Short answer is we don't > > > > support non persistent

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-04-04 Thread Adam Manzanares
On Fri, Apr 01, 2022 at 02:30:34PM +0100, Jonathan Cameron wrote: > On Thu, 31 Mar 2022 22:13:20 + > Adam Manzanares wrote: > > > On Wed, Mar 30, 2022 at 06:48:48PM +0100, Jonathan Cameron wrote: > > > On Tue, 29 Mar 2022 18:13:59 + > > > Adam Manzanares wrote: > > > > > > > On Fri, M

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-04-01 Thread Jonathan Cameron via
On Thu, 31 Mar 2022 22:13:20 + Adam Manzanares wrote: > On Wed, Mar 30, 2022 at 06:48:48PM +0100, Jonathan Cameron wrote: > > On Tue, 29 Mar 2022 18:13:59 + > > Adam Manzanares wrote: > > > > > On Fri, Mar 18, 2022 at 03:05:53PM +, Jonathan Cameron wrote: > > > > From: Ben Widaw

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-31 Thread Adam Manzanares
On Wed, Mar 30, 2022 at 06:48:48PM +0100, Jonathan Cameron wrote: > On Tue, 29 Mar 2022 18:13:59 + > Adam Manzanares wrote: > > > On Fri, Mar 18, 2022 at 03:05:53PM +, Jonathan Cameron wrote: > > > From: Ben Widawsky > > > > > > A CXL device is a type of CXL component. Conceptually, a C

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-31 Thread Adam Manzanares
On Wed, Mar 30, 2022 at 01:15:58PM +0100, Jonathan Cameron wrote: > On Tue, 29 Mar 2022 12:53:51 -0700 > Davidlohr Bueso wrote: > > > On Tue, 29 Mar 2022, Adam Manzanares wrote: > > >> +typedef struct cxl_device_state { > > >> +MemoryRegion device_registers; > > >> + > > >> +/* mmio for d

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-30 Thread Jonathan Cameron via
On Tue, 29 Mar 2022 18:13:59 + Adam Manzanares wrote: > On Fri, Mar 18, 2022 at 03:05:53PM +, Jonathan Cameron wrote: > > From: Ben Widawsky > > > > A CXL device is a type of CXL component. Conceptually, a CXL device > > would be a leaf node in a CXL topology. From an emulation perspect

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-30 Thread Jonathan Cameron via
On Tue, 29 Mar 2022 12:53:51 -0700 Davidlohr Bueso wrote: > On Tue, 29 Mar 2022, Adam Manzanares wrote: > >> +typedef struct cxl_device_state { > >> +MemoryRegion device_registers; > >> + > >> +/* mmio for device capabilities array - 8.2.8.2 */ > >> +MemoryRegion device; > >> +Mem

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-29 Thread Davidlohr Bueso
On Tue, 29 Mar 2022, Adam Manzanares wrote: +typedef struct cxl_device_state { +MemoryRegion device_registers; + +/* mmio for device capabilities array - 8.2.8.2 */ +MemoryRegion device; +MemoryRegion caps; + +/* mmio for the mailbox registers 8.2.8.4 */ +MemoryRegion mail

Re: [PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-29 Thread Adam Manzanares
On Fri, Mar 18, 2022 at 03:05:53PM +, Jonathan Cameron wrote: > From: Ben Widawsky > > A CXL device is a type of CXL component. Conceptually, a CXL device > would be a leaf node in a CXL topology. From an emulation perspective, > CXL devices are the most complex and so the actual implementati

[PATCH v8 04/46] hw/cxl/device: Introduce a CXL device (8.2.8)

2022-03-18 Thread Jonathan Cameron via
From: Ben Widawsky A CXL device is a type of CXL component. Conceptually, a CXL device would be a leaf node in a CXL topology. From an emulation perspective, CXL devices are the most complex and so the actual implementation is reserved for discrete commits. This new device type is specifically c