[libvirt] [PATCH 2/9] qemu: alias: Rename qemuAliasFromDisk to qemuAliasDiskDriveFromDisk

Peter Krempa pkrempa at redhat.com
Fri Jun 1 15:51:05 UTC 2018


Emphasize that it's for the 'drive' part of the disk.

Signed-off-by: Peter Krempa <pkrempa at redhat.com>
---
 src/qemu/qemu_alias.c     |  4 ++--
 src/qemu/qemu_alias.h     |  2 +-
 src/qemu/qemu_block.c     |  2 +-
 src/qemu/qemu_command.c   |  4 ++--
 src/qemu/qemu_driver.c    | 18 +++++++++---------
 src/qemu/qemu_hotplug.c   |  6 +++---
 src/qemu/qemu_migration.c |  6 +++---
 src/qemu/qemu_process.c   |  2 +-
 8 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/src/qemu/qemu_alias.c b/src/qemu/qemu_alias.c
index 89dec91ed1..989cbb4ba9 100644
--- a/src/qemu/qemu_alias.c
+++ b/src/qemu/qemu_alias.c
@@ -650,7 +650,7 @@ qemuAssignDeviceAliases(virDomainDefPtr def, virQEMUCapsPtr qemuCaps)
 }


-/* qemuAliasFromDisk
+/* qemuAliasDiskDriveFromDisk
  * @disk: Pointer to a disk definition
  *
  * Generate and return an alias for the device disk '-drive'
@@ -658,7 +658,7 @@ qemuAssignDeviceAliases(virDomainDefPtr def, virQEMUCapsPtr qemuCaps)
  * Returns NULL with error or a string containing the alias
  */
 char *
