Re: [PATCH 79/80] qemu: Assume and retire QEMU_CAPS_BLOCKDEV_BACKUP

2022-08-05 Thread Pavel Hrdina
On Tue, Jul 26, 2022 at 04:37:57PM +0200, Peter Krempa wrote:
> The blockdev-backup QMP command was introduced in qemu-2.3.
> 
> Signed-off-by: Peter Krempa 
> ---
>  src/qemu/qemu_backup.c| 6 --
>  src/qemu/qemu_capabilities.c  | 6 ++
>  src/qemu/qemu_capabilities.h  | 2 +-
>  tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_5.0.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_5.0.0.ppc64.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_5.0.0.riscv64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_5.0.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_5.1.0.sparc.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_5.1.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_5.2.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_5.2.0.ppc64.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_5.2.0.riscv64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_5.2.0.s390x.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_5.2.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_6.0.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_6.0.0.s390x.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_6.0.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_6.1.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_6.2.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_6.2.0.ppc64.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_6.2.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_7.0.0.aarch64.xml | 1 -
>  tests/qemucapabilitiesdata/caps_7.0.0.ppc64.xml   | 1 -
>  tests/qemucapabilitiesdata/caps_7.0.0.x86_64.xml  | 1 -
>  tests/qemucapabilitiesdata/caps_7.1.0.x86_64.xml  | 1 -
>  29 files changed, 3 insertions(+), 37 deletions(-)

Reviewed-by: Pavel Hrdina 


signature.asc
Description: PGP signature


[PATCH 79/80] qemu: Assume and retire QEMU_CAPS_BLOCKDEV_BACKUP

2022-07-26 Thread Peter Krempa
The blockdev-backup QMP command was introduced in qemu-2.3.

Signed-off-by: Peter Krempa 
---
 src/qemu/qemu_backup.c| 6 --
 src/qemu/qemu_capabilities.c  | 6 ++
 src/qemu/qemu_capabilities.h  | 2 +-
 tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_4.2.0.ppc64.xml   | 1 -
 tests/qemucapabilitiesdata/caps_4.2.0.s390x.xml   | 1 -
 tests/qemucapabilitiesdata/caps_4.2.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_5.0.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_5.0.0.ppc64.xml   | 1 -
 tests/qemucapabilitiesdata/caps_5.0.0.riscv64.xml | 1 -
 tests/qemucapabilitiesdata/caps_5.0.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_5.1.0.sparc.xml   | 1 -
 tests/qemucapabilitiesdata/caps_5.1.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_5.2.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_5.2.0.ppc64.xml   | 1 -
 tests/qemucapabilitiesdata/caps_5.2.0.riscv64.xml | 1 -
 tests/qemucapabilitiesdata/caps_5.2.0.s390x.xml   | 1 -
 tests/qemucapabilitiesdata/caps_5.2.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_6.0.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_6.0.0.s390x.xml   | 1 -
 tests/qemucapabilitiesdata/caps_6.0.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_6.1.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_6.2.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_6.2.0.ppc64.xml   | 1 -
 tests/qemucapabilitiesdata/caps_6.2.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_7.0.0.aarch64.xml | 1 -
 tests/qemucapabilitiesdata/caps_7.0.0.ppc64.xml   | 1 -
 tests/qemucapabilitiesdata/caps_7.0.0.x86_64.xml  | 1 -
 tests/qemucapabilitiesdata/caps_7.1.0.x86_64.xml  | 1 -
 29 files changed, 3 insertions(+), 37 deletions(-)

diff --git a/src/qemu/qemu_backup.c b/src/qemu/qemu_backup.c
index 60cb79d70d..1012e3aee7 100644
--- a/src/qemu/qemu_backup.c
+++ b/src/qemu/qemu_backup.c
@@ -802,12 +802,6 @@ qemuBackupBegin(virDomainObj *vm,
 goto endjob;
 }

