[libvirt] [PATCH v2 18/33] cpu_x86: Introduce virCPUx86DataSetSignature

[John Ferlan]

Again would be nice to have a bit more information here.

On 02/15/2017 11:44 AM, Jiri Denemark wrote:
> Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> ---
> 
> Notes:
>     Version 2:
>     - no change
> 
>  src/cpu/cpu_x86.c            | 11 +++++++++++
>  src/cpu/cpu_x86.h            |  4 ++++
>  src/libvirt_private.syms     |  1 +
>  src/qemu/qemu_capabilities.c |  1 +
>  4 files changed, 17 insertions(+)
> 
> diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
> index 029a7d475..e3f860bc6 100644
> --- a/src/cpu/cpu_x86.c
> +++ b/src/cpu/cpu_x86.c
> @@ -2720,6 +2720,17 @@ virCPUx86DataAddCPUID(virCPUDataPtr cpuData,
>  }
>  
>  
> +int
> +virCPUx86DataSetSignature(virCPUDataPtr cpuData,
> +                          unsigned int family,
> +                          unsigned int model)
> +{
> +    uint32_t signature = x86MakeSignature(family, model);
> +
> +    return x86DataAddSignature(&cpuData->data.x86, signature);
> +}
> +
> +
>  struct cpuArchDriver cpuDriverX86 = {
>      .name = "x86",
>      .arch = archs,
> diff --git a/src/cpu/cpu_x86.h b/src/cpu/cpu_x86.h
> index 79e78a007..ffbd064b4 100644
> --- a/src/cpu/cpu_x86.h
> +++ b/src/cpu/cpu_x86.h
> @@ -32,4 +32,8 @@ extern struct cpuArchDriver cpuDriverX86;
>  int virCPUx86DataAddCPUID(virCPUDataPtr cpuData,
>                            const virCPUx86CPUID *cpuid);
>  
> +int virCPUx86DataSetSignature(virCPUDataPtr cpuData,
> +                              unsigned int family,
> +                              unsigned int model);
> +
>  #endif /* __VIR_CPU_X86_H__ */
> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> index 56db100b5..6b2dddc95 100644
> --- a/src/libvirt_private.syms
> +++ b/src/libvirt_private.syms
> @@ -1006,6 +1006,7 @@ virCPUUpdate;
>  
>  # cpu/cpu_x86.h
>  virCPUx86DataAddCPUID;
> +virCPUx86DataSetSignature;
>  
>  
>  # datatypes.h
> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
> index 6ca6ea702..f115f1e23 100644
> --- a/src/qemu/qemu_capabilities.c
> +++ b/src/qemu/qemu_capabilities.c
> @@ -33,6 +33,7 @@
>  #include "virprocess.h"
>  #include "nodeinfo.h"
>  #include "cpu/cpu.h"
> +#include "cpu/cpu_x86.h"

Is this necessary now?

ACK - would like to see a commit message though

John
>  #include "domain_conf.h"
>  #include "vircommand.h"
>  #include "virbitmap.h"
>