mbox series

[0/2] Fix Cooperlake CPU model

Message ID 20191225063018.20038-1-xiaoyao.li@intel.com (mailing list archive)
Headers show
Series Fix Cooperlake CPU model | expand

Message

Xiaoyao Li Dec. 25, 2019, 6:30 a.m. UTC
Current Cooperlake CPU model lacks VMX features which are introduced by Paolo
several months ago, and it also lacks 2 security features in
MSR_IA32_ARCH_CAPABILITIES disclosed recently.

Xiaoyao Li (2):
  target/i386: Add new bit definitions of MSR_IA32_ARCH_CAPABILITIES
  target/i386: Add missed features to Cooperlake CPU model

 target/i386/cpu.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++-
 target/i386/cpu.h | 13 +++++++-----
 2 files changed, 58 insertions(+), 6 deletions(-)

Comments

Paolo Bonzini Jan. 7, 2020, 1:31 p.m. UTC | #1
On 25/12/19 07:30, Xiaoyao Li wrote:
> Current Cooperlake CPU model lacks VMX features which are introduced by Paolo
> several months ago, and it also lacks 2 security features in
> MSR_IA32_ARCH_CAPABILITIES disclosed recently.
> 
> Xiaoyao Li (2):
>   target/i386: Add new bit definitions of MSR_IA32_ARCH_CAPABILITIES
>   target/i386: Add missed features to Cooperlake CPU model
> 
>  target/i386/cpu.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++-
>  target/i386/cpu.h | 13 +++++++-----
>  2 files changed, 58 insertions(+), 6 deletions(-)
> 

Queued, thanks.

Paolo
Zhang, Cathy March 16, 2020, 1:39 a.m. UTC | #2
On 1/7/2020 9:31 PM, Paolo Bonzini wrote:
> On 25/12/19 07:30, Xiaoyao Li wrote:
>> Current Cooperlake CPU model lacks VMX features which are introduced by Paolo
>> several months ago, and it also lacks 2 security features in
>> MSR_IA32_ARCH_CAPABILITIES disclosed recently.
>>
>> Xiaoyao Li (2):
>>    target/i386: Add new bit definitions of MSR_IA32_ARCH_CAPABILITIES
>>    target/i386: Add missed features to Cooperlake CPU model
>>
>>   target/i386/cpu.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++-
>>   target/i386/cpu.h | 13 +++++++-----
>>   2 files changed, 58 insertions(+), 6 deletions(-)
>>
> Queued, thanks.
>
> Paolo

Hi Paolo,

Can I ask one question that will you put all the patches for Cooper Lake 
Cpu model into QEMU v5.0-rc0?
Paolo Bonzini March 16, 2020, 8:41 a.m. UTC | #3
On 16/03/20 02:39, Zhang, Cathy wrote:
> On 1/7/2020 9:31 PM, Paolo Bonzini wrote:
>> On 25/12/19 07:30, Xiaoyao Li wrote:
>>> Current Cooperlake CPU model lacks VMX features which are introduced
>>> by Paolo
>>> several months ago, and it also lacks 2 security features in
>>> MSR_IA32_ARCH_CAPABILITIES disclosed recently.
>>>
>>> Xiaoyao Li (2):
>>>    target/i386: Add new bit definitions of MSR_IA32_ARCH_CAPABILITIES
>>>    target/i386: Add missed features to Cooperlake CPU model
>>>
>>>   target/i386/cpu.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++-
>>>   target/i386/cpu.h | 13 +++++++-----
>>>   2 files changed, 58 insertions(+), 6 deletions(-)
>>>
>> Queued, thanks.
>>
>> Paolo
> 
> Hi Paolo,
> 
> Can I ask one question that will you put all the patches for Cooper Lake
> Cpu model into QEMU v5.0-rc0?

These are included already:

commit b952544fe8a061f0c0cccfd50a58220bc6ac94da
Merge: dc65a5bdc9 083b266f69
Author: Peter Maydell <peter.maydell@linaro.org>
Date:   Fri Jan 10 17:16:49 2020 +0000

    Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging
    
    * Compat machines fix (Denis)
    * Command line parsing fixes (Michal, Peter, Xiaoyao)
    * Cooperlake CPU model fixes (Xiaoyao)
    * i386 gdb fix (mkdolata)
    * IOEventHandler cleanup (Philippe)
    * icount fix (Pavel)
    * RR support for random number sources (Pavel)
    * Kconfig fixes (Philippe)
    
Paolo
Zhang, Cathy March 16, 2020, 10:19 a.m. UTC | #4
On 3/16/2020 4:41 PM, Paolo Bonzini wrote:
> On 16/03/20 02:39, Zhang, Cathy wrote:
>> On 1/7/2020 9:31 PM, Paolo Bonzini wrote:
>>> On 25/12/19 07:30, Xiaoyao Li wrote:
>>>> Current Cooperlake CPU model lacks VMX features which are introduced
>>>> by Paolo
>>>> several months ago, and it also lacks 2 security features in
>>>> MSR_IA32_ARCH_CAPABILITIES disclosed recently.
>>>>
>>>> Xiaoyao Li (2):
>>>>     target/i386: Add new bit definitions of MSR_IA32_ARCH_CAPABILITIES
>>>>     target/i386: Add missed features to Cooperlake CPU model
>>>>
>>>>    target/i386/cpu.c | 51 ++++++++++++++++++++++++++++++++++++++++++++++-
>>>>    target/i386/cpu.h | 13 +++++++-----
>>>>    2 files changed, 58 insertions(+), 6 deletions(-)
>>>>
>>> Queued, thanks.
>>>
>>> Paolo
>> Hi Paolo,
>>
>> Can I ask one question that will you put all the patches for Cooper Lake
>> Cpu model into QEMU v5.0-rc0?
> These are included already:
>
> commit b952544fe8a061f0c0cccfd50a58220bc6ac94da
> Merge: dc65a5bdc9 083b266f69
> Author: Peter Maydell <peter.maydell@linaro.org>
> Date:   Fri Jan 10 17:16:49 2020 +0000
>
>      Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging
>      
>      * Compat machines fix (Denis)
>      * Command line parsing fixes (Michal, Peter, Xiaoyao)
>      * Cooperlake CPU model fixes (Xiaoyao)
>      * i386 gdb fix (mkdolata)
>      * IOEventHandler cleanup (Philippe)
>      * icount fix (Pavel)
>      * RR support for random number sources (Pavel)
>      * Kconfig fixes (Philippe)
>      
> Paolo
Yes, I see they are already in master, but not in v4.2 yet, so will they 
be in the next release v5.0?
Paolo Bonzini March 16, 2020, 10:24 a.m. UTC | #5
On 16/03/20 11:19, Zhang, Cathy wrote:
> Yes, I see they are already in master, but not in v4.2 yet, so will they
> be in the next release v5.0?

Yes, that's what master will become.

Paolo
Zhang, Cathy March 16, 2020, 11:43 a.m. UTC | #6
On 3/16/2020 6:24 PM, Paolo Bonzini wrote:
> On 16/03/20 11:19, Zhang, Cathy wrote:
>> Yes, I see they are already in master, but not in v4.2 yet, so will they
>> be in the next release v5.0?
> Yes, that's what master will become.
>
> Paolo
That's great! Thanks for your response!