[PATCH v4 2/2] domain_addr.c: Update console port reservation comment and allowZero variable for clarity

[Aaron M. Brown]

Reviewed-by: Boris Fiuczynski <fiu...@linux.ibm.com>
Signed-off-by: Aaron M. Brown <aaron...@linux.ibm.com>
---
 src/conf/domain_addr.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/conf/domain_addr.c b/src/conf/domain_addr.c
index bc2b0f50e8..86f5cfc337 100644
--- a/src/conf/domain_addr.c
+++ b/src/conf/domain_addr.c
@@ -1638,14 +1638,14 @@ static int
 virDomainVirtioSerialAddrNext(virDomainDef *def,
                               virDomainVirtioSerialAddrSet *addrs,
                               virDomainDeviceVirtioSerialAddress *addr,
-                              bool allowZero)
+                              bool allowPortZero)
 {
     ssize_t port, startPort = 0;
     ssize_t i;
     unsigned int controller;
 
-    /* port number 0 is reserved for virtconsoles */
-    if (allowZero)
+    /* port number 0 is reserved for the first virtconsole */
+    if (allowPortZero)
         startPort = -1;
 
     if (addrs->ncontrollers == 0) {
@@ -1725,11 +1725,11 @@ static int ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3)
 virDomainVirtioSerialAddrAssign(virDomainDef *def,
                                 virDomainVirtioSerialAddrSet *addrs,
                                 virDomainDeviceInfo *info,
-                                bool allowZero,
+                                bool allowPortZero,
                                 bool portOnly)
 {
     virDomainDeviceInfo nfo = { 0 };
-    virDomainDeviceInfo *ptr = allowZero ? &nfo : info;
+    virDomainDeviceInfo *ptr = allowPortZero ? &nfo : info;
 
     ptr->type = VIR_DOMAIN_DEVICE_ADDRESS_TYPE_VIRTIO_SERIAL;
 
@@ -1745,7 +1745,7 @@ virDomainVirtioSerialAddrAssign(virDomainDef *def,
 
     } else {
         if (virDomainVirtioSerialAddrNext(def, addrs, &ptr->addr.vioserial,
-                                          allowZero) < 0)
+                                          allowPortZero) < 0)
             return -1;
     }
 
@@ -1764,20 +1764,20 @@ int
 virDomainVirtioSerialAddrAutoAssignFromCache(virDomainDef *def,
                                              virDomainVirtioSerialAddrSet 
*addrs,
                                              virDomainDeviceInfo *info,
-                                             bool allowZero)
+                                             bool allowPortZero)
 {
     bool portOnly = info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_VIRTIO_SERIAL;
     if (info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_VIRTIO_SERIAL &&
         info->addr.vioserial.port)
         return virDomainVirtioSerialAddrReserve(NULL, NULL, info, addrs);
     else
-        return virDomainVirtioSerialAddrAssign(def, addrs, info, allowZero, 
portOnly);
+        return virDomainVirtioSerialAddrAssign(def, addrs, info, 
allowPortZero, portOnly);
 }
 
 int
 virDomainVirtioSerialAddrAutoAssign(virDomainDef *def,
                                     virDomainDeviceInfo *info,
-                                    bool allowZero)
+                                    bool allowPortZero)
 {
     virDomainVirtioSerialAddrSet *addrs = NULL;
     int ret = -1;
@@ -1785,7 +1785,7 @@ virDomainVirtioSerialAddrAutoAssign(virDomainDef *def,
     if (!(addrs = virDomainVirtioSerialAddrSetCreateFromDomain(def)))
         goto cleanup;
 
-    if (virDomainVirtioSerialAddrAutoAssignFromCache(def, addrs, info, 
allowZero) < 0)
+    if (virDomainVirtioSerialAddrAutoAssignFromCache(def, addrs, info, 
allowPortZero) < 0)
         goto cleanup;
 
     ret = 0;
-- 
2.39.5 (Apple Git-154)