[PATCH v8 0/7] DEVICE_UNPLUG_GUEST_ERROR QAPI event

Daniel Henrique Barboza posted 7 patches 2 years, 7 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210907004755.424931-1-danielhb413@gmail.com
Maintainers: Eric Blake <eblake@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>, Greg Kurz <groug@kaod.org>, "Daniel P. Berrangé" <berrange@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Ani Sinha <ani@anisinha.ca>, Markus Armbruster <armbru@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, David Gibson <david@gibson.dropbear.id.au>
docs/about/deprecated.rst | 10 ++++++++++
hw/acpi/memory_hotplug.c  | 11 ++++++++++-
hw/ppc/spapr.c            | 12 ++++++++++--
hw/ppc/spapr_drc.c        | 16 ++++++++++------
qapi/machine.json         |  7 ++++++-
qapi/qdev.json            | 31 ++++++++++++++++++++++++++++---
stubs/qdev.c              |  7 +++++++
7 files changed, 81 insertions(+), 13 deletions(-)
[PATCH v8 0/7] DEVICE_UNPLUG_GUEST_ERROR QAPI event
Posted by Daniel Henrique Barboza 2 years, 7 months ago
Hi,

This new version amends the QAPI doc in patch 5, as suggested
by David and Markus, and added all reviewed-by and acked-by
tags.

changes from v7:
- patch 5:
  * s/internal guest/guest reported/
- v7 link: https://lists.gnu.org/archive/html/qemu-devel/2021-08/msg04115.html

Daniel Henrique Barboza (7):
  memory_hotplug.c: handle dev->id = NULL in acpi_memory_hotplug_write()
  spapr.c: handle dev->id in spapr_memory_unplug_rollback()
  spapr_drc.c: do not error_report() when drc->dev->id == NULL
  qapi/qdev.json: fix DEVICE_DELETED parameters doc
  qapi/qdev.json: add DEVICE_UNPLUG_GUEST_ERROR QAPI event
  spapr: use DEVICE_UNPLUG_GUEST_ERROR to report unplug errors
  memory_hotplug.c: send DEVICE_UNPLUG_GUEST_ERROR in
    acpi_memory_hotplug_write()

 docs/about/deprecated.rst | 10 ++++++++++
 hw/acpi/memory_hotplug.c  | 11 ++++++++++-
 hw/ppc/spapr.c            | 12 ++++++++++--
 hw/ppc/spapr_drc.c        | 16 ++++++++++------
 qapi/machine.json         |  7 ++++++-
 qapi/qdev.json            | 31 ++++++++++++++++++++++++++++---
 stubs/qdev.c              |  7 +++++++
 7 files changed, 81 insertions(+), 13 deletions(-)

-- 
2.31.1


Re: [PATCH v8 0/7] DEVICE_UNPLUG_GUEST_ERROR QAPI event
Posted by David Gibson 2 years, 7 months ago
On Mon, Sep 06, 2021 at 09:47:48PM -0300, Daniel Henrique Barboza wrote:
> Hi,
> 
> This new version amends the QAPI doc in patch 5, as suggested
> by David and Markus, and added all reviewed-by and acked-by
> tags.

I've staged this in the ppc-for-6.2 tree.  Obviously it has some stuff
that isn't purely ppc related, but I'm figuring that should be ok with
the acks from Igor and Markus.  Let me know if there are any
objections.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
Re: [PATCH v8 0/7] DEVICE_UNPLUG_GUEST_ERROR QAPI event
Posted by Igor Mammedov 2 years, 7 months ago
On Tue, 7 Sep 2021 11:17:21 +1000
David Gibson <david@gibson.dropbear.id.au> wrote:

> On Mon, Sep 06, 2021 at 09:47:48PM -0300, Daniel Henrique Barboza wrote:
> > Hi,
> > 
> > This new version amends the QAPI doc in patch 5, as suggested
> > by David and Markus, and added all reviewed-by and acked-by
> > tags.  
>
> I've staged this in the ppc-for-6.2 tree.  Obviously it has some stuff
> that isn't purely ppc related, but I'm figuring that should be ok with
> the acks from Igor and Markus.  Let me know if there are any
> objections.
Looks good to me, so:

Acked-by: Igor Mammedov <imammedo@redhat.com>