]> xenbits.xensource.com Git - libvirt.git/commitdiff
src: fix incorrect indentation in function body by checking first line
authorShi Lei <shi_lei@massclouds.com>
Thu, 13 Sep 2018 08:55:20 +0000 (16:55 +0800)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 17 Sep 2018 11:29:00 +0000 (13:29 +0200)
Signed-off-by: Shi Lei <shi_lei@massclouds.com>
13 files changed:
src/bhyve/bhyve_conf.c
src/bhyve/bhyve_driver.c
src/conf/domain_conf.c
src/cpu/cpu_s390.c
src/esx/esx_vi_types.c
src/qemu/qemu_domain_address.c
src/util/vircommand.c
src/util/virnetdev.c
src/util/virnetdevmacvlan.c
src/util/virnetdevvportprofile.c
src/util/virutil.c
src/vbox/vbox_tmpl.c
src/xenconfig/xen_common.c

index 153de7b391a3298b33444218dee5cd82717514f9..60baa2e8486accbd586151ef9c8e0f53c4b5e178 100644 (file)
@@ -36,10 +36,10 @@ static void virBhyveDriverConfigDispose(void *obj);
 
 static int virBhyveConfigOnceInit(void)
 {
-     if (!VIR_CLASS_NEW(virBhyveDriverConfig, virClassForObject()))
-         return -1;
+    if (!VIR_CLASS_NEW(virBhyveDriverConfig, virClassForObject()))
+        return -1;
 
-     return 0;
+    return 0;
 }
 
 VIR_ONCE_GLOBAL_INIT(virBhyveConfig)
index 9284b51783e44a23ea76e0efab12b2611e284cc1..c9f58e1a88212ccc62021610247db6a2b321052d 100644 (file)
@@ -197,27 +197,27 @@ bhyveConnectOpen(virConnectPtr conn,
                  virConfPtr conf ATTRIBUTE_UNUSED,
                  unsigned int flags)
 {
-     virCheckFlags(VIR_CONNECT_RO, VIR_DRV_OPEN_ERROR);
+    virCheckFlags(VIR_CONNECT_RO, VIR_DRV_OPEN_ERROR);
 
-     if (STRNEQ(conn->uri->path, "/system")) {
-         virReportError(VIR_ERR_INTERNAL_ERROR,
-                        _("Unexpected bhyve URI path '%s', try bhyve:///system"),
-                        conn->uri->path);
-         return VIR_DRV_OPEN_ERROR;
-     }
+    if (STRNEQ(conn->uri->path, "/system")) {
+        virReportError(VIR_ERR_INTERNAL_ERROR,
+                       _("Unexpected bhyve URI path '%s', try bhyve:///system"),
+                       conn->uri->path);
+        return VIR_DRV_OPEN_ERROR;
+    }
 
-     if (bhyve_driver == NULL) {
-         virReportError(VIR_ERR_INTERNAL_ERROR,
-                        "%s", _("bhyve state driver is not active"));
-         return VIR_DRV_OPEN_ERROR;
-     }
+    if (bhyve_driver == NULL) {
+        virReportError(VIR_ERR_INTERNAL_ERROR,
+                       "%s", _("bhyve state driver is not active"));
+        return VIR_DRV_OPEN_ERROR;
+    }
 
-     if (virConnectOpenEnsureACL(conn) < 0)
-         return VIR_DRV_OPEN_ERROR;
+    if (virConnectOpenEnsureACL(conn) < 0)
+        return VIR_DRV_OPEN_ERROR;
 
-     conn->privateData = bhyve_driver;
+    conn->privateData = bhyve_driver;
 
-     return VIR_DRV_OPEN_SUCCESS;
+    return VIR_DRV_OPEN_SUCCESS;
 }
 
 static int