-qemuAliasFromDisk(const virDomainDiskDef *disk)
+qemuAliasDiskDriveFromDisk(const virDomainDiskDef *disk)
 {
     char *ret;

diff --git a/src/qemu/qemu_alias.h b/src/qemu/qemu_alias.h
index 51f64624d7..0e2370b737 100644
--- a/src/qemu/qemu_alias.h
+++ b/src/qemu/qemu_alias.h
@@ -75,7 +75,7 @@ int qemuAssignDeviceAliases(virDomainDefPtr def, virQEMUCapsPtr qemuCaps);
 int qemuDomainDeviceAliasIndex(const virDomainDeviceInfo *info,
                                const char *prefix);

-char *qemuAliasFromDisk(const virDomainDiskDef *disk);
+char *qemuAliasDiskDriveFromDisk(const virDomainDiskDef *disk);

 const char *qemuAliasDiskDriveSkipPrefix(const char *dev_name);

diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c
index d1c2d756c2..85176925c9 100644
--- a/src/qemu/qemu_block.c
+++ b/src/qemu/qemu_block.c
@@ -297,7 +297,7 @@ qemuBlockDiskDetectNodes(virDomainDiskDefPtr disk,
     if (src->nodeformat || src->nodestorage)
         return 0;

-    if (!(alias = qemuAliasFromDisk(disk)))
+    if (!(alias = qemuAliasDiskDriveFromDisk(disk)))
         goto cleanup;

     if (!(entry = virHashLookup(disktable, alias))) {
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 1e30c79d6f..484ac261b0 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -1652,7 +1652,7 @@ qemuBuildDriveStr(virDomainDiskDefPtr disk,
         goto error;

     if (qemuDiskBusNeedsDeviceArg(disk->bus)) {
-        char *drivealias = qemuAliasFromDisk(disk);
+        char *drivealias = qemuAliasDiskDriveFromDisk(disk);
         if (!drivealias)
             goto error;

@@ -2087,7 +2087,7 @@ qemuBuildDriveDevStr(const virDomainDef *def,
         virQEMUCapsGet(qemuCaps, QEMU_CAPS_DISK_SHARE_RW))
         virBufferAddLit(&opt, ",share-rw=on");

-    if (!(drivealias = qemuAliasFromDisk(disk)))
+    if (!(drivealias = qemuAliasDiskDriveFromDisk(disk)))
         goto error;
     virBufferAsprintf(&opt, ",drive=%s,id=%s", drivealias, disk->info.alias);
     VIR_FREE(drivealias);
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 05a09eb706..b7c3348550 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -10960,7 +10960,7 @@ qemuDomainBlockResize(virDomainPtr dom,
         disk->src->format == VIR_STORAGE_FILE_QED)
         size = VIR_ROUND_UP(size, 512);

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     qemuDomainObjEnterMonitor(driver, vm);
@@ -14928,7 +14928,7 @@ qemuDomainSnapshotCreateSingleDiskActive(virQEMUDriverPtr driver,
     const char *formatStr = NULL;
     int ret = -1, rc;

-    if (!(device = qemuAliasFromDisk(dd->disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(dd->disk)))
         goto cleanup;

     if (qemuGetDriveSourceString(dd->src, NULL, &source) < 0)
@@ -16976,7 +16976,7 @@ qemuDomainBlockPullCommon(virQEMUDriverPtr driver,
     if (!(disk = qemuDomainDiskByName(vm->def, path)))
         goto endjob;

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     if (qemuDomainDiskBlockJobIsActive(disk))
@@ -17089,7 +17089,7 @@ qemuDomainBlockJobAbort(virDomainPtr dom,
     if (!(disk = qemuDomainDiskByName(vm->def, path)))
         goto endjob;

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     if (disk->mirrorState != VIR_DOMAIN_DISK_MIRROR_STATE_NONE &&
@@ -17320,7 +17320,7 @@ qemuDomainBlockJobSetSpeed(virDomainPtr dom,
     if (!(disk = qemuDomainDiskByName(vm->def, path)))
         goto endjob;

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     qemuDomainObjEnterMonitor(driver, vm);
@@ -17452,7 +17452,7 @@ qemuDomainBlockCopyCommon(virDomainObjPtr vm,
     if (!(disk = qemuDomainDiskByName(vm->def, path)))
         goto endjob;

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     if (qemuDomainDiskBlockJobIsActive(disk))
@@ -17832,7 +17832,7 @@ qemuDomainBlockCommit(virDomainPtr dom,
     if (!(disk = qemuDomainDiskByName(vm->def, path)))
         goto endjob;

-    if (!(device = qemuAliasFromDisk(disk)))
+    if (!(device = qemuAliasDiskDriveFromDisk(disk)))
         goto endjob;

     if (!disk->src->path) {
@@ -18463,7 +18463,7 @@ qemuDomainSetBlockIoTune(virDomainPtr dom,
         if (!(disk = qemuDomainDiskByName(def, path)))
             goto endjob;

-        if (!(device = qemuAliasFromDisk(disk)))
+        if (!(device = qemuAliasDiskDriveFromDisk(disk)))
             goto endjob;

         if (qemuDomainSetBlockIoTuneDefaults(&info, &disk->blkdeviotune,
@@ -18636,7 +18636,7 @@ qemuDomainGetBlockIoTune(virDomainPtr dom,
         if (!(disk = qemuDomainDiskByName(def, path)))
             goto endjob;

-        if (!(device = qemuAliasFromDisk(disk)))
+        if (!(device = qemuAliasDiskDriveFromDisk(disk)))
             goto endjob;
         qemuDomainObjEnterMonitor(driver, vm);
         ret = qemuMonitorGetBlockIoThrottle(priv->mon, device, &reply);
diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
index d8ca894519..62fb652093 100644
--- a/src/qemu/qemu_hotplug.c
+++ b/src/qemu/qemu_hotplug.c
@@ -270,7 +270,7 @@ qemuDomainChangeEjectableMedia(virQEMUDriverPtr driver,
     if (qemuHotplugPrepareDiskAccess(driver, vm, disk, newsrc, false) < 0)
         goto cleanup;

-    if (!(driveAlias = qemuAliasFromDisk(disk)))
+    if (!(driveAlias = qemuAliasDiskDriveFromDisk(disk)))
         goto error;

     qemuDomainObjEnterMonitor(driver, vm);
@@ -446,7 +446,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriverPtr driver,
     if (!(drivestr = qemuBuildDriveStr(disk, false, priv->qemuCaps)))
         goto error;

-    if (!(drivealias = qemuAliasFromDisk(disk)))
+    if (!(drivealias = qemuAliasDiskDriveFromDisk(disk)))
         goto error;

     if (!(devstr = qemuBuildDriveDevStr(vm->def, disk, 0, priv->qemuCaps)))
@@ -3776,7 +3776,7 @@ qemuDomainRemoveDiskDevice(virQEMUDriverPtr driver,

     /* build the actual drive id string as the disk->info.alias doesn't
      * contain the QEMU_DRIVE_HOST_PREFIX that is passed to qemu */
-    if (!(drivestr = qemuAliasFromDisk(disk)))
+    if (!(drivestr = qemuAliasDiskDriveFromDisk(disk)))
         return -1;

     if (diskPriv) {
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index 5cf9be56b4..13d131f6b6 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -400,7 +400,7 @@ qemuMigrationDstStartNBDServer(virQEMUDriverPtr driver,
         }

         VIR_FREE(diskAlias);
-        if (!(diskAlias = qemuAliasFromDisk(disk)))
+        if (!(diskAlias = qemuAliasDiskDriveFromDisk(disk)))
             goto cleanup;

         if (qemuDomainObjEnterMonitorAsync(driver, vm,
@@ -651,7 +651,7 @@ qemuMigrationSrcNBDCopyCancelOne(virQEMUDriverPtr driver,
         goto cleanup;
     }

-    if (!(diskAlias = qemuAliasFromDisk(disk)))
+    if (!(diskAlias = qemuAliasDiskDriveFromDisk(disk)))
         return -1;

     if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0)
@@ -951,7 +951,7 @@ qemuMigrationSrcNBDStorageCopy(virQEMUDriverPtr driver,
         if (!qemuMigrationAnyCopyDisk(disk, nmigrate_disks, migrate_disks))
             continue;

-        if (!(diskAlias = qemuAliasFromDisk(disk)))
+        if (!(diskAlias = qemuAliasDiskDriveFromDisk(disk)))
             goto cleanup;

         qemuBlockJobSyncBegin(disk);
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 30cc5904e0..87fc6ffc67 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -2774,7 +2774,7 @@ qemuProcessInitPasswords(virQEMUDriverPtr driver,
             goto cleanup;

         VIR_FREE(alias);
-        if (!(alias = qemuAliasFromDisk(vm->def->disks[i])))
+        if (!(alias = qemuAliasDiskDriveFromDisk(vm->def->disks[i])))
             goto cleanup;
         if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0)
             goto cleanup;
-- 
2.16.2




More information about the libvir-list mailing list