-if (!virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_BLOCKDEV_BACKUP)) {
-virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
-   _("backup is not supported with this QEMU"));
-goto endjob;
-}
-
 if (virDomainBackupAlignDisks(def, vm->def, suffix) < 0)
 goto endjob;

diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index 1caae40d0f..9ab57dd9f3 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -621,7 +621,7 @@ VIR_ENUM_IMPL(virQEMUCaps,
   /* 395 */
   "vnc-power-control", /* QEMU_CAPS_VNC_POWER_CONTROL */
   "audiodev", /* X_QEMU_CAPS_AUDIODEV */
-  "blockdev-backup", /* QEMU_CAPS_BLOCKDEV_BACKUP */
+  "blockdev-backup", /* X_QEMU_CAPS_BLOCKDEV_BACKUP */
   "object.qapified", /* QEMU_CAPS_OBJECT_JSON */
   "rotation-rate", /* QEMU_CAPS_ROTATION_RATE */

@@ -1586,7 +1586,6 @@ static struct virQEMUCapsStringFlags 
virQEMUCapsQMPSchemaQueries[] = {
 { "blockdev-add/arg-type/+file/aio/^io_uring", QEMU_CAPS_AIO_IO_URING },
 { "blockdev-add/arg-type/+rbd/encrypt", QEMU_CAPS_RBD_ENCRYPTION },
 { "blockdev-add/arg-type/+nbd/tls-hostname", 
QEMU_CAPS_BLOCKDEV_NBD_TLS_HOSTNAME },
-{ "blockdev-backup", QEMU_CAPS_BLOCKDEV_BACKUP },
 { "blockdev-snapshot/$allow-write-only-overlay", 
QEMU_CAPS_BLOCKDEV_SNAPSHOT_ALLOW_WRITE_ONLY },
 { "chardev-add/arg-type/backend/+socket/data/reconnect", 
QEMU_CAPS_CHARDEV_RECONNECT },
 { "chardev-add/arg-type/backend/+file/data/logfile", 
QEMU_CAPS_CHARDEV_LOGFILE },
@@ -5241,8 +5240,7 @@ virQEMUCapsInitQMPVersionCaps(virQEMUCaps *qemuCaps)
 void
 virQEMUCapsInitProcessCapsInterlock(virQEMUCaps *qemuCaps)
 {
-if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_BLOCKDEV_BACKUP) &&
-virQEMUCapsGet(qemuCaps, QEMU_CAPS_BLOCKDEV_REOPEN) &&
+if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_BLOCKDEV_REOPEN) &&
 virQEMUCapsGet(qemuCaps, 
QEMU_CAPS_MIGRATION_PARAM_BLOCK_BITMAP_MAPPING))
 virQEMUCapsSet(qemuCaps, QEMU_CAPS_INCREMENTAL_BACKUP);

diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
index 097ecdca32..78147ee75f 100644
--- a/src/qemu/qemu_capabilities.h
+++ b/src/qemu/qemu_capabilities.h
@@ -600,7 +600,7 @@ typedef enum { /* virQEMUCapsFlags grouping marker for 
syntax-check */
 /* 395 */
 QEMU_CAPS_VNC_POWER_CONTROL, /* -vnc power-control option */
 X_QEMU_CAPS_AUDIODEV, /* -audiodev instead of QEMU_AUDIO_DRV */
-QEMU_CAPS_BLOCKDEV_BACKUP, /* qemu supports the blockdev-backup job */
+X_QEMU_CAPS_BLOCKDEV_BACKUP, /* qemu supports the blockdev-backup job */
 QEMU_CAPS_OBJECT_JSON, /* parameters for object-add are formally described 
*/
 QEMU_CAPS_ROTATION_RATE, /* scsi-disk / ide-drive rotation-rate prop */

diff --git a/tests/qemucapabilitiesdata/caps_4.2.0.aarch64.xml