qemu: Remove 'qemuBuildCommandLineFlags' and associated code

The -netdev formatter code switched to a real virQEMUCaps flag so we can
remove the old flags which used to enable JSON for -netdev for
validation purposes.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/qemu/qemu_command.h b/src/qemu/qemu_command.h
index 6b8f90f737e7665b19796b9cd0d65d6da2654542..00efcfd5fd3af51c5815b92db97bc7424599e4a6 100644 (file)
--- a/src/qemu/qemu_command.h
+++ b/src/qemu/qemu_command.h
@@ -43,10 +43,6 @@
 VIR_ENUM_DECL(qemuVideo);
 VIR_ENUM_DECL(qemuSoundCodec);
 
-typedef enum {
-    QEMU_BUILD_COMMANDLINE_VALIDATE_KEEP_JSON = 1 << 0,
-} qemuBuildCommandLineFlags;
-
 virCommand *qemuBuildCommandLine(virQEMUDriver *driver,
                                    virLogManager *logManager,
                                    virSecurityManager *secManager,

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 760d30a867f75069b958d4b427e5532b7af8473c..e0a3b4d978e0dad1b92c39b396470b7216b543dc 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -6423,7 +6423,7 @@ static char *qemuConnectDomainXMLToNative(virConnectPtr conn,
         goto cleanup;
 
     if (!(cmd = qemuProcessCreatePretendCmdBuild(driver, vm, NULL,
-                                                 qemuCheckFips(vm), true, false)))
+                                                 qemuCheckFips(vm), true)))
         goto cleanup;
 
     ret = virCommandToString(cmd, false);

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 1d0165af6daaf44db6db28110f5c3b445bae15e7..9faaeeadbace217322e710cdea91f5a6393f6d45 100644 (file)
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -7756,14 +7756,8 @@ qemuProcessCreatePretendCmdBuild(virQEMUDriver *driver,
                                  virDomainObj *vm,
                                  const char *migrateURI,
                                  bool enableFips,
-                                 bool standalone,
-                                 bool jsonPropsValidation)
+                                 bool standalone)
 {
-    unsigned int buildflags = 0;
-
-    if (jsonPropsValidation)
-        buildflags = QEMU_BUILD_COMMANDLINE_VALIDATE_KEEP_JSON;
-
     VIR_DEBUG("Building emulator command line");
     return qemuBuildCommandLine(driver,
                                 NULL,
@@ -7776,7 +7770,7 @@ qemuProcessCreatePretendCmdBuild(virQEMUDriver *driver,
                                 enableFips,
                                 NULL,
                                 NULL,
-                                buildflags);
+                                0);
 }
 
 

diff --git a/src/qemu/qemu_process.h b/src/qemu/qemu_process.h
index f9fa140e6df90152b7ea86c0794742b31670ac6a..c0f1c85aabc19563c394ebbc8afa5d3394f2acac 100644 (file)
--- a/src/qemu/qemu_process.h
+++ b/src/qemu/qemu_process.h
@@ -105,8 +105,7 @@ virCommand *qemuProcessCreatePretendCmdBuild(virQEMUDriver *driver,
                                                virDomainObj *vm,
                                                const char *migrateURI,
                                                bool enableFips,
-                                               bool standalone,
-                                               bool jsonPropsValidation);
+                                               bool standalone);
 
 int qemuProcessInit(virQEMUDriver *driver,
                     virDomainObj *vm,

diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
index 660ca4148c4c2079c952ccbe56591f07d7a67595..12ce6d9559cdefadaa6de3b8f40803425d62b7a3 100644 (file)
--- a/tests/qemuxml2argvtest.c
+++ b/tests/qemuxml2argvtest.c
@@ -386,8 +386,7 @@ testCompareXMLToArgvCreateArgs(virQEMUDriver *drv,
                                virDomainObj *vm,
                                const char *migrateURI,
                                struct testQemuInfo *info,
-                               unsigned int flags,
-                               bool jsonPropsValidation)
+                               unsigned int flags)
 {
     qemuDomainObjPrivate *priv = vm->privateData;
     bool enableFips = !!(flags & FLAG_FIPS_HOST);
@@ -484,8 +483,7 @@ testCompareXMLToArgvCreateArgs(virQEMUDriver *drv,
         enableFips = false;
 
     return qemuProcessCreatePretendCmdBuild(drv, vm, migrateURI,
-                                            enableFips, false,
-                                            jsonPropsValidation);
+                                            enableFips, false);
 }
 
 
@@ -538,8 +536,7 @@ testCompareXMLToArgvValidateSchema(virQEMUDriver *drv,
     if (virBitmapParse("0-3", &priv->autoNodeset, 4) < 0)
         return -1;
 
-    if (!(cmd = testCompareXMLToArgvCreateArgs(drv, vm, migrateURI, info, flags,
-                                               true)))
+    if (!(cmd = testCompareXMLToArgvCreateArgs(drv, vm, migrateURI, info, flags)))
         return -1;
 
     if (virCommandGetArgList(cmd, &args, &nargs) < 0)
@@ -743,7 +740,7 @@ testCompareXMLToArgv(const void *data)
     virResetLastError();
 
     if (!(cmd = testCompareXMLToArgvCreateArgs(&driver, vm, migrateURI, info,
-                                               flags, false))) {
+                                               flags))) {
         err = virGetLastError();
         if (!err) {
             VIR_TEST_DEBUG("no error was reported for expected failure");