[libvirt] [PATCH v2 15/30] cpu_x86: Log decoded CPU model and signatures
Ján Tomko
jtomko at redhat.com
Tue Mar 5 11:35:48 UTC 2019
On Tue, Mar 05, 2019 at 10:40:31AM +0100, Jiri Denemark wrote:
>The log message may be useful when debugging why a specific CPU model
>was selected for a given set of CPUID data.
>
>Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
>---
>
>Notes:
> Version 2:
> - separated from 11/26 cpu_x86: Allow multiple signatures for a CPU model
> - signature formatting code moved into a dedicated function
>
> src/cpu/cpu_x86.c | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
>diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
>index 08677ef7ff..5a1071de4d 100644
>--- a/src/cpu/cpu_x86.c
>+++ b/src/cpu/cpu_x86.c
>@@ -1773,6 +1773,26 @@ x86ModelHasSignature(virCPUx86ModelPtr model,
> }
>
>
>+static char *
>+x86FormatSignatures(virCPUx86ModelPtr model)
>+{
>+ virBuffer buf = VIR_BUFFER_INITIALIZER;
>+ size_t i;
>+
>+ for (i = 0; i < model->nsignatures; i++) {
>+ virBufferAsprintf(&buf, "%06lx,",
>+ (unsigned long)model->signatures[i]);
>+ }
>+
>+ virBufferTrim(&buf, ",", -1);
>+
>+ if (virBufferCheckError(&buf) < 0)
>+ return NULL;
>+
>+ return virBufferContentAndReset(&buf);
>+}
>+
>+
> /*
> * Checks whether a candidate model is a better fit for the CPU data than the
> * current model.
>@@ -1896,6 +1916,7 @@ x86Decode(virCPUDefPtr cpu,
> virCPUx86Data features = VIR_CPU_X86_DATA_INIT;
> virCPUx86VendorPtr vendor;
> virDomainCapsCPUModelPtr hvModel = NULL;
>+ char *sigs = NULL;
VIR_AUTOFREE(char *)
> uint32_t signature;
> ssize_t i;
> int rc;
>@@ -1988,6 +2009,11 @@ x86Decode(virCPUDefPtr cpu,
> if (vendor && VIR_STRDUP(cpu->vendor, vendor->name) < 0)
> goto cleanup;
>
>+ sigs = x86FormatSignatures(model);
>+
>+ VIR_DEBUG("Using CPU model %s (signatures %s) for CPU with signature %06lx",
>+ model->name, NULLSTR(sigs), (unsigned long)signature);
>+
> VIR_STEAL_PTR(cpu->model, cpuModel->model);
> VIR_STEAL_PTR(cpu->features, cpuModel->features);
> cpu->nfeatures = cpuModel->nfeatures;
Reviewed-by: Ján Tomko <jtomko at redhat.com>
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190305/e5682e49/attachment-0001.sig>
More information about the libvir-list
mailing list