Re: [PATCH 21/32] Convert 'int i' to 'size_t i' in src/node_device/ files

["Daniel P. Berrange" <berrange@xxxxxxxxxx>]

On Tue, Jul 09, 2013 at 02:35:24PM +0200, Ján Tomko wrote:
> On 07/08/2013 04:21 PM, Daniel P. Berrange wrote:
> > From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> > 
> > Convert the type of loop iterators named 'i', 'j', k',
> > 'ii', 'jj', 'kk', to be 'size_t' instead of 'int' or
> > 'unsigned int', also santizing 'ii', 'jj', 'kk' to use
> > the normal 'i', 'j', 'k' naming
> > 
> > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> > ---
> >  src/node_device/node_device_driver.c |  6 +++---
> >  src/node_device/node_device_hal.c    |  9 ++++++---
> >  src/nodeinfo.c                       | 24 ++++++++++++------------
> >  3 files changed, 21 insertions(+), 18 deletions(-)
> > 
> 
> > diff --git a/src/nodeinfo.c b/src/nodeinfo.c
> > index a50f892..d48c7ed 100644
> > --- a/src/nodeinfo.c
> > +++ b/src/nodeinfo.c
> > @@ -1041,7 +1041,7 @@ nodeGetCPUCount(void)
> >       * will be consecutive.
> >       */
> >      char *cpupath = NULL;
> > -    int i = 0;
> > +    size_t i = 0;
> >  
> >      if (virFileExists(SYSFS_SYSTEM_PATH "/cpu/present")) {
> >          i = linuxParseCPUmax(SYSFS_SYSTEM_PATH "/cpu/present");
> 
> linuxParseCPUmax can return -1.

Squashing in

diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c
index 56a0023..67fcc58 100644
--- a/src/node_device/node_device_udev.c
+++ b/src/node_device/node_device_udev.c
@@ -317,7 +317,8 @@ static int udevGenerateDeviceName(struct udev_device *device,
                                   virNodeDeviceDefPtr def,
                                   const char *s)
 {
-    int ret = 0, i = 0;
+    int ret = 0;
+    size_t i;
     virBuffer buf = VIR_BUFFER_INITIALIZER;
 
     virBufferAsprintf(&buf, "%s_%s",
diff --git a/src/nodeinfo.c b/src/nodeinfo.c
index c0de21d..a2a2f73 100644
--- a/src/nodeinfo.c
+++ b/src/nodeinfo.c
@@ -1025,16 +1025,17 @@ nodeGetCPUCount(void)
      * will be consecutive.
      */
     char *cpupath = NULL;
-    size_t i = 0;
+    int ncpu;
 
     if (virFileExists(SYSFS_SYSTEM_PATH "/cpu/present")) {
-        i = linuxParseCPUmax(SYSFS_SYSTEM_PATH "/cpu/present");
+        ncpu = linuxParseCPUmax(SYSFS_SYSTEM_PATH "/cpu/present");
     } else if (virFileExists(SYSFS_SYSTEM_PATH "/cpu/cpu0")) {
+        ncpu = 0;
         do {
-            i++;
+            ncpu++;
             VIR_FREE(cpupath);
-            if (virAsprintf(&cpupath, "%s/cpu/cpu%zu",
-                            SYSFS_SYSTEM_PATH, i) < 0)
+            if (virAsprintf(&cpupath, "%s/cpu/cpu%d",
+                            SYSFS_SYSTEM_PATH, ncpu) < 0)
                 return -1;
         } while (virFileExists(cpupath));
     } else {
@@ -1045,7 +1046,7 @@ nodeGetCPUCount(void)
     }
 
     VIR_FREE(cpupath);
-    return i;
+    return ncpu;
 #elif defined(__FreeBSD__)
     return freebsdNodeGetCPUCount();
 #else


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

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