[PATCH v1 0/2] Update CXL documentation

Raghu H posted 2 patches 1 year ago
Failed in applying to current master (apply log)
docs/system/devices/cxl.rst | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
[PATCH v1 0/2] Update CXL documentation
Posted by Raghu H 1 year ago
Thanks Jonathan for quick review/comments on earlier patch, as suggested
splitting into two separate patches
https://www.mail-archive.com/qemu-devel@nongnu.org/msg952999.html

Removed the unsupported size option for cxl-type3 device, Qemu reads
the device size directly from the backend memory device config.

Currently the Qemu CXL emulation for AARCH64 is not available and its
only supported on x86_64 platform emulations. Removing the incorrect
information and populating with supported x86_64 sample command to
emulate cxl devices.

The document will be updated when the AARCH64 support is mainlined.


Raghu H (2):
  docs/cxl: Remove incorrect CXL type 3 size parameter
  docs/cxl: Replace unsupported AARCH64 with x86_64

 docs/system/devices/cxl.rst | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)


base-commit: 7d0334e49111787ae19fbc8d29ff6e7347f0605e
-- 
2.34.1
Re: [PATCH v1 0/2] Update CXL documentation
Posted by Jonathan Cameron via 1 year ago
On Thu,  6 Apr 2023 18:58:37 +0530
Raghu H <raghuhack78@gmail.com> wrote:

> Thanks Jonathan for quick review/comments on earlier patch, as suggested
> splitting into two separate patches
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg952999.html
> 
> Removed the unsupported size option for cxl-type3 device, Qemu reads
> the device size directly from the backend memory device config.
> 
> Currently the Qemu CXL emulation for AARCH64 is not available and its
> only supported on x86_64 platform emulations. Removing the incorrect
> information and populating with supported x86_64 sample command to
> emulate cxl devices.
> 
> The document will be updated when the AARCH64 support is mainlined.

Both look good to me.  No need to rush these in.

I'll queue these up in my local tree (and update gitlab/jic23/qemu
sometime later this week) but if anyone wants to pick
them directly that's fine too.

Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> 
> 
> Raghu H (2):
>   docs/cxl: Remove incorrect CXL type 3 size parameter
>   docs/cxl: Replace unsupported AARCH64 with x86_64
> 
>  docs/system/devices/cxl.rst | 17 ++++++++++-------
>  1 file changed, 10 insertions(+), 7 deletions(-)
> 
> 
> base-commit: 7d0334e49111787ae19fbc8d29ff6e7347f0605e
Re: [PATCH v1 0/2] Update CXL documentation
Posted by RAGHU H 1 year ago
Thanks Jonathan! I assume it will be merged sometime later with your
other patches in the 8.x version!

On Thu, Apr 13, 2023 at 2:58 PM Jonathan Cameron
<Jonathan.Cameron@huawei.com> wrote:
>
> On Thu,  6 Apr 2023 18:58:37 +0530
> Raghu H <raghuhack78@gmail.com> wrote:
>
> > Thanks Jonathan for quick review/comments on earlier patch, as suggested
> > splitting into two separate patches
> > https://www.mail-archive.com/qemu-devel@nongnu.org/msg952999.html
> >
> > Removed the unsupported size option for cxl-type3 device, Qemu reads
> > the device size directly from the backend memory device config.
> >
> > Currently the Qemu CXL emulation for AARCH64 is not available and its
> > only supported on x86_64 platform emulations. Removing the incorrect
> > information and populating with supported x86_64 sample command to
> > emulate cxl devices.
> >
> > The document will be updated when the AARCH64 support is mainlined.
>
> Both look good to me.  No need to rush these in.
>
> I'll queue these up in my local tree (and update gitlab/jic23/qemu
> sometime later this week) but if anyone wants to pick
> them directly that's fine too.
>
> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>
> >
> >
> > Raghu H (2):
> >   docs/cxl: Remove incorrect CXL type 3 size parameter
> >   docs/cxl: Replace unsupported AARCH64 with x86_64
> >
> >  docs/system/devices/cxl.rst | 17 ++++++++++-------
> >  1 file changed, 10 insertions(+), 7 deletions(-)
> >
> >
> > base-commit: 7d0334e49111787ae19fbc8d29ff6e7347f0605e
>