[libvirt] [PATCH v5 10/16] qemu: Use virDomainPCIControllerOpts in qemuBuildControllerDevStr

Friday, 5 January 2018

Shorten up a few characters and reference the pciopts pointer

Signed-off-by: John Ferlan <jferlan(a)redhat.com&gt;
---
 src/qemu/qemu_command.c | 118 ++++++++++++++++++++++++------------------------
 1 file changed, 60 insertions(+), 58 deletions(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index d62531110..0175daee3 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -2612,6 +2612,7 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
 {
     virBuffer buf = VIR_BUFFER_INITIALIZER;
     int model = def->model;
+    const virDomainPCIControllerOpts *pciopts;
     const char *modelName = NULL;
 
     *devstr = NULL;
@@ -2718,24 +2719,26 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
         break;
 
     case VIR_DOMAIN_CONTROLLER_TYPE_PCI:
+        pciopts = &def->opts.pciopts;
+
         switch ((virDomainControllerModelPCI) def->model) {
         case VIR_DOMAIN_CONTROLLER_MODEL_PCI_BRIDGE:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.chassisNr == -1) {
+                pciopts->chassisNr == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pci-bridge options not set"));
                 goto error;
             }
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pci-bridge model name value %d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PCI_BRIDGE) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2750,26 +2753,26 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                 goto error;
             }
             virBufferAsprintf(&buf, "%s,chassis_nr=%d,id=%s",
-                              modelName, def->opts.pciopts.chassisNr,
+                              modelName, pciopts->chassisNr,
                               def->info.alias);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCI_EXPANDER_BUS:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.busNr == -1) {
+                pciopts->busNr == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pci-expander-bus options not
set"));
                 goto error;
             }
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pci-expander-bus model name value
%d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PXB) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2784,28 +2787,28 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                 goto error;
             }
             virBufferAsprintf(&buf, "%s,bus_nr=%d,id=%s",
-                              modelName, def->opts.pciopts.busNr,
+                              modelName, pciopts->busNr,
                               def->info.alias);
-            if (def->opts.pciopts.numaNode != -1)
+            if (pciopts->numaNode != -1)
                virBufferAsprintf(&buf, ",numa_node=%d",
-                                 def->opts.pciopts.numaNode);
+                                 pciopts->numaNode);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_DMI_TO_PCI_BRIDGE:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated dmi-to-pci-bridge options not
set"));
                 goto error;
             }
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown dmi-to-pci-bridge model name value
%d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_I82801B11_BRIDGE) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2822,24 +2825,23 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
             virBufferAsprintf(&buf, "%s,id=%s", modelName,
def->info.alias);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCIE_ROOT_PORT:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.chassis == -1 ||
-                def->opts.pciopts.port == -1) {
+                pciopts->chassis == -1 || pciopts->port == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pcie-root-port options not
set"));
                 goto error;
             }
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pcie-root-port model name value
%d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if ((def->opts.pciopts.modelName !=
+            if ((pciopts->modelName !=
                  VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_IOH3420) &&
-                (def->opts.pciopts.modelName !=
+                (pciopts->modelName !=
                  VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PCIE_ROOT_PORT)) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2847,7 +2849,7 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                                modelName);
                 goto error;
             }
-            if ((def->opts.pciopts.modelName ==
+            if ((pciopts->modelName ==
                  VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_IOH3420) &&
                 !virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_IOH3420)) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
@@ -2855,7 +2857,7 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                                  "controller is not supported in this QEMU
binary"));
                 goto error;
             }
-            if ((def->opts.pciopts.modelName ==
+            if ((pciopts->modelName ==
                  VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PCIE_ROOT_PORT) &&
                 !virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE_PCIE_ROOT_PORT)) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
@@ -2865,24 +2867,24 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
             }
 
             virBufferAsprintf(&buf, "%s,port=0x%x,chassis=%d,id=%s",
-                              modelName, def->opts.pciopts.port,
-                              def->opts.pciopts.chassis, def->info.alias);
+                              modelName, pciopts->port,
+                              pciopts->chassis, def->info.alias);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCIE_SWITCH_UPSTREAM_PORT:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pcie-switch-upstream-port options
not set"));
                 goto error;
             }
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pcie-switch-upstream-port model name value
%d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_X3130_UPSTREAM) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2900,24 +2902,24 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
             virBufferAsprintf(&buf, "%s,id=%s", modelName,
def->info.alias);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCIE_SWITCH_DOWNSTREAM_PORT:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.chassis == -1 ||
-                def->opts.pciopts.port == -1) {
+                pciopts->chassis == -1 ||
+                pciopts->port == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pcie-switch-downstream-port "
                                  "options not set"));
                 goto error;
             }
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pcie-switch-downstream-port model name
value %d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_XIO3130_DOWNSTREAM) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2933,26 +2935,26 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                 goto error;
             }
             virBufferAsprintf(&buf, "%s,port=0x%x,chassis=%d,id=%s",
-                              modelName, def->opts.pciopts.port,
-                              def->opts.pciopts.chassis, def->info.alias);
+                              modelName, pciopts->port,
+                              pciopts->chassis, def->info.alias);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCIE_EXPANDER_BUS:
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.busNr == -1) {
+                pciopts->busNr == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pcie-expander-bus options not
set"));
                 goto error;
             }
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pcie-expander-bus model name value
%d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName
+            if (pciopts->modelName
                 != VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PXB_PCIE) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' "
