[PATCH 00/33] qemu: monitor: Clean up qemu monitor event callbacks

Peter Krempa posted 33 patches 2 years, 8 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1626864131.git.pkrempa@redhat.com
src/qemu/qemu_monitor.c      | 215 +++++------------
src/qemu/qemu_monitor.h      | 443 +++++++++++++++++------------------
src/qemu/qemu_monitor_json.c |   7 -
src/qemu/qemu_process.c      | 236 ++++++-------------
src/qemu/qemu_processpriv.h  |   8 +-
5 files changed, 363 insertions(+), 546 deletions(-)
[PATCH 00/33] qemu: monitor: Clean up qemu monitor event callbacks
Posted by Peter Krempa 2 years, 8 months ago
Peter Krempa (33):
  qemu: monitor: Remove handlers for the 'POWERDOWN' event
  qemu: monitor: Remove return value from qemuMonitorEmit* functions
  qemu: Remove return value from qemuMonitorDomainEventCallback
  qemu: Remove return value from qemuMonitorDomainShutdownCallback
  qemu: Remove return value from qemuMonitorDomainResetCallback
  qemu: Remove return value from qemuMonitorDomainStopCallback
  qemu: Remove return value from qemuMonitorDomainResumeCallback
  qemu: Remove return value from qemuMonitorDomainRTCChangeCallback
  qemu: Remove return value from qemuMonitorDomainWatchdogCallback
  qemu: Remove return value from qemuMonitorDomainIOErrorCallback
  qemu: Remove return value from qemuMonitorDomainGraphicsCallback
  qemu: Remove return value from qemuMonitorDomainBlockJobCallback
  qemu: Remove return value from
    qemuMonitorDomainJobStatusChangeCallback
  qemu: Remove return value from qemuMonitorDomainTrayChangeCallback
  qemu: Remove return value from qemuMonitorDomainPMWakeupCallback
  qemu: Remove return value from qemuMonitorDomainPMSuspendCallback
  qemu: Remove return value from qemuMonitorDomainBalloonChangeCallback
  qemu: Remove return value from qemuMonitorDomainPMSuspendDiskCallback
  qemu: Remove return value from qemuMonitorDomainGuestPanicCallback
  qemu: Remove return value from qemuMonitorDomainDeviceDeletedCallback
  qemu: Remove return value from
    qemuMonitorDomainNicRxFilterChangedCallback
  qemu: Remove return value from qemuMonitorDomainSerialChangeCallback
  qemu: Remove return value from qemuMonitorDomainSpiceMigratedCallback
  qemu: Remove return value from
    qemuMonitorDomainMigrationStatusCallback
  qemu: Remove return value from qemuMonitorDomainMigrationPassCallback
  qemu: Remove return value from qemuMonitorDomainAcpiOstInfoCallback
  qemu: Remove return value from qemuMonitorDomainBlockThresholdCallback
  qemu: Remove return value from qemuMonitorDomainDumpCompletedCallback
  qemu: Remove return value from
    qemuMonitorDomainPRManagerStatusChangedCallback
  qemu: Remove return value from
    qemuMonitorDomainRdmaGidStatusChangedCallback
  qemu: Remove return value from
    qemuMonitorDomainGuestCrashloadedCallback
  qemu: Remove return value from qemuMonitorDomainMemoryFailureCallback
  qemu: process: Extract code for submitting event handling to separate
    thread

 src/qemu/qemu_monitor.c      | 215 +++++------------
 src/qemu/qemu_monitor.h      | 443 +++++++++++++++++------------------
 src/qemu/qemu_monitor_json.c |   7 -
 src/qemu/qemu_process.c      | 236 ++++++-------------
 src/qemu/qemu_processpriv.h  |   8 +-
 5 files changed, 363 insertions(+), 546 deletions(-)

-- 
2.31.1

Re: [PATCH 00/33] qemu: monitor: Clean up qemu monitor event callbacks
Posted by Jano Tomko 2 years, 8 months ago
On a %A in %Y, Peter Krempa wrote:
> Peter Krempa (33):
>   qemu: monitor: Remove handlers for the 'POWERDOWN' event
>   qemu: monitor: Remove return value from qemuMonitorEmit* functions
>   qemu: Remove return value from qemuMonitorDomainEventCallback
>   qemu: Remove return value from qemuMonitorDomainShutdownCallback
>   qemu: Remove return value from qemuMonitorDomainResetCallback
>   qemu: Remove return value from qemuMonitorDomainStopCallback
>   qemu: Remove return value from qemuMonitorDomainResumeCallback
>   qemu: Remove return value from qemuMonitorDomainRTCChangeCallback
>   qemu: Remove return value from qemuMonitorDomainWatchdogCallback
>   qemu: Remove return value from qemuMonitorDomainIOErrorCallback
>   qemu: Remove return value from qemuMonitorDomainGraphicsCallback
>   qemu: Remove return value from qemuMonitorDomainBlockJobCallback
>   qemu: Remove return value from
>     qemuMonitorDomainJobStatusChangeCallback
>   qemu: Remove return value from qemuMonitorDomainTrayChangeCallback
>   qemu: Remove return value from qemuMonitorDomainPMWakeupCallback
>   qemu: Remove return value from qemuMonitorDomainPMSuspendCallback
>   qemu: Remove return value from qemuMonitorDomainBalloonChangeCallback
>   qemu: Remove return value from qemuMonitorDomainPMSuspendDiskCallback
>   qemu: Remove return value from qemuMonitorDomainGuestPanicCallback
>   qemu: Remove return value from qemuMonitorDomainDeviceDeletedCallback
>   qemu: Remove return value from
>     qemuMonitorDomainNicRxFilterChangedCallback
>   qemu: Remove return value from qemuMonitorDomainSerialChangeCallback
>   qemu: Remove return value from qemuMonitorDomainSpiceMigratedCallback
>   qemu: Remove return value from
>     qemuMonitorDomainMigrationStatusCallback
>   qemu: Remove return value from qemuMonitorDomainMigrationPassCallback
>   qemu: Remove return value from qemuMonitorDomainAcpiOstInfoCallback
>   qemu: Remove return value from qemuMonitorDomainBlockThresholdCallback
>   qemu: Remove return value from qemuMonitorDomainDumpCompletedCallback
>   qemu: Remove return value from
>     qemuMonitorDomainPRManagerStatusChangedCallback
>   qemu: Remove return value from
>     qemuMonitorDomainRdmaGidStatusChangedCallback
>   qemu: Remove return value from
>     qemuMonitorDomainGuestCrashloadedCallback
>   qemu: Remove return value from qemuMonitorDomainMemoryFailureCallback
>   qemu: process: Extract code for submitting event handling to separate
>     thread
> 
>  src/qemu/qemu_monitor.c      | 215 +++++------------
>  src/qemu/qemu_monitor.h      | 443 +++++++++++++++++------------------
>  src/qemu/qemu_monitor_json.c |   7 -
>  src/qemu/qemu_process.c      | 236 ++++++-------------
>  src/qemu/qemu_processpriv.h  |   8 +-
>  5 files changed, 363 insertions(+), 546 deletions(-)
> 

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano