9ae3a8
From 8b2c0bd8595d7e175c04f80fbc3035cd538aab2b Mon Sep 17 00:00:00 2001
9ae3a8
From: Vadim Rozenfeld <vrozenfe@redhat.com>
9ae3a8
Date: Mon, 3 Mar 2014 12:09:21 +0100
9ae3a8
Subject: [PATCH 09/12] target-i386: Convert 'hv_spinlocks' to static property
9ae3a8
MIME-Version: 1.0
9ae3a8
Content-Type: text/plain; charset=UTF-8
9ae3a8
Content-Transfer-Encoding: 8bit
9ae3a8
9ae3a8
RH-Author: Vadim Rozenfeld <vrozenfe@redhat.com>
9ae3a8
Message-id: <1393848564-10511-10-git-send-email-vrozenfe@redhat.com>
9ae3a8
Patchwork-id: 57965
9ae3a8
O-Subject: [RHEL-7.0 qemu-kvm v4 PATCH 09/12] target-i386: Convert 'hv_spinlocks' to static property
9ae3a8
Bugzilla: 1057173
9ae3a8
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
9ae3a8
RH-Acked-by: Igor Mammedov <imammedo@redhat.com>
9ae3a8
RH-Acked-by: Eduardo Habkost <ehabkost@redhat.com>
9ae3a8
RH-Acked-by: Juan Quintela <quintela@redhat.com>
9ae3a8
9ae3a8
From: Igor Mammedov <imammedo@redhat.com>
9ae3a8
9ae3a8
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
9ae3a8
Signed-off-by: Andreas Färber <afaerber@suse.de>
9ae3a8
(cherry picked from commit c8f0f88e2a4cf27bde27a31a98badd61fe212652)
9ae3a8
---
9ae3a8
 target-i386/cpu.c | 45 ++++++++++++++++++++++++++++++++++++++++++++-
9ae3a8
 1 file changed, 44 insertions(+), 1 deletion(-)
9ae3a8
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 target-i386/cpu.c |   45 ++++++++++++++++++++++++++++++++++++++++++++-
9ae3a8
 1 files changed, 44 insertions(+), 1 deletions(-)
9ae3a8
9ae3a8
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
9ae3a8
index 7a8cb01..db51f72 100644
9ae3a8
--- a/target-i386/cpu.c
9ae3a8
+++ b/target-i386/cpu.c
9ae3a8
@@ -1511,6 +1511,46 @@ static void x86_cpu_get_feature_words(Object *obj, Visitor *v, void *opaque,
9ae3a8
     error_propagate(errp, err);
9ae3a8
 }
9ae3a8
 
9ae3a8
+static void x86_get_hv_spinlocks(Object *obj, Visitor *v, void *opaque,
9ae3a8
+                                 const char *name, Error **errp)
9ae3a8
+{
9ae3a8
+    X86CPU *cpu = X86_CPU(obj);
9ae3a8
+    int64_t value = cpu->hyperv_spinlock_attempts;
9ae3a8
+
9ae3a8
+    visit_type_int(v, &value, name, errp);
9ae3a8
+}
9ae3a8
+
9ae3a8
+static void x86_set_hv_spinlocks(Object *obj, Visitor *v, void *opaque,
9ae3a8
+                                 const char *name, Error **errp)
9ae3a8
+{
9ae3a8
+    const int64_t min = 0xFFF;
9ae3a8
+    const int64_t max = UINT_MAX;
9ae3a8
+    X86CPU *cpu = X86_CPU(obj);
9ae3a8
+    Error *err = NULL;
9ae3a8
+    int64_t value;
9ae3a8
+
9ae3a8
+    visit_type_int(v, &value, name, &err;;
9ae3a8
+    if (err) {
9ae3a8
+        error_propagate(errp, err);
9ae3a8
+        return;
9ae3a8
+    }
9ae3a8
+
9ae3a8
+    if (value < min || value > max) {
9ae3a8
+        error_setg(errp, "Property %s.%s doesn't take value %" PRId64
9ae3a8
+                  " (minimum: %" PRId64 ", maximum: %" PRId64 ")",
9ae3a8
+                  object_get_typename(obj), name ? name : "null",
9ae3a8
+                  value, min, max);
9ae3a8
+        return;
9ae3a8
+    }
9ae3a8
+    cpu->hyperv_spinlock_attempts = value;
9ae3a8
+}
9ae3a8
+
9ae3a8
+static PropertyInfo qdev_prop_spinlocks = {
9ae3a8
+    .name  = "int",
9ae3a8
+    .get   = x86_get_hv_spinlocks,
9ae3a8
+    .set   = x86_set_hv_spinlocks,
9ae3a8
+};
9ae3a8
+
9ae3a8
 static int cpu_x86_find_by_name(X86CPU *cpu, x86_def_t *x86_cpu_def,
9ae3a8
                                 const char *name)
9ae3a8
 {
9ae3a8
@@ -1628,6 +1668,7 @@ static void cpu_x86_parse_featurestr(X86CPU *cpu, char *features, Error **errp)
9ae3a8
             } else if (!strcmp(featurestr, "hv-spinlocks")) {
9ae3a8
                 char *err;
9ae3a8
                 const int min = 0xFFF;
9ae3a8
+                char num[32];
9ae3a8
                 numvalue = strtoul(val, &err, 0);
9ae3a8
                 if (!*val || *err) {
9ae3a8
                     error_setg(errp, "bad numerical value %s", val);
9ae3a8
@@ -1639,7 +1680,8 @@ static void cpu_x86_parse_featurestr(X86CPU *cpu, char *features, Error **errp)
9ae3a8
                             min);
9ae3a8
                     numvalue = min;
9ae3a8
                 }
9ae3a8
-                cpu->hyperv_spinlock_attempts = numvalue;
9ae3a8
+                snprintf(num, sizeof(num), "%" PRId32, numvalue);
9ae3a8
+                object_property_parse(OBJECT(cpu), num, featurestr, errp);
9ae3a8
             } else {
9ae3a8
                 error_setg(errp, "unrecognized feature %s", featurestr);
9ae3a8
                 goto out;
9ae3a8
@@ -2588,6 +2630,7 @@ static int64_t x86_cpu_get_arch_id(CPUState *cs)
9ae3a8
 
9ae3a8
 static Property x86_cpu_properties[] = {
9ae3a8
     DEFINE_PROP_BOOL("pmu", X86CPU, enable_pmu, false),
9ae3a8
+    { .name  = "hv-spinlocks", .info  = &qdev_prop_spinlocks },
9ae3a8
     DEFINE_PROP_BOOL("hv-relaxed", X86CPU, hyperv_relaxed_timing, false),
9ae3a8
     DEFINE_PROP_BOOL("hv-vapic", X86CPU, hyperv_vapic, false),
9ae3a8
     DEFINE_PROP_END_OF_LIST()
9ae3a8
-- 
9ae3a8
1.7.1
9ae3a8