[re-adding libvir-list]

On Wed, May 30, 2018 at 19:36:10 +0800, Dou Liyang wrote:
> Hi Peter,
> 
> Sorry to trouble you offline.

Please keep conversations on the list, so that it does not get lost in
private inboxes.

> Recently, I forced on the PCID feature in CPU and
> found the SandyBridge didn't include the PCID feature
> in libvirt.
> 
> commit cad8054ece285d1712880cd108e8a39f833f7e88
> Author: Peter Krempa <[email protected]>
> Date:   Wed Feb 8 12:43:07 2012 +0100
> 
>   cpu: Add cpu definition for Intel Sandy Bridge cpu type
> 
> I knew the SandyBridge CPU models in QEMU also
> didn't include the PCID feature.
> 
> I want to know:
> 
>  - If we can't make sure PCID was supported in all SandyBridge?
>  - If we avoided PCID definition intentionally ?
>  - If we forgot to add it, and need to be fixed?

Qemu added it later than libvirt added the sandy-bridge cpu:

commit 434acb817b8ae747f31e91ec152f9f47ac514433
Author: Mao, Junjie <[email protected]>
Date:   Fri Jul 20 07:08:21 2012 +0000

    Recognize PCID feature
    
    This patch makes Qemu recognize the PCID feature specified from 
configuration or command line options.

Jiri Denemark already covered why we can't/will not change existing
models in cpu_map.xml

> 
> Thanks,
>       dou

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to