On 05/03/2012 01:53 PM, Li Zhang wrote:
There are three spapces for a indent every line in the
function qemuDomainNetsRestart.
Yeah, the codes exist indentation issues, in addition, also need to align function
arguments in 'qemuDomainNetsRestart'():

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 86e82d6..e2bad02 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -417,8 +417,8 @@ cleanup:


 static void qemuDomainNetsRestart(void *payload,
-                                const void *name ATTRIBUTE_UNUSED,
-                                void *data ATTRIBUTE_UNUSED)
+                                  const void *name ATTRIBUTE_UNUSED,
+                                  void *data ATTRIBUTE_UNUSED)

This patch is to correct it.

Signed-off-by: Li Zhang<zhlci...@linux.vnet.ibm.com>
---
  src/qemu/qemu_driver.c |   42 +++++++++++++++++++++---------------------
  1 file changed, 21 insertions(+), 21 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 86e82d6..bed8edb 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -420,28 +420,28 @@ static void qemuDomainNetsRestart(void *payload,
                                  const void *name ATTRIBUTE_UNUSED,
                                  void *data ATTRIBUTE_UNUSED)
  {
-   int i;
-   virDomainObjPtr vm = (virDomainObjPtr)payload;
-   virDomainDefPtr def = vm->def;
-
-   virDomainObjLock(vm);
-
-   for (i = 0; i<  def->nnets; i++) {
-       virDomainNetDefPtr net = def->nets[i];
-       if (virDomainNetGetActualType(net) == VIR_DOMAIN_NET_TYPE_DIRECT&&
-           virDomainNetGetActualDirectMode(net) == 
VIR_NETDEV_MACVLAN_MODE_VEPA) {
-           VIR_DEBUG("VEPA mode device %s active in domain %s. Reassociating.",
-                     net->ifname, def->name);
-           ignore_value(virNetDevMacVLanRestartWithVPortProfile(net->ifname,
-                                                                net->mac,
-                                                                
virDomainNetGetActualDirectDev(net),
-                                                                def->uuid,
-                                                                
virDomainNetGetActualVirtPortProfile(net),
-                                                                
VIR_NETDEV_VPORT_PROFILE_OP_CREATE));
-       }
-   }
+    int i;
+    virDomainObjPtr vm = (virDomainObjPtr)payload;
+    virDomainDefPtr def = vm->def;
+
+    virDomainObjLock(vm);

-   virDomainObjUnlock(vm);
+    for (i = 0; i<  def->nnets; i++) {
+        virDomainNetDefPtr net = def->nets[i];
+        if (virDomainNetGetActualType(net) == VIR_DOMAIN_NET_TYPE_DIRECT&&
+            virDomainNetGetActualDirectMode(net) == 
VIR_NETDEV_MACVLAN_MODE_VEPA) {
+            VIR_DEBUG("VEPA mode device %s active in domain %s. 
Reassociating.",
+                      net->ifname, def->name);
+            ignore_value(virNetDevMacVLanRestartWithVPortProfile(net->ifname,
+                                                                 net->mac,
+                                                                 
virDomainNetGetActualDirectDev(net),
+                                                                 def->uuid,
+                                                                 
virDomainNetGetActualVirtPortProfile(net),
+                                                                 
VIR_NETDEV_VPORT_PROFILE_OP_CREATE));
+        }
+    }
+
+    virDomainObjUnlock(vm);
  }

  /**

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

Reply via email to