diff mbox

qemu kvm: Recognize PCID feature

Message ID EF5A1D57CFBD5A4BA5EB3ED985B6DC6E0AC47E@SHSMSX101.ccr.corp.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mao, Junjie July 13, 2012, 4:57 a.m. UTC
This patch makes Qemu recognize the PCID feature specified from configuration or command line options.

Signed-off-by: Junjie Mao <junjie.mao@intel.com>
---
 target-i386/cpu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Mao, Junjie July 18, 2012, 8:44 a.m. UTC | #1
Hi, Avi

Any comments on this patch? :)

> -----Original Message-----
> From: kvm-owner@vger.kernel.org [mailto:kvm-owner@vger.kernel.org] On
> Behalf Of Mao, Junjie
> Sent: Friday, July 13, 2012 12:58 PM
> To: 'kvm@vger.kernel.org'
> Subject: [PATCH] qemu kvm: Recognize PCID feature
> 
> This patch makes Qemu recognize the PCID feature specified from
> configuration or command line options.
> 
> Signed-off-by: Junjie Mao <junjie.mao@intel.com>
> ---
>  target-i386/cpu.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 5521709..efc6ece
> 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -50,7 +50,7 @@ static const char *ext_feature_name[] = {
>      "ds_cpl", "vmx", "smx", "est",
>      "tm2", "ssse3", "cid", NULL,
>      "fma", "cx16", "xtpr", "pdcm",
> -    NULL, NULL, "dca", "sse4.1|sse4_1",
> +    NULL, "pcid", "dca", "sse4.1|sse4_1",
>      "sse4.2|sse4_2", "x2apic", "movbe", "popcnt",
>      "tsc-deadline", "aes", "xsave", "osxsave",
>      "avx", NULL, NULL, "hypervisor",
> --
> 1.7.1
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a
> message to majordomo@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jan Kiszka July 18, 2012, 9:29 a.m. UTC | #2
On 2012-07-18 10:44, Mao, Junjie wrote:
> Hi, Avi
> 
> Any comments on this patch? :)

Always include qemu-devel when your are changing QEMU, qemu-kvm is just
staging for the latter. This patch can actually go into upstream
directly, maybe even via qemu-trivial as it just makes that flag selectable.

Jan

> 
>> -----Original Message-----
>> From: kvm-owner@vger.kernel.org [mailto:kvm-owner@vger.kernel.org] On
>> Behalf Of Mao, Junjie
>> Sent: Friday, July 13, 2012 12:58 PM
>> To: 'kvm@vger.kernel.org'
>> Subject: [PATCH] qemu kvm: Recognize PCID feature
>>
>> This patch makes Qemu recognize the PCID feature specified from
>> configuration or command line options.
>>
>> Signed-off-by: Junjie Mao <junjie.mao@intel.com>
>> ---
>>  target-i386/cpu.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 5521709..efc6ece
>> 100644
>> --- a/target-i386/cpu.c
>> +++ b/target-i386/cpu.c
>> @@ -50,7 +50,7 @@ static const char *ext_feature_name[] = {
>>      "ds_cpl", "vmx", "smx", "est",
>>      "tm2", "ssse3", "cid", NULL,
>>      "fma", "cx16", "xtpr", "pdcm",
>> -    NULL, NULL, "dca", "sse4.1|sse4_1",
>> +    NULL, "pcid", "dca", "sse4.1|sse4_1",
>>      "sse4.2|sse4_2", "x2apic", "movbe", "popcnt",
>>      "tsc-deadline", "aes", "xsave", "osxsave",
>>      "avx", NULL, NULL, "hypervisor",
>> --
>> 1.7.1
>> --
>> To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a
>> message to majordomo@vger.kernel.org More majordomo info at
>> http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
Marcelo Tosatti July 18, 2012, 8:23 p.m. UTC | #3
On Wed, Jul 18, 2012 at 11:29:36AM +0200, Jan Kiszka wrote:
> On 2012-07-18 10:44, Mao, Junjie wrote:
> > Hi, Avi
> > 
> > Any comments on this patch? :)
> 
> Always include qemu-devel when your are changing QEMU, qemu-kvm is just
> staging for the latter. This patch can actually go into upstream
> directly, maybe even via qemu-trivial as it just makes that flag selectable.
> 
> Jan

Agreed, CCing Stefan.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 5521709..efc6ece 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -50,7 +50,7 @@  static const char *ext_feature_name[] = {
     "ds_cpl", "vmx", "smx", "est",
     "tm2", "ssse3", "cid", NULL,
     "fma", "cx16", "xtpr", "pdcm",
-    NULL, NULL, "dca", "sse4.1|sse4_1",
+    NULL, "pcid", "dca", "sse4.1|sse4_1",
     "sse4.2|sse4_2", "x2apic", "movbe", "popcnt",
     "tsc-deadline", "aes", "xsave", "osxsave",
     "avx", NULL, NULL, "hypervisor",