[libvirt] [PATCH 02/17] cpu_x86: Rename struct x86_feature

Jiri Denemark jdenemar at redhat.com
Sat May 14 08:30:21 UTC 2016


Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
---
 src/cpu/cpu_x86.c | 56 ++++++++++++++++++++++++++++---------------------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
index f6fa9e7..29680b4 100644
--- a/src/cpu/cpu_x86.c
+++ b/src/cpu/cpu_x86.c
@@ -53,11 +53,13 @@ struct _virCPUx86Vendor {
     virCPUx86VendorPtr next;
 };
 
-struct x86_feature {
+typedef struct _virCPUx86Feature virCPUx86Feature;
+typedef virCPUx86Feature *virCPUx86FeaturePtr;
+struct _virCPUx86Feature {
     char *name;
     virCPUx86Data *data;
 
-    struct x86_feature *next;
+    virCPUx86FeaturePtr next;
 };
 
 struct x86_kvm_feature {
@@ -97,9 +99,9 @@ struct x86_model {
 
 struct x86_map {
     virCPUx86VendorPtr vendors;
-    struct x86_feature *features;
+    virCPUx86FeaturePtr features;
     struct x86_model *models;
-    struct x86_feature *migrate_blockers;
+    virCPUx86FeaturePtr migrate_blockers;
 };
 
 static struct x86_map* virCPUx86Map;
@@ -404,7 +406,7 @@ x86DataToCPUFeatures(virCPUDefPtr cpu,
                      virCPUx86Data *data,
                      const struct x86_map *map)
 {
-    const struct x86_feature *feature = map->features;
+    virCPUx86FeaturePtr feature = map->features;
 
     while (feature != NULL) {
         if (x86DataIsSubset(data, feature->data)) {
@@ -573,10 +575,10 @@ x86VendorLoad(xmlXPathContextPtr ctxt,
 }
 
 
-static struct x86_feature *
+static virCPUx86FeaturePtr
 x86FeatureNew(void)
 {
-    struct x86_feature *feature;
+    virCPUx86FeaturePtr feature;
 
     if (VIR_ALLOC(feature) < 0)
         return NULL;
@@ -591,7 +593,7 @@ x86FeatureNew(void)
 
 
 static void
-x86FeatureFree(struct x86_feature *feature)
+x86FeatureFree(virCPUx86FeaturePtr feature)
 {
     if (feature == NULL)
         return;
@@ -602,10 +604,10 @@ x86FeatureFree(struct x86_feature *feature)
 }
 
 
-static struct x86_feature *
-x86FeatureCopy(const struct x86_feature *src)
+static virCPUx86FeaturePtr
+x86FeatureCopy(virCPUx86FeaturePtr src)
 {
-    struct x86_feature *feature;
+    virCPUx86FeaturePtr feature;
 
     if (VIR_ALLOC(feature) < 0)
         return NULL;
@@ -624,11 +626,11 @@ x86FeatureCopy(const struct x86_feature *src)
 }
 
 
-static struct x86_feature *
+static virCPUx86FeaturePtr
 x86FeatureFind(const struct x86_map *map,
                const char *name)
 {
-    struct x86_feature *feature;
+    virCPUx86FeaturePtr feature;
 
     feature = map->features;
     while (feature != NULL) {
@@ -650,7 +652,7 @@ x86FeatureNames(const struct x86_map *map,
     virBuffer ret = VIR_BUFFER_INITIALIZER;
     bool first = true;
 
-    struct x86_feature *next_feature = map->features;
+    virCPUx86FeaturePtr next_feature = map->features;
 
     virBufferAdd(&ret, "", 0);
 
@@ -705,14 +707,14 @@ x86FeatureLoad(xmlXPathContextPtr ctxt,
 {
     xmlNodePtr *nodes = NULL;
     xmlNodePtr ctxt_node = ctxt->node;
-    struct x86_feature *feature;
+    virCPUx86FeaturePtr feature;
     virCPUx86CPUID cpuid;
     int ret = 0;
     size_t i;
     int n;
     char *str = NULL;
     bool migratable = true;
-    struct x86_feature *migrate_blocker = NULL;
+    virCPUx86FeaturePtr migrate_blocker = NULL;
 
     if (!(feature = x86FeatureNew()))
         goto error;
@@ -793,7 +795,7 @@ x86DataFromCPUFeatures(virCPUDefPtr cpu,
         return NULL;
 
     for (i = 0; i < cpu->nfeatures; i++) {
-        const struct x86_feature *feature;
+        virCPUx86FeaturePtr feature;
         if (!(feature = x86FeatureFind(map, cpu->features[i].name))) {
             virReportError(VIR_ERR_INTERNAL_ERROR,
                            _("Unknown CPU feature %s"), cpu->features[i].name);
@@ -901,7 +903,7 @@ x86ModelFromCPU(const virCPUDef *cpu,
     }
 
     for (i = 0; i < cpu->nfeatures; i++) {
-        const struct x86_feature *feature;
+        virCPUx86FeaturePtr feature;
 
         if (cpu->type == VIR_CPU_TYPE_GUEST
             && cpu->features[i].policy != policy)
@@ -943,7 +945,7 @@ x86ModelSubtractCPU(struct x86_model *model,
     x86DataSubtract(model->data, cpu_model->data);
 
     for (i = 0; i < cpu->nfeatures; i++) {
-        const struct x86_feature *feature;
+        virCPUx86FeaturePtr feature;
 
         if (!(feature = x86FeatureFind(map, cpu->features[i].name))) {
             virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -1076,7 +1078,7 @@ x86ModelLoad(xmlXPathContextPtr ctxt,
         goto ignore;
 
     for (i = 0; i < n; i++) {
-        const struct x86_feature *feature;
+        virCPUx86FeaturePtr feature;
         char *name;
 
         if ((name = virXMLPropString(nodes[i], "name")) == NULL) {
@@ -1126,7 +1128,7 @@ x86MapFree(struct x86_map *map)
         return;
 
     while (map->features != NULL) {
-        struct x86_feature *feature = map->features;
+        virCPUx86FeaturePtr feature = map->features;
         map->features = feature->next;
         x86FeatureFree(feature);
     }
@@ -1144,7 +1146,7 @@ x86MapFree(struct x86_map *map)
     }
 
     while (map->migrate_blockers != NULL) {
-        struct x86_feature *migrate_blocker = map->migrate_blockers;
+        virCPUx86FeaturePtr migrate_blocker = map->migrate_blockers;
         map->migrate_blockers = migrate_blocker->next;
         x86FeatureFree(migrate_blocker);
     }
@@ -1179,7 +1181,7 @@ static int
 x86MapLoadInternalFeatures(struct x86_map *map)
 {
     size_t i;
-    struct x86_feature *feature = NULL;
+    virCPUx86FeaturePtr feature = NULL;
 
     for (i = 0; i < ARRAY_CARDINALITY(x86_kvm_features); i++) {
         const char *name = x86_kvm_features[i].name;
@@ -1655,7 +1657,7 @@ x86Decode(virCPUDefPtr cpu,
      * features directly */
     if (flags & VIR_CONNECT_BASELINE_CPU_MIGRATABLE) {
         for (i = 0; i < cpuModel->nfeatures; i++) {
-            const struct x86_feature *feat;
+            virCPUx86FeaturePtr feat;
             for (feat = map->migrate_blockers; feat; feat = feat->next) {
                 if (STREQ(feat->name, cpuModel->features[i].name)) {
                     VIR_FREE(cpuModel->features[i].name);
@@ -2070,7 +2072,7 @@ x86UpdateCustom(virCPUDefPtr guest,
 
     for (i = 0; i < guest->nfeatures; i++) {
         if (guest->features[i].policy == VIR_CPU_FEATURE_OPTIONAL) {
-            const struct x86_feature *feature;
+            virCPUx86FeaturePtr feature;
             if (!(feature = x86FeatureFind(map, guest->features[i].name))) {
                 virReportError(VIR_ERR_INTERNAL_ERROR,
                                _("Unknown CPU feature %s"),
@@ -2108,7 +2110,7 @@ x86UpdateHostModel(virCPUDefPtr guest,
 {
     virCPUDefPtr oldguest = NULL;
     const struct x86_map *map;
-    const struct x86_feature *feat;
+    virCPUx86FeaturePtr feat;
     size_t i;
     int ret = -1;
 
@@ -2186,7 +2188,7 @@ x86HasFeature(const virCPUData *data,
               const char *name)
 {
     const struct x86_map *map;
-    struct x86_feature *feature;
+    virCPUx86FeaturePtr feature;
     int ret = -1;
 
     if (!(map = virCPUx86GetMap()))
-- 
2.8.2




More information about the libvir-list mailing list