index 88b0ad45f88e9e70db7ddbc8362d116f89682111..41e7af06e928e4e0a8209701d6397af1e10c9f0f 100644 (file)
@@ -14435,7 +14435,7 @@ virDomainNVRAMDefParseXML(virDomainXMLOptionPtr xmlopt,
                           xmlNodePtr node,
                           unsigned int flags)
 {
-   virDomainNVRAMDefPtr def;
+    virDomainNVRAMDefPtr def;
 
     if (VIR_ALLOC(def) < 0)
         return NULL;
index 3d10f920bad17d7b6a2f6135024f076ddc1838e3..1abbb584a1f9b5440f79d1d98c5699468dccc924 100644 (file)
@@ -48,50 +48,50 @@ static int
 virCPUs390Update(virCPUDefPtr guest,
                  const virCPUDef *host)
 {
-     virCPUDefPtr updated = NULL;
-     int ret = -1;
-     size_t i;
-
-     if (guest->match == VIR_CPU_MATCH_MINIMUM) {
-         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
-                        _("match mode %s not supported"),
-                        virCPUMatchTypeToString(guest->match));
-         goto cleanup;
-     }
-
-     if (guest->mode != VIR_CPU_MODE_HOST_MODEL) {
-         ret = 0;
-         goto cleanup;
-     }
-
-     if (!host) {
-         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                        _("unknown host CPU model"));
-         goto cleanup;
-     }
-
-     if (!(updated = virCPUDefCopyWithoutModel(guest)))
-         goto cleanup;
-
-     updated->mode = VIR_CPU_MODE_CUSTOM;
-     if (virCPUDefCopyModel(updated, host, true) < 0)
-         goto cleanup;
-
-     for (i = 0; i < guest->nfeatures; i++) {
-        if (virCPUDefUpdateFeature(updated,
-                                   guest->features[i].name,
-                                   guest->features[i].policy) < 0)
-            goto cleanup;
-     }
-
-     virCPUDefStealModel(guest, updated, false);
-     guest->mode = VIR_CPU_MODE_CUSTOM;
-     guest->match = VIR_CPU_MATCH_EXACT;
-     ret = 0;
+    virCPUDefPtr updated = NULL;
+    int ret = -1;
+    size_t i;
+
+    if (guest->match == VIR_CPU_MATCH_MINIMUM) {
+        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+                       _("match mode %s not supported"),
+                       virCPUMatchTypeToString(guest->match));
+        goto cleanup;
+    }
+
+    if (guest->mode != VIR_CPU_MODE_HOST_MODEL) {
+        ret = 0;
+        goto cleanup;
+    }
+
+    if (!host) {
+        virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
+                       _("unknown host CPU model"));
+        goto cleanup;
+    }
+
+    if (!(updated = virCPUDefCopyWithoutModel(guest)))
+        goto cleanup;
+
+    updated->mode = VIR_CPU_MODE_CUSTOM;
+    if (virCPUDefCopyModel(updated, host, true) < 0)
+        goto cleanup;
+
+    for (i = 0; i < guest->nfeatures; i++) {
+       if (virCPUDefUpdateFeature(updated,
+                                  guest->features[i].name,
+                                  guest->features[i].policy) < 0)
+           goto cleanup;
+    }
+
+    virCPUDefStealModel(guest, updated, false);
+    guest->mode = VIR_CPU_MODE_CUSTOM;
+    guest->match = VIR_CPU_MATCH_EXACT;
+    ret = 0;
 
  cleanup:
-     virCPUDefFree(updated);
-     return ret;
+    virCPUDefFree(updated);
+    return ret;
 }
 
 
index 7b234f0c87d2740f2388bb1f1d49895bdf01b216..a21a32973a5167dd4685f4a3ef649a9e32806758 100644 (file)
@@ -1781,7 +1781,7 @@ ESX_VI__TEMPLATE__LIST__APPEND(Event)
 /* esxVI_Event_CastFromAnyType */
 ESX_VI__TEMPLATE__DYNAMIC_CAST_FROM_ANY_TYPE(Event,
 {
-      case esxVI_Type_Other:
+    case esxVI_Type_Other:
         /* Just accept everything here */
         break;
 })
