[libvirt] [PATCH 4/5] hostcpu: add support for reporting die_id in NUMA topology

Daniel Henrique Barboza danielhb413 at gmail.com
Tue Jan 7 17:41:12 UTC 2020



On 12/19/19 9:42 AM, Daniel P. Berrangé wrote:
> Update the host CPU code to report the die_id in the NUMA topology
> capabilities. On systems with mulitple dies, this fixes the bug

s/mulitple/multiple

> where CPU cores can't be distinguished:
> 
>   <cpus num='12'>
>     <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
>     <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
>     <cpu id='2' socket_id='0' core_id='0' siblings='2'/>
>     <cpu id='3' socket_id='0' core_id='1' siblings='3'/>
>   </cpus>
> 
> Notes core_id is repeated within the scope of the socket.

Not sure what you meant by 'Notes code_id'. 'Note that code_id ...' perhaps?




> 
> It now reports
> 
>   <cpus num='12'>
>     <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
>     <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
>     <cpu id='2' socket_id='0' die_id='1' core_id='0' siblings='2'/>
>     <cpu id='3' socket_id='0' die_id='1' core_id='1' siblings='3'/>
>   </cpus>
> 
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
>   docs/schemas/capability.rng                   |  3 ++
>   src/conf/capabilities.c                       |  5 ++-
>   src/conf/capabilities.h                       |  1 +
>   src/libvirt_linux.syms                        |  1 +
>   src/util/virhostcpu.c                         | 16 ++++++++++
>   src/util/virhostcpu.h                         |  1 +
>   .../vircaps2xmldata/vircaps-aarch64-basic.xml | 32 +++++++++----------
>   .../vircaps2xmldata/vircaps-x86_64-basic.xml  | 32 +++++++++----------
>   .../vircaps2xmldata/vircaps-x86_64-caches.xml | 16 +++++-----
>   .../vircaps-x86_64-resctrl-cdp.xml            | 24 +++++++-------
>   .../vircaps-x86_64-resctrl-cmt.xml            | 24 +++++++-------
>   .../vircaps-x86_64-resctrl-fake-feature.xml   | 24 +++++++-------
>   .../vircaps-x86_64-resctrl-skx-twocaches.xml  |  2 +-
>   .../vircaps-x86_64-resctrl-skx.xml            |  2 +-
>   .../vircaps-x86_64-resctrl.xml                | 24 +++++++-------
>   15 files changed, 116 insertions(+), 91 deletions(-)
> 
> diff --git a/docs/schemas/capability.rng b/docs/schemas/capability.rng
> index 26d313d652..165f704ef3 100644
> --- a/docs/schemas/capability.rng
> +++ b/docs/schemas/capability.rng
> @@ -265,6 +265,9 @@
>           <attribute name='socket_id'>
>             <ref name='unsignedInt'/>
>           </attribute>
> +        <attribute name='die_id'>
> +          <ref name='unsignedInt'/>
> +        </attribute>
>           <attribute name='core_id'>
>             <ref name='unsignedInt'/>
>           </attribute>
> diff --git a/src/conf/capabilities.c b/src/conf/capabilities.c
> index e54e46f54e..2324a7cb8a 100644
> --- a/src/conf/capabilities.c
> +++ b/src/conf/capabilities.c
> @@ -885,8 +885,9 @@ virCapabilitiesHostNUMAFormat(virCapsHostNUMAPtr caps,
>                       return -1;
>   
>                   virBufferAsprintf(buf,
> -                                  " socket_id='%d' core_id='%d' siblings='%s'",
> +                                  " socket_id='%d' die_id='%d' core_id='%d' siblings='%s'",
>                                     cell->cpus[j].socket_id,
> +                                  cell->cpus[j].die_id,
>                                     cell->cpus[j].core_id,
>                                     siblings);
>                   VIR_FREE(siblings);
> @@ -1477,6 +1478,7 @@ virCapabilitiesFillCPUInfo(int cpu_id G_GNUC_UNUSED,
>       cpu->id = cpu_id;
>   
>       if (virHostCPUGetSocket(cpu_id, &cpu->socket_id) < 0 ||
> +        virHostCPUGetDie(cpu_id, &cpu->die_id) < 0 ||
>           virHostCPUGetCore(cpu_id, &cpu->core_id) < 0)
>           return -1;
>   
> @@ -1605,6 +1607,7 @@ virCapabilitiesHostNUMAInitFake(virCapsHostNUMAPtr caps)
>                           goto error;
>                       if (tmp) {
>                           cpus[cid].id = id;
> +                        cpus[cid].die_id = 0;
>                           cpus[cid].socket_id = s;
>                           cpus[cid].core_id = c;
>                           if (!(cpus[cid].siblings = virBitmapNew(ncpus)))
> diff --git a/src/conf/capabilities.h b/src/conf/capabilities.h
> index 4a49e94aa5..75f29666c9 100644
> --- a/src/conf/capabilities.h
> +++ b/src/conf/capabilities.h
> @@ -88,6 +88,7 @@ struct _virCapsGuest {
>   struct _virCapsHostNUMACellCPU {
>       unsigned int id;
>       unsigned int socket_id;
> +    unsigned int die_id;
>       unsigned int core_id;
>       virBitmapPtr siblings;
>   };
> diff --git a/src/libvirt_linux.syms b/src/libvirt_linux.syms
> index 5fa2c790ef..55649ae39c 100644
> --- a/src/libvirt_linux.syms
> +++ b/src/libvirt_linux.syms
> @@ -4,6 +4,7 @@
>   
>   # util/virhostcpu.h
>   virHostCPUGetCore;
> +virHostCPUGetDie;
>   virHostCPUGetInfoPopulateLinux;
>   virHostCPUGetSiblingsList;
>   virHostCPUGetSocket;
> diff --git a/src/util/virhostcpu.c b/src/util/virhostcpu.c
> index 22102f2c75..0c174702a4 100644
> --- a/src/util/virhostcpu.c
> +++ b/src/util/virhostcpu.c
> @@ -218,6 +218,22 @@ virHostCPUGetSocket(unsigned int cpu, unsigned int *socket)
>       return 0;
>   }
>   
> +int
> +virHostCPUGetDie(unsigned int cpu, unsigned int *die)
> +{
> +    int ret = virFileReadValueUint(die,
> +                                   "%s/cpu/cpu%u/topology/die_id",
> +                                   SYSFS_SYSTEM_PATH, cpu);
> +
> +    /* If the file is not there, it's 0 */
> +    if (ret == -2)
> +        *die = 0;
> +    else if (ret < 0)
> +        return -1;
> +
> +    return 0;
> +}
> +
>   int
>   virHostCPUGetCore(unsigned int cpu, unsigned int *core)
>   {
> diff --git a/src/util/virhostcpu.h b/src/util/virhostcpu.h
> index d95d380d4a..9be2e51a38 100644
> --- a/src/util/virhostcpu.h
> +++ b/src/util/virhostcpu.h
> @@ -65,6 +65,7 @@ int virHostCPUStatsAssign(virNodeCPUStatsPtr param,
>   
>   #ifdef __linux__
>   int virHostCPUGetSocket(unsigned int cpu, unsigned int *socket);
> +int virHostCPUGetDie(unsigned int cpu, unsigned int *die);
>   int virHostCPUGetCore(unsigned int cpu, unsigned int *core);
>   
>   virBitmapPtr virHostCPUGetSiblingsList(unsigned int cpu);
> diff --git a/tests/vircaps2xmldata/vircaps-aarch64-basic.xml b/tests/vircaps2xmldata/vircaps-aarch64-basic.xml
> index 50466f9162..0a04052c40 100644
> --- a/tests/vircaps2xmldata/vircaps-aarch64-basic.xml
> +++ b/tests/vircaps2xmldata/vircaps-aarch64-basic.xml
> @@ -16,10 +16,10 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='4'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -28,10 +28,10 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='4'>
> -            <cpu id='4' socket_id='1' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='1' core_id='5' siblings='5'/>
> -            <cpu id='6' socket_id='1' core_id='6' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='7' siblings='7'/>
> +            <cpu id='4' socket_id='1' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='1' die_id='0' core_id='5' siblings='5'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='6' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='7' siblings='7'/>
>             </cpus>
>           </cell>
>           <cell id='2'>
> @@ -40,10 +40,10 @@
>             <pages unit='KiB' size='2048'>8192</pages>
>             <pages unit='KiB' size='1048576'>10240</pages>
>             <cpus num='4'>
> -            <cpu id='8' socket_id='2' core_id='8' siblings='8'/>
> -            <cpu id='9' socket_id='2' core_id='9' siblings='9'/>
> -            <cpu id='10' socket_id='2' core_id='10' siblings='10'/>
> -            <cpu id='11' socket_id='2' core_id='11' siblings='11'/>
> +            <cpu id='8' socket_id='2' die_id='0' core_id='8' siblings='8'/>
> +            <cpu id='9' socket_id='2' die_id='0' core_id='9' siblings='9'/>
> +            <cpu id='10' socket_id='2' die_id='0' core_id='10' siblings='10'/>
> +            <cpu id='11' socket_id='2' die_id='0' core_id='11' siblings='11'/>
>             </cpus>
>           </cell>
>           <cell id='3'>
> @@ -52,10 +52,10 @@
>             <pages unit='KiB' size='2048'>10240</pages>
>             <pages unit='KiB' size='1048576'>12288</pages>
>             <cpus num='4'>
> -            <cpu id='12' socket_id='3' core_id='12' siblings='12'/>
> -            <cpu id='13' socket_id='3' core_id='13' siblings='13'/>
> -            <cpu id='14' socket_id='3' core_id='14' siblings='14'/>
> -            <cpu id='15' socket_id='3' core_id='15' siblings='15'/>
> +            <cpu id='12' socket_id='3' die_id='0' core_id='12' siblings='12'/>
> +            <cpu id='13' socket_id='3' die_id='0' core_id='13' siblings='13'/>
> +            <cpu id='14' socket_id='3' die_id='0' core_id='14' siblings='14'/>
> +            <cpu id='15' socket_id='3' die_id='0' core_id='15' siblings='15'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-basic.xml b/tests/vircaps2xmldata/vircaps-x86_64-basic.xml
> index e7be6def3e..4da09f889c 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-basic.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-basic.xml
> @@ -14,10 +14,10 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='4'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -26,10 +26,10 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='4'>
> -            <cpu id='4' socket_id='1' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='1' core_id='5' siblings='5'/>
> -            <cpu id='6' socket_id='1' core_id='6' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='7' siblings='7'/>
> +            <cpu id='4' socket_id='1' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='1' die_id='0' core_id='5' siblings='5'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='6' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='7' siblings='7'/>
>             </cpus>
>           </cell>
>           <cell id='2'>
> @@ -38,10 +38,10 @@
>             <pages unit='KiB' size='2048'>8192</pages>
>             <pages unit='KiB' size='1048576'>10240</pages>
>             <cpus num='4'>
> -            <cpu id='8' socket_id='2' core_id='8' siblings='8'/>
> -            <cpu id='9' socket_id='2' core_id='9' siblings='9'/>
> -            <cpu id='10' socket_id='2' core_id='10' siblings='10'/>
> -            <cpu id='11' socket_id='2' core_id='11' siblings='11'/>
> +            <cpu id='8' socket_id='2' die_id='0' core_id='8' siblings='8'/>
> +            <cpu id='9' socket_id='2' die_id='0' core_id='9' siblings='9'/>
> +            <cpu id='10' socket_id='2' die_id='0' core_id='10' siblings='10'/>
> +            <cpu id='11' socket_id='2' die_id='0' core_id='11' siblings='11'/>
>             </cpus>
>           </cell>
>           <cell id='3'>
> @@ -50,10 +50,10 @@
>             <pages unit='KiB' size='2048'>10240</pages>
>             <pages unit='KiB' size='1048576'>12288</pages>
>             <cpus num='4'>
> -            <cpu id='12' socket_id='3' core_id='12' siblings='12'/>
> -            <cpu id='13' socket_id='3' core_id='13' siblings='13'/>
> -            <cpu id='14' socket_id='3' core_id='14' siblings='14'/>
> -            <cpu id='15' socket_id='3' core_id='15' siblings='15'/>
> +            <cpu id='12' socket_id='3' die_id='0' core_id='12' siblings='12'/>
> +            <cpu id='13' socket_id='3' die_id='0' core_id='13' siblings='13'/>
> +            <cpu id='14' socket_id='3' die_id='0' core_id='14' siblings='14'/>
> +            <cpu id='15' socket_id='3' die_id='0' core_id='15' siblings='15'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-caches.xml b/tests/vircaps2xmldata/vircaps-x86_64-caches.xml
> index ca671a1640..28f00c0a90 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-caches.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-caches.xml
> @@ -17,14 +17,14 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='8'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0,4'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1,5'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2,6'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3,7'/>
> -            <cpu id='4' socket_id='0' core_id='0' siblings='0,4'/>
> -            <cpu id='5' socket_id='0' core_id='1' siblings='1,5'/>
> -            <cpu id='6' socket_id='0' core_id='2' siblings='2,6'/>
> -            <cpu id='7' socket_id='0' core_id='3' siblings='3,7'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0,4'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1,5'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2,6'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3,7'/>
> +            <cpu id='4' socket_id='0' die_id='0' core_id='0' siblings='0,4'/>
> +            <cpu id='5' socket_id='0' die_id='0' core_id='1' siblings='1,5'/>
> +            <cpu id='6' socket_id='0' die_id='0' core_id='2' siblings='2,6'/>
> +            <cpu id='7' socket_id='0' die_id='0' core_id='3' siblings='3,7'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cdp.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cdp.xml
> index 1d3df318c5..ee26fe9464 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cdp.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cdp.xml
> @@ -17,12 +17,12 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='6'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> -            <cpu id='4' socket_id='0' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='0' core_id='5' siblings='5'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
> +            <cpu id='4' socket_id='0' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='0' die_id='0' core_id='5' siblings='5'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -31,12 +31,12 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='6'>
> -            <cpu id='6' socket_id='1' core_id='0' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='1' siblings='7'/>
> -            <cpu id='8' socket_id='1' core_id='2' siblings='8'/>
> -            <cpu id='9' socket_id='1' core_id='3' siblings='9'/>
> -            <cpu id='10' socket_id='1' core_id='4' siblings='10'/>
> -            <cpu id='11' socket_id='1' core_id='5' siblings='11'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='0' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='1' siblings='7'/>
> +            <cpu id='8' socket_id='1' die_id='0' core_id='2' siblings='8'/>
> +            <cpu id='9' socket_id='1' die_id='0' core_id='3' siblings='9'/>
> +            <cpu id='10' socket_id='1' die_id='0' core_id='4' siblings='10'/>
> +            <cpu id='11' socket_id='1' die_id='0' core_id='5' siblings='11'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cmt.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cmt.xml
> index 6a8cd0e909..acdd97ec58 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cmt.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-cmt.xml
> @@ -17,12 +17,12 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='6'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> -            <cpu id='4' socket_id='0' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='0' core_id='5' siblings='5'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
> +            <cpu id='4' socket_id='0' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='0' die_id='0' core_id='5' siblings='5'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -31,12 +31,12 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='6'>
> -            <cpu id='6' socket_id='1' core_id='0' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='1' siblings='7'/>
> -            <cpu id='8' socket_id='1' core_id='2' siblings='8'/>
> -            <cpu id='9' socket_id='1' core_id='3' siblings='9'/>
> -            <cpu id='10' socket_id='1' core_id='4' siblings='10'/>
> -            <cpu id='11' socket_id='1' core_id='5' siblings='11'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='0' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='1' siblings='7'/>
> +            <cpu id='8' socket_id='1' die_id='0' core_id='2' siblings='8'/>
> +            <cpu id='9' socket_id='1' die_id='0' core_id='3' siblings='9'/>
> +            <cpu id='10' socket_id='1' die_id='0' core_id='4' siblings='10'/>
> +            <cpu id='11' socket_id='1' die_id='0' core_id='5' siblings='11'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-fake-feature.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-fake-feature.xml
> index 4e46ead616..5f3678e072 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-fake-feature.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-fake-feature.xml
> @@ -17,12 +17,12 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='6'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> -            <cpu id='4' socket_id='0' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='0' core_id='5' siblings='5'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
> +            <cpu id='4' socket_id='0' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='0' die_id='0' core_id='5' siblings='5'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -31,12 +31,12 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='6'>
> -            <cpu id='6' socket_id='1' core_id='0' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='1' siblings='7'/>
> -            <cpu id='8' socket_id='1' core_id='2' siblings='8'/>
> -            <cpu id='9' socket_id='1' core_id='3' siblings='9'/>
> -            <cpu id='10' socket_id='1' core_id='4' siblings='10'/>
> -            <cpu id='11' socket_id='1' core_id='5' siblings='11'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='0' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='1' siblings='7'/>
> +            <cpu id='8' socket_id='1' die_id='0' core_id='2' siblings='8'/>
> +            <cpu id='9' socket_id='1' die_id='0' core_id='3' siblings='9'/>
> +            <cpu id='10' socket_id='1' die_id='0' core_id='4' siblings='10'/>
> +            <cpu id='11' socket_id='1' die_id='0' core_id='5' siblings='11'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx-twocaches.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx-twocaches.xml
> index 44c1042afe..6769bd0591 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx-twocaches.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx-twocaches.xml
> @@ -17,7 +17,7 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='1'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx.xml
> index 8382a26c7a..bc52480905 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl-skx.xml
> @@ -17,7 +17,7 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='1'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
>             </cpus>
>           </cell>
>         </cells>
> diff --git a/tests/vircaps2xmldata/vircaps-x86_64-resctrl.xml b/tests/vircaps2xmldata/vircaps-x86_64-resctrl.xml
> index a27b3e247e..c386edd4b0 100644
> --- a/tests/vircaps2xmldata/vircaps-x86_64-resctrl.xml
> +++ b/tests/vircaps2xmldata/vircaps-x86_64-resctrl.xml
> @@ -17,12 +17,12 @@
>             <pages unit='KiB' size='2048'>4096</pages>
>             <pages unit='KiB' size='1048576'>6144</pages>
>             <cpus num='6'>
> -            <cpu id='0' socket_id='0' core_id='0' siblings='0'/>
> -            <cpu id='1' socket_id='0' core_id='1' siblings='1'/>
> -            <cpu id='2' socket_id='0' core_id='2' siblings='2'/>
> -            <cpu id='3' socket_id='0' core_id='3' siblings='3'/>
> -            <cpu id='4' socket_id='0' core_id='4' siblings='4'/>
> -            <cpu id='5' socket_id='0' core_id='5' siblings='5'/>
> +            <cpu id='0' socket_id='0' die_id='0' core_id='0' siblings='0'/>
> +            <cpu id='1' socket_id='0' die_id='0' core_id='1' siblings='1'/>
> +            <cpu id='2' socket_id='0' die_id='0' core_id='2' siblings='2'/>
> +            <cpu id='3' socket_id='0' die_id='0' core_id='3' siblings='3'/>
> +            <cpu id='4' socket_id='0' die_id='0' core_id='4' siblings='4'/>
> +            <cpu id='5' socket_id='0' die_id='0' core_id='5' siblings='5'/>
>             </cpus>
>           </cell>
>           <cell id='1'>
> @@ -31,12 +31,12 @@
>             <pages unit='KiB' size='2048'>6144</pages>
>             <pages unit='KiB' size='1048576'>8192</pages>
>             <cpus num='6'>
> -            <cpu id='6' socket_id='1' core_id='0' siblings='6'/>
> -            <cpu id='7' socket_id='1' core_id='1' siblings='7'/>
> -            <cpu id='8' socket_id='1' core_id='2' siblings='8'/>
> -            <cpu id='9' socket_id='1' core_id='3' siblings='9'/>
> -            <cpu id='10' socket_id='1' core_id='4' siblings='10'/>
> -            <cpu id='11' socket_id='1' core_id='5' siblings='11'/>
> +            <cpu id='6' socket_id='1' die_id='0' core_id='0' siblings='6'/>
> +            <cpu id='7' socket_id='1' die_id='0' core_id='1' siblings='7'/>
> +            <cpu id='8' socket_id='1' die_id='0' core_id='2' siblings='8'/>
> +            <cpu id='9' socket_id='1' die_id='0' core_id='3' siblings='9'/>
> +            <cpu id='10' socket_id='1' die_id='0' core_id='4' siblings='10'/>
> +            <cpu id='11' socket_id='1' die_id='0' core_id='5' siblings='11'/>
>             </cpus>
>           </cell>
>         </cells>
> 





More information about the libvir-list mailing list