@@ -2967,32 +2969,32 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                 goto error;
             }
             virBufferAsprintf(&buf, "%s,bus_nr=%d,id=%s",
-                              modelName, def->opts.pciopts.busNr,
+                              modelName, pciopts->busNr,
                               def->info.alias);
-            if (def->opts.pciopts.numaNode != -1)
+            if (pciopts->numaNode != -1)
                virBufferAsprintf(&buf, ",numa_node=%d",
-                                 def->opts.pciopts.numaNode);
+                                 pciopts->numaNode);
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCI_ROOT:
-            if (def->opts.pciopts.modelName ==
VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
-                def->opts.pciopts.targetIndex == -1) {
+            if (pciopts->modelName == VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE ||
+                pciopts->targetIndex == -1) {
                 virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                                _("autogenerated pci-root options not set"));
                 goto error;
             }
 
             /* Skip the implicit one */
-            if (def->opts.pciopts.targetIndex == 0)
+            if (pciopts->targetIndex == 0)
                 goto done;
 
-            modelName =
virDomainControllerPCIModelNameTypeToString(def->opts.pciopts.modelName);
+            modelName =
virDomainControllerPCIModelNameTypeToString(pciopts->modelName);
             if (!modelName) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("unknown pci-root model name value %d"),
-                               def->opts.pciopts.modelName);
+                               pciopts->modelName);
                 goto error;
             }
-            if (def->opts.pciopts.modelName !=
VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_SPAPR_PCI_HOST_BRIDGE) {
+            if (pciopts->modelName !=
VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_SPAPR_PCI_HOST_BRIDGE) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                                _("PCI controller model name '%s' is not
valid for a pci-root"),
                                modelName);
@@ -3005,17 +3007,17 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
                 goto error;
             }
             virBufferAsprintf(&buf, "%s,index=%d,id=%s",
-                              modelName, def->opts.pciopts.targetIndex,
+                              modelName, pciopts->targetIndex,
                               def->info.alias);
 
-            if (def->opts.pciopts.numaNode != -1) {
+            if (pciopts->numaNode != -1) {
                 if (!virQEMUCapsGet(qemuCaps, QEMU_CAPS_SPAPR_PCI_HOST_BRIDGE_NUMA_NODE))
{
                     virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                                    _("the spapr-pci-host-bridge controller "
                                      "doesn't support numa_node on this QEMU
binary"));
                     goto error;
                 }
-                virBufferAsprintf(&buf, ",numa_node=%d",
def->opts.pciopts.numaNode);
+                virBufferAsprintf(&buf, ",numa_node=%d",
pciopts->numaNode);
             }
             break;
         case VIR_DOMAIN_CONTROLLER_MODEL_PCIE_ROOT:
-- 
2.13.6


    

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

[libvirt] [PATCH v5 10/16] qemu: Use virDomainPCIControllerOpts in qemuBuildControllerDevStr