]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Remove unused argument 'bootable' from qemuBuildDriveStr
authorPeter Krempa <pkrempa@redhat.com>
Thu, 9 Aug 2018 11:33:41 +0000 (13:33 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Thu, 9 Aug 2018 15:00:54 +0000 (17:00 +0200)
Now that the argument is unused we can remove it transitively from all
the call graphs.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
src/qemu/qemu_command.c
src/qemu/qemu_command.h
src/qemu/qemu_hotplug.c

index 78090898be71d66395733d0062fc5ff0cf7f1295..daf037328ff01eae139c4e8ca6128fd6744a79f8 100644 (file)
@@ -1632,7 +1632,6 @@ qemuBuildDiskFrontendAttributes(virDomainDiskDefPtr disk,
 
 static char *
 qemuBuildDriveStr(virDomainDiskDefPtr disk,
-                  bool bootable ATTRIBUTE_UNUSED,
                   virQEMUCapsPtr qemuCaps)
 {
     virBuffer opt = VIR_BUFFER_INITIALIZER;
@@ -2244,14 +2243,12 @@ qemuBuildBlockStorageSourceAttachDataCommandline(virCommandPtr cmd,
 static int
 qemuBuildDiskSourceCommandLine(virCommandPtr cmd,
                                virDomainDiskDefPtr disk,
-                               virQEMUCapsPtr qemuCaps,
-                               bool driveBoot)
+                               virQEMUCapsPtr qemuCaps)
 {
     qemuBlockStorageSourceAttachDataPtr data = NULL;
     int ret = -1;
 
-    if (!(data = qemuBuildStorageSourceAttachPrepareDrive(disk, qemuCaps,
-                                                          driveBoot)))
+    if (!(data = qemuBuildStorageSourceAttachPrepareDrive(disk, qemuCaps)))
         return -1;
 
     if (qemuBuildStorageSourceAttachPrepareCommon(disk->src, data, qemuCaps) < 0 ||
@@ -2271,12 +2268,11 @@ qemuBuildDiskCommandLine(virCommandPtr cmd,
                          const virDomainDef *def,
                          virDomainDiskDefPtr disk,
                          virQEMUCapsPtr qemuCaps,
-                         unsigned int bootindex,
-                         bool driveBoot)
+                         unsigned int bootindex)
 {
     char *optstr;
 
-    if (qemuBuildDiskSourceCommandLine(cmd, disk, qemuCaps, driveBoot) < 0)
+    if (qemuBuildDiskSourceCommandLine(cmd, disk, qemuCaps) < 0)
         return -1;
 
     if (!qemuDiskBusNeedsDriveArg(disk->bus)) {
@@ -2326,7 +2322,6 @@ qemuBuildDisksCommandLine(virCommandPtr cmd,
     for (i = 0; i < def->ndisks; i++) {
         virDomainDiskDefPtr disk = def->disks[i];
         unsigned int bootindex = 0;
-        bool driveBoot = false;
 
         if (disk->info.bootIndex) {
             bootindex = disk->info.bootIndex;
@@ -2349,7 +2344,7 @@ qemuBuildDisksCommandLine(virCommandPtr cmd,
         }
 
         if (qemuBuildDiskCommandLine(cmd, def, disk, qemuCaps,
-                                     bootindex, driveBoot) < 0)
+                                     bootindex) < 0)
             return -1;
     }
 
@@ -10691,15 +10686,14 @@ qemuBuildHotpluggableCPUProps(const virDomainVcpuDef *vcpu)
  */
 qemuBlockStorageSourceAttachDataPtr
 qemuBuildStorageSourceAttachPrepareDrive(virDomainDiskDefPtr disk,
-                                         virQEMUCapsPtr qemuCaps,
-                                         bool driveBoot)
+                                         virQEMUCapsPtr qemuCaps)
 {
     qemuBlockStorageSourceAttachDataPtr data = NULL;
 
     if (VIR_ALLOC(data) < 0)
         return NULL;
 
-    if (!(data->driveCmd = qemuBuildDriveStr(disk, driveBoot, qemuCaps)) ||
+    if (!(data->driveCmd = qemuBuildDriveStr(disk, qemuCaps)) ||
         !(data->driveAlias = qemuAliasDiskDriveFromDisk(disk))) {
         qemuBlockStorageSourceAttachDataFree(data);
         return NULL;
index cf17dc1ede623a8eda01240b94bc8451c03c9242..283bf3120d6a9c4eeb37d41d049dc5d12a726ad9 100644 (file)
@@ -103,8 +103,7 @@ bool qemuDiskBusNeedsDriveArg(int bus);
 
 qemuBlockStorageSourceAttachDataPtr
 qemuBuildStorageSourceAttachPrepareDrive(virDomainDiskDefPtr disk,
-                                         virQEMUCapsPtr qemuCaps,
-                                         bool driveBoot);
+                                         virQEMUCapsPtr qemuCaps);
 int
 qemuBuildStorageSourceAttachPrepareCommon(virStorageSourcePtr src,
                                           qemuBlockStorageSourceAttachDataPtr data,
index 1488f0a7c269bb5b14546658cc48dade0815c38c..5b52fe9edc942f9b6811d010df7bdf8c57a4e497 100644 (file)
@@ -509,7 +509,7 @@ qemuHotplugDiskSourceAttachPrepare(virDomainDiskDefPtr disk,
     if (VIR_ALLOC(data) < 0)
         return NULL;
 
-    if (!(backend = qemuBuildStorageSourceAttachPrepareDrive(disk, qemuCaps, false)))
+    if (!(backend = qemuBuildStorageSourceAttachPrepareDrive(disk, qemuCaps)))
         goto cleanup;
 
     if (qemuBuildStorageSourceAttachPrepareCommon(disk->src, backend, qemuCaps) < 0)