A Fixup for "QEMU: CXL mailbox rework and features (Part 1)"

[PATCH v1 0/2] A Fixup for "QEMU: CXL mailbox rework and features (Part 1)"

Posted by Hyeonggon Yoo 1 year ago

Hi, this is a fixup for the recent patch series "QEMU: CXL mailbox rework and
features (Part 1)" [1].

This fixes two problems:

   1. Media Status in memory device status register not being correctly
      read as "Disabled" while sanitation is in progress.

   2. QEMU assertion failure when it issues an MSI-X interrupt
      (indicating the completion of the sanitize command).

[1] https://lore.kernel.org/linux-cxl/20231023160806.13206-1-Jonathan.Cameron@huawei.com

Hyeonggon Yoo (2):
  hw/cxl/device: read from register values in mdev_reg_read()
  hw/mem/cxl_type3: allocate more vectors for MSI-X

 hw/cxl/cxl-device-utils.c   | 17 +++++++++++------
 hw/mem/cxl_type3.c          |  2 +-
 include/hw/cxl/cxl_device.h |  4 +++-
 3 files changed, 15 insertions(+), 8 deletions(-)

-- 
2.39.1

Re: [PATCH v1 0/2] A Fixup for "QEMU: CXL mailbox rework and features (Part 1)"

Posted by Michael S. Tsirkin 12 months ago

On Mon, Nov 27, 2023 at 07:58:28PM +0900, Hyeonggon Yoo wrote:
> Hi, this is a fixup for the recent patch series "QEMU: CXL mailbox rework and
> features (Part 1)" [1].


To clarify do you plan v2 of this?

> This fixes two problems:
> 
>    1. Media Status in memory device status register not being correctly
>       read as "Disabled" while sanitation is in progress.
> 
>    2. QEMU assertion failure when it issues an MSI-X interrupt
>       (indicating the completion of the sanitize command).
> 
> [1] https://lore.kernel.org/linux-cxl/20231023160806.13206-1-Jonathan.Cameron@huawei.com
> 
> Hyeonggon Yoo (2):
>   hw/cxl/device: read from register values in mdev_reg_read()
>   hw/mem/cxl_type3: allocate more vectors for MSI-X
> 
>  hw/cxl/cxl-device-utils.c   | 17 +++++++++++------
>  hw/mem/cxl_type3.c          |  2 +-
>  include/hw/cxl/cxl_device.h |  4 +++-
>  3 files changed, 15 insertions(+), 8 deletions(-)
> 
> -- 
> 2.39.1

Re: [PATCH v1 0/2] A Fixup for "QEMU: CXL mailbox rework and features (Part 1)"

Posted by Hyeonggon Yoo 12 months ago

On Tue, Nov 28, 2023 at 4:31 PM Michael S. Tsirkin <mst@redhat.com> wrote:
>
> On Mon, Nov 27, 2023 at 07:58:28PM +0900, Hyeonggon Yoo wrote:
> > Hi, this is a fixup for the recent patch series "QEMU: CXL mailbox rework and
> > features (Part 1)" [1].
>
>
> To clarify do you plan v2 of this?

Yes, I will send v2. Thanks for asking!

> > This fixes two problems:
> >
> >    1. Media Status in memory device status register not being correctly
> >       read as "Disabled" while sanitation is in progress.
> >
> >    2. QEMU assertion failure when it issues an MSI-X interrupt
> >       (indicating the completion of the sanitize command).
> >
> > [1] https://lore.kernel.org/linux-cxl/20231023160806.13206-1-Jonathan.Cameron@huawei.com
> >
> > Hyeonggon Yoo (2):
> >   hw/cxl/device: read from register values in mdev_reg_read()
> >   hw/mem/cxl_type3: allocate more vectors for MSI-X
> >
> >  hw/cxl/cxl-device-utils.c   | 17 +++++++++++------
> >  hw/mem/cxl_type3.c          |  2 +-
> >  include/hw/cxl/cxl_device.h |  4 +++-
> >  3 files changed, 15 insertions(+), 8 deletions(-)
> >
> > --
> > 2.39.1
>