From e8639315f3f6116b3f589657bad57a15f119ddbc Mon Sep 17 00:00:00 2001
Message-Id: <e8639315f3f6116b3f589657bad57a15f119ddbc@dist-git>
From: Jiri Denemark <jdenemar@redhat.com>
Date: Thu, 11 Jan 2018 14:33:07 +0100
Subject: [PATCH] cpu_x86: Copy CPU signature from ancestor

When specifying a new CPU model in cpu_map.xml as an extension to an
existing model, we forgot to copy the signature (family + model) from
the original CPU model.

We don't use this way of specifying CPU models, but it's still supported
and it becomes useful when someone wants to quickly hack up a CPU model
for testing or when creating additional variants of existing models to
help with fixing some spectral issues.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
(cherry picked from commit b427cf4831d0ea7aac9dd1a3aa7682478356a483)

https://bugzilla.redhat.com/show_bug.cgi?id=1533418

The new -IBRS and -IBPB CPU models were defined via inheritance from
their original models in RHEL. Thus when the host CPU matches the
signature of the original model from cpu_map.xml, libvirt will detect
the host CPU as the original model + the new feature rather than
reporting it as the -IBRS/-IBPB variant.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 src/cpu/cpu_x86.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
index efef7f235d..41aaa61c35 100644
--- a/src/cpu/cpu_x86.c
+++ b/src/cpu/cpu_x86.c
@@ -1209,6 +1209,7 @@ x86ModelParse(xmlXPathContextPtr ctxt,
         VIR_FREE(name);
 
         model->vendor = ancestor->vendor;
+        model->signature = ancestor->signature;
         if (x86DataCopy(&model->data, &ancestor->data) < 0)
             goto error;
     }
-- 
2.15.1