index 24fdf12128bfd6b451f3329cb1f30f6ea05dde46..8f83ae261de974d89307fd78f538da3de11ecf55 100644 (file)
@@ -2301,48 +2301,49 @@ qemuDomainAddressFindNewTargetIndex(virDomainDefPtr def)
 static int
 qemuDomainAddressFindNewBusNr(virDomainDefPtr def)
 {
-/* Try to find a nice default for busNr for a new pci-expander-bus.
- * This is a bit tricky, since you need to satisfy the following:
- *
- * 1) There need to be enough unused bus numbers between busNr of this
- *    bus and busNr of the next highest bus for the guest to assign a
- *    unique bus number to each PCI bus that is a child of this
- *    bus. Each PCI controller. On top of this, the pxb device (which
- *    implements the pci-expander-bus) includes a pci-bridge within
- *    it, and that bridge also uses one bus number (so each pxb device
- *    requires at least 2 bus numbers).
- *
- * 2) There need to be enough bus numbers *below* this for all the
- *    child controllers of the pci-expander-bus with the next lower
- *    busNr (or the pci-root bus if there are no lower
- *    pci-expander-buses).
- *
- * 3) If at all possible, we want to avoid needing to change the busNr
- *    of a bus in the future, as that changes the guest's device ABI,
- *    which could potentially lead to issues with a guest OS that is
- *    picky about such things.
- *
- *  Due to the impossibility of predicting what might be added to the
- *  config in the future, we can't make a foolproof choice, but since
- *  a pci-expander-bus (pxb) has slots for 32 devices, and the only
- *  practical use for it is to assign real devices on a particular
- *  NUMA node in the host, it's reasonably safe to assume it should
- *  never need any additional child buses (probably only a few of the
- *  32 will ever be used). So for pci-expander-bus we find the lowest
- *  existing busNr, and set this one to the current lowest - 2 (one
- *  for the pxb, one for the intergrated pci-bridge), thus leaving the
- *  maximum possible bus numbers available for other buses plugged
- *  into pci-root (i.e. pci-bridges and other
- *  pci-expander-buses). Anyone who needs more than 32 devices
- *  descended from one pci-expander-bus should set the busNr manually
- *  in the config.
- *
- *  There is room for more error checking here - in particular we
- *  can/should determine the ultimate parent (root-bus) of each PCI
- *  controller and determine if there is enough space for all the
- *  buses within the current range allotted to the bus just prior to
- *  this one.
- */
+    /* Try to find a nice default for busNr for a new pci-expander-bus.
+     * This is a bit tricky, since you need to satisfy the following:
+     *
+     * 1) There need to be enough unused bus numbers between busNr of this
+     *    bus and busNr of the next highest bus for the guest to assign a
+     *    unique bus number to each PCI bus that is a child of this
+     *    bus. Each PCI controller. On top of this, the pxb device (which
+     *    implements the pci-expander-bus) includes a pci-bridge within
+     *    it, and that bridge also uses one bus number (so each pxb device
+     *    requires at least 2 bus numbers).
+     *
+     * 2) There need to be enough bus numbers *below* this for all the
+     *    child controllers of the pci-expander-bus with the next lower
+     *    busNr (or the pci-root bus if there are no lower
+     *    pci-expander-buses).
+     *
+     * 3) If at all possible, we want to avoid needing to change the busNr
+     *    of a bus in the future, as that changes the guest's device ABI,
+     *    which could potentially lead to issues with a guest OS that is
+     *    picky about such things.
+     *
+     *  Due to the impossibility of predicting what might be added to the
+     *  config in the future, we can't make a foolproof choice, but since
+     *  a pci-expander-bus (pxb) has slots for 32 devices, and the only
+     *  practical use for it is to assign real devices on a particular
+     *  NUMA node in the host, it's reasonably safe to assume it should
+     *  never need any additional child buses (probably only a few of the
+     *  32 will ever be used). So for pci-expander-bus we find the lowest
+     *  existing busNr, and set this one to the current lowest - 2 (one
+     *  for the pxb, one for the intergrated pci-bridge), thus leaving the
+     *  maximum possible bus numbers available for other buses plugged
+     *  into pci-root (i.e. pci-bridges and other
+     *  pci-expander-buses). Anyone who needs more than 32 devices
+     *  descended from one pci-expander-bus should set the busNr manually
+     *  in the config.
+     *
+     *  There is room for more error checking here - in particular we
+     *  can/should determine the ultimate parent (root-bus) of each PCI
+     *  controller and determine if there is enough space for all the
+     *  buses within the current range allotted to the bus just prior to
+     *  this one.
+     */
+
     size_t i;
     int lowestBusNr = 256;
 
index 8be3fdffbc468dd255a5c96b1dd348a5181a09bf..9a9927566c302a6326aa3858c16753be15be9afd 100644 (file)
@@ -2872,10 +2872,10 @@ virCommandFree(virCommandPtr cmd)
 void
 virCommandDoAsyncIO(virCommandPtr cmd)
 {
-   if (!cmd || cmd->has_error)
-       return;
+    if (!cmd || cmd->has_error)
+        return;
 
-   cmd->flags |= VIR_EXEC_ASYNC_IO | VIR_EXEC_NONBLOCK;
+    cmd->flags |= VIR_EXEC_ASYNC_IO | VIR_EXEC_NONBLOCK;
 }
 
 /**
index a8922516a963bafab7be81f040f944462ed4bc6c..c30a60072a3ece89c36a9d3b319aa318834fa075 100644 (file)
@@ -295,36 +295,36 @@ virNetDevSetMACInternal(const char *ifname,
                         const virMacAddr *macaddr,
                         bool quiet)
 {
-        struct ifreq ifr;
-        struct sockaddr_dl sdl;
-        char mac[VIR_MAC_STRING_BUFLEN + 1] = ":";
-        VIR_AUTOCLOSE s = -1;
-
-        if ((s = virNetDevSetupControl(ifname, &ifr)) < 0)
-            return -1;
+    struct ifreq ifr;
+    struct sockaddr_dl sdl;
+    char mac[VIR_MAC_STRING_BUFLEN + 1] = ":";
+    VIR_AUTOCLOSE s = -1;
 
-        virMacAddrFormat(macaddr, mac + 1);
-        sdl.sdl_len = sizeof(sdl);
-        link_addr(mac, &sdl);
+    if ((s = virNetDevSetupControl(ifname, &ifr)) < 0)
+        return -1;
 
-        memcpy(ifr.ifr_addr.sa_data, sdl.sdl_data, VIR_MAC_BUFLEN);
-        ifr.ifr_addr.sa_len = VIR_MAC_BUFLEN;
+    virMacAddrFormat(macaddr, mac + 1);
+    sdl.sdl_len = sizeof(sdl);
+    link_addr(mac, &sdl);
 
-        if (ioctl(s, SIOCSIFLLADDR, &ifr) < 0) {
-            if (quiet &&
-                (errno == EADDRNOTAVAIL || errno == EPERM)) {
-                VIR_DEBUG("SIOCSIFLLADDR %s MAC=%s - Fail", ifname, mac + 1);
-                return -1;
-            }
+    memcpy(ifr.ifr_addr.sa_data, sdl.sdl_data, VIR_MAC_BUFLEN);
+    ifr.ifr_addr.sa_len = VIR_MAC_BUFLEN;
 
-            virReportSystemError(errno,
-                                 _("Cannot set interface MAC to %s on '%s'"),
-                                 mac + 1, ifname);
+    if (ioctl(s, SIOCSIFLLADDR, &ifr) < 0) {
+        if (quiet &&
+            (errno == EADDRNOTAVAIL || errno == EPERM)) {
+            VIR_DEBUG("SIOCSIFLLADDR %s MAC=%s - Fail", ifname, mac + 1);
             return -1;
         }
 
-        VIR_DEBUG("SIOCSIFLLADDR %s MAC=%s - Success", ifname, mac + 1);
-        return 0;
+        virReportSystemError(errno,
+                             _("Cannot set interface MAC to %s on '%s'"),
+                             mac + 1, ifname);
+        return -1;
+    }
+
+    VIR_DEBUG("SIOCSIFLLADDR %s MAC=%s - Success", ifname, mac + 1);
+    return 0;
 }
 
 
index a66ab59952901c357e5548ec91b367d409dc3b42..eff3ef94d0bcdedc9195e527e45e69661231c8ae 100644 (file)
@@ -546,11 +546,11 @@ virNetDevMacVLanVPortProfileCallback(struct nlmsghdr *hdr,
                                      bool *handled,
                                      void *opaque)
 {
-   struct nla_policy ifla_vf_policy[IFLA_VF_MAX + 1] = {
-       [IFLA_VF_MAC] = {.minlen = sizeof(struct ifla_vf_mac),
-                        .maxlen = sizeof(struct ifla_vf_mac)},
-       [IFLA_VF_VLAN] = {.minlen = sizeof(struct ifla_vf_vlan),
-                         .maxlen = sizeof(struct ifla_vf_vlan)},
+    struct nla_policy ifla_vf_policy[IFLA_VF_MAX + 1] = {
+        [IFLA_VF_MAC] = {.minlen = sizeof(struct ifla_vf_mac),
+                         .maxlen = sizeof(struct ifla_vf_mac)},
+        [IFLA_VF_VLAN] = {.minlen = sizeof(struct ifla_vf_vlan),
+                          .maxlen = sizeof(struct ifla_vf_vlan)},
     };
 
     struct nla_policy ifla_port_policy[IFLA_PORT_MAX + 1] = {
index 3ebf757fb30564a33efe1a4f97b77629d4a5ff70..215d649c33af7d39bacd40b86fb52e9ae46ca996 100644 (file)
@@ -451,7 +451,7 @@ int virNetDevVPortProfileMerge3(virNetDevVPortProfilePtr *result,
 
 static struct nla_policy ifla_port_policy[IFLA_PORT_MAX + 1] =
 {
-  [IFLA_PORT_RESPONSE]      = { .type = NLA_U16 },
+    [IFLA_PORT_RESPONSE] = { .type = NLA_U16 },
 };
 
 static uint32_t
index b0334edb36a087381c7e69e455dc72649027ac55..aaa3cfae686e6f71adc742c258e06cae3ffe7835 100644 (file)
@@ -940,7 +940,7 @@ char *virGetUserConfigDirectory(void)
 
 char *virGetUserCacheDirectory(void)
 {
-     return virGetXDGDirectory("XDG_CACHE_HOME", ".cache");
+    return virGetXDGDirectory("XDG_CACHE_HOME", ".cache");
 }
 
 char *virGetUserRuntimeDirectory(void)
index 88792c9928970ba42bda4e1c94affb3e58dc3ab5..53d4fd7e1804c31a76223ef11566fce7843dbf74 100644 (file)
@@ -1253,11 +1253,11 @@ _systemPropertiesGetMaxNetworkAdapters(ISystemProperties *systemProperties, PRUi
                                        PRUint32 *maxNetworkAdapters)
 {
 #if VBOX_API_VERSION < 4001000
-        return systemProperties->vtbl->GetNetworkAdapterCount(systemProperties,
-                                                              maxNetworkAdapters);
+    return systemProperties->vtbl->GetNetworkAdapterCount(systemProperties,
+                                                          maxNetworkAdapters);
 #else  /* VBOX_API_VERSION >= 4000000 */
-        return systemProperties->vtbl->GetMaxNetworkAdapters(systemProperties, chipset,
-                                                             maxNetworkAdapters);
+    return systemProperties->vtbl->GetMaxNetworkAdapters(systemProperties, chipset,
+                                                         maxNetworkAdapters);
 #endif /* VBOX_API_VERSION >= 4000000 */
 }
 
index fdca9845aa544a0a8041927339395a35b1f4c57a..be9e2328b0bdc2e2b57eaa50cec40e02e1aac1d7 100644 (file)
@@ -1911,11 +1911,11 @@ xenFormatVif(virConfPtr conf,
              virDomainDefPtr def,
              const char *vif_typename)
 {
-   virConfValuePtr netVal = NULL;
-   size_t i;
-   int hvm = def->os.type == VIR_DOMAIN_OSTYPE_HVM;
+    virConfValuePtr netVal = NULL;
+    size_t i;
+    int hvm = def->os.type == VIR_DOMAIN_OSTYPE_HVM;
 
-   if (VIR_ALLOC(netVal) < 0)
+    if (VIR_ALLOC(netVal) < 0)
         goto cleanup;
     netVal->type = VIR_CONF_LIST;
     netVal->list = NULL;
@@ -1923,7 +1923,7 @@ xenFormatVif(virConfPtr conf,
     for (i = 0; i < def->nnets; i++) {
         if (xenFormatNet(conn, netVal, def->nets[i],
                          hvm, vif_typename) < 0)
-           goto cleanup;
+            goto cleanup;
     }
 
     if (netVal->list != NULL) {