|
 |
9bac43 |
From 146d96d35b69fea9311b15de60eb85a076ec2e14 Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Date: Fri, 19 Jan 2018 02:34:42 +0100
|
|
|
9bac43 |
Subject: [PATCH 14/21] spapr: Remove unnecessary 'options' field from
|
|
|
9bac43 |
sPAPRCapabilityInfo
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Message-id: <20180119023442.28577-8-dgibson@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 78673
|
|
|
9bac43 |
O-Subject: [RHEL-7.5 qemu-kvm-rhev PATCH 7/7] spapr: Remove unnecessary 'options' field from sPAPRCapabilityInfo
|
|
|
9bac43 |
Bugzilla: 1523414
|
|
|
9bac43 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
From: David Gibson <david@gibson.dropbear.id.au>
|
|
|
9bac43 |
|
|
|
9bac43 |
The options field here is intended to list the available values for the
|
|
|
9bac43 |
capability. It's not used yet, because the existing capabilities are
|
|
|
9bac43 |
boolean.
|
|
|
9bac43 |
|
|
|
9bac43 |
We're going to add capabilities that aren't, but in that case the info on
|
|
|
9bac43 |
the possible values can be folded into the .description field.
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
|
|
|
9bac43 |
(cherry picked from commit 895d5cd620d358c2623c639a150d9320d581b4f8)
|
|
|
9bac43 |
|
|
|
9bac43 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1523414
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
hw/ppc/spapr_caps.c | 6 +-----
|
|
|
9bac43 |
1 file changed, 1 insertion(+), 5 deletions(-)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/hw/ppc/spapr_caps.c b/hw/ppc/spapr_caps.c
|
|
|
9bac43 |
index f95a785..d5c9ce7 100644
|
|
|
9bac43 |
--- a/hw/ppc/spapr_caps.c
|
|
|
9bac43 |
+++ b/hw/ppc/spapr_caps.c
|
|
|
9bac43 |
@@ -35,7 +35,6 @@
|
|
|
9bac43 |
typedef struct sPAPRCapabilityInfo {
|
|
|
9bac43 |
const char *name;
|
|
|
9bac43 |
const char *description;
|
|
|
9bac43 |
- const char *options; /* valid capability values */
|
|
|
9bac43 |
int index;
|
|
|
9bac43 |
|
|
|
9bac43 |
/* Getter and Setter Function Pointers */
|
|
|
9bac43 |
@@ -126,7 +125,6 @@ sPAPRCapabilityInfo capability_table[SPAPR_CAP_NUM] = {
|
|
|
9bac43 |
[SPAPR_CAP_HTM] = {
|
|
|
9bac43 |
.name = "htm",
|
|
|
9bac43 |
.description = "Allow Hardware Transactional Memory (HTM)",
|
|
|
9bac43 |
- .options = "",
|
|
|
9bac43 |
.index = SPAPR_CAP_HTM,
|
|
|
9bac43 |
.get = spapr_cap_get_bool,
|
|
|
9bac43 |
.set = spapr_cap_set_bool,
|
|
|
9bac43 |
@@ -136,7 +134,6 @@ sPAPRCapabilityInfo capability_table[SPAPR_CAP_NUM] = {
|
|
|
9bac43 |
[SPAPR_CAP_VSX] = {
|
|
|
9bac43 |
.name = "vsx",
|
|
|
9bac43 |
.description = "Allow Vector Scalar Extensions (VSX)",
|
|
|
9bac43 |
- .options = "",
|
|
|
9bac43 |
.index = SPAPR_CAP_VSX,
|
|
|
9bac43 |
.get = spapr_cap_get_bool,
|
|
|
9bac43 |
.set = spapr_cap_set_bool,
|
|
|
9bac43 |
@@ -146,7 +143,6 @@ sPAPRCapabilityInfo capability_table[SPAPR_CAP_NUM] = {
|
|
|
9bac43 |
[SPAPR_CAP_DFP] = {
|
|
|
9bac43 |
.name = "dfp",
|
|
|
9bac43 |
.description = "Allow Decimal Floating Point (DFP)",
|
|
|
9bac43 |
- .options = "",
|
|
|
9bac43 |
.index = SPAPR_CAP_DFP,
|
|
|
9bac43 |
.get = spapr_cap_get_bool,
|
|
|
9bac43 |
.set = spapr_cap_set_bool,
|
|
|
9bac43 |
@@ -338,7 +334,7 @@ void spapr_caps_add_properties(sPAPRMachineClass *smc, Error **errp)
|
|
|
9bac43 |
return;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
- desc = g_strdup_printf("%s%s", cap->description, cap->options);
|
|
|
9bac43 |
+ desc = g_strdup_printf("%s", cap->description);
|
|
|
9bac43 |
object_class_property_set_description(klass, name, desc, &local_err);
|
|
|
9bac43 |
g_free(desc);
|
|
|
9bac43 |
if (local_err) {
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|