]> xenbits.xensource.com Git - libvirt.git/commitdiff
conf: nodedev: Rename virNodeDevObjHasCap to virNodeDevObjHasCapStr
authorErik Skultety <eskultet@redhat.com>
Thu, 11 Jan 2018 12:43:29 +0000 (13:43 +0100)
committerErik Skultety <eskultet@redhat.com>
Mon, 29 Jan 2018 14:34:30 +0000 (15:34 +0100)
We currently have 2 methods that do the capability matching. This should
be condensed to a single function and all the derivates should just call
into that using a proper type conversion.

Signed-off-by: Erik Skultety <eskultet@redhat.com>
src/conf/virnodedeviceobj.c

index c4e3a40d3ab38bedf6980433baf35cf5880c5b9e..a4d38b3a1ceb0a751cd395b8e8e494747a9ce41d 100644 (file)
@@ -53,6 +53,8 @@ static virClassPtr virNodeDeviceObjClass;
 static virClassPtr virNodeDeviceObjListClass;
 static void virNodeDeviceObjDispose(void *opaque);
 static void virNodeDeviceObjListDispose(void *opaque);
+static bool virNodeDeviceObjHasCapStr(const virNodeDeviceObj *obj,
+                                      const char *cap);
 
 static int
 virNodeDeviceObjOnceInit(void)
@@ -121,8 +123,8 @@ virNodeDeviceObjGetDef(virNodeDeviceObjPtr obj)
 
 
 static bool
-virNodeDeviceObjHasCap(const virNodeDeviceObj *obj,
-                       const char *cap)
+virNodeDeviceObjHasCapStr(const virNodeDeviceObj *obj,
+                          const char *cap)
 {
     virNodeDevCapsDefPtr caps = obj->def->caps;
     const char *fc_host_cap =
@@ -375,7 +377,7 @@ virNodeDeviceObjListFindByCapCallback(const void *payload,
     int want = 0;
 
     virObjectLock(obj);
-    if (virNodeDeviceObjHasCap(obj, matchstr))
+    if (virNodeDeviceObjHasCapStr(obj, matchstr))
         want = 1;
     virObjectUnlock(obj);
     return want;
@@ -750,7 +752,7 @@ virNodeDeviceObjListNumOfDevicesCallback(void *payload,
     virObjectLock(obj);
     def = obj->def;
     if ((!filter || filter(data->conn, def)) &&
-        (!data->matchstr || virNodeDeviceObjHasCap(obj, data->matchstr)))
+        (!data->matchstr || virNodeDeviceObjHasCapStr(obj, data->matchstr)))
         data->count++;
 
     virObjectUnlock(obj);
@@ -805,7 +807,7 @@ virNodeDeviceObjListGetNamesCallback(void *payload,
     def = obj->def;
 
     if ((!filter || filter(data->conn, def)) &&
-        (!data->matchstr || virNodeDeviceObjHasCap(obj, data->matchstr))) {
+        (!data->matchstr || virNodeDeviceObjHasCapStr(obj, data->matchstr))) {
         if (VIR_STRDUP(data->names[data->nnames], def->name) < 0) {
             data->error = true;
             goto cleanup;