Re: [libvirt] [PATCH 3/6] qemu: Remove unused argument 'bootable' from qemuBuildDriveStr

2018-08-09 Thread Ján Tomko

On Thu, Aug 09, 2018 at 02:48:56PM +0200, Peter Krempa wrote:

Now that the argument is unused we can remove it transitively from all
the call graphs.

Signed-off-by: Peter Krempa 
---
src/qemu/qemu_command.c | 20 +++-
src/qemu/qemu_command.h |  3 +--
src/qemu/qemu_hotplug.c |  2 +-
3 files changed, 9 insertions(+), 16 deletions(-)



Reviewed-by: Ján Tomko 

Jano


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 3/6] qemu: Remove unused argument 'bootable' from qemuBuildDriveStr

2018-08-09 Thread Peter Krempa
Now that the argument is unused we can remove it transitively from all
the call graphs.

Signed-off-by: Peter Krempa 
---
 src/qemu/qemu_command.c | 20 +++-
 src/qemu/qemu_command.h |  3 +--
 src/qemu/qemu_hotplug.c |  2 +-
 3 files changed, 9 insertions(+), 16 deletions(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 78090898be..daf037328f 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -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;
diff --git a/src/qemu/qemu_command.h b/src/qemu/qemu_command.h
index cf17dc1ede..283bf3120d 100644
--- a/src/qemu/qemu_command.h
+++ b/src/qemu/qemu_command.h
@@ -103,8 +103,7 @@ bool qemuDiskBusNeedsDriveArg(int bus);

 qemuBlockStorageSourceAttachDataPtr
 qemuBuildStorageSourceAttachPrepareDrive(virDomainDiskDefPtr disk,
- virQEMUCapsPtr qemuCaps,
- bool driveBoot);
+ virQEMUCapsPtr qemuCaps);
 int
 qemuBuildStorageSourceAttachPrepareCommon(virStorageSourcePtr src,
   qemuBlockStorageSourceAttachDataPtr 
data,
diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
index 1488f0a7c2..5b52fe9edc 100644
--- a/src/qemu/qemu_hotplug.c
+++ b/src/qemu/qemu_hotplug.c
@@ -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)
-- 
2.16.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list