From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=50098 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1OOu4q-00080Z-2M
	for qemu-devel@nongnu.org; Wed, 16 Jun 2010 10:58:53 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <sheng@linux.intel.com>) id 1OOu4o-0007mP-Qa
	for qemu-devel@nongnu.org; Wed, 16 Jun 2010 10:58:51 -0400
Received: from mga03.intel.com ([143.182.124.21]:54384)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <sheng@linux.intel.com>) id 1OOu4o-0007lv-IR
	for qemu-devel@nongnu.org; Wed, 16 Jun 2010 10:58:50 -0400
From: Sheng Yang <sheng@linux.intel.com>
Date: Wed, 16 Jun 2010 22:59:07 +0800
References: <1276140663-29954-1-git-send-email-sheng@linux.intel.com>
	<1276140663-29954-2-git-send-email-sheng@linux.intel.com>
In-Reply-To: <1276140663-29954-2-git-send-email-sheng@linux.intel.com>
MIME-Version: 1.0
Content-Type: Text/Plain;
  charset="us-ascii"
Content-Transfer-Encoding: 7bit
Message-Id: <201006162259.08042.sheng@linux.intel.com>
Subject: [Qemu-devel] Re: [PATCH 2/3] qemu: Enable XSAVE related CPUID
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Marcelo Tosatti <mtosatti@redhat.com>
Cc: qemu-devel@nongnu.org, Avi Kivity <avi@redhat.com>, kvm@vger.kernel.org

On Thursday 10 June 2010 11:31:02 Sheng Yang wrote:
> We can support it in KVM now. The 0xd leaf is queried from KVM.

Hi Marcelo & Avi

How about patch 1 and 2 in this series? They are used to enable XSAVE cpuid.

--
regards
Yang, Sheng
					     						
> 
> Signed-off-by: Sheng Yang <sheng@linux.intel.com>
> ---
>  target-i386/cpuid.c |   21 +++++++++++++++++++++
>  1 files changed, 21 insertions(+), 0 deletions(-)
> 
> diff --git a/target-i386/cpuid.c b/target-i386/cpuid.c
> index fb78061..26e4054 100644
> --- a/target-i386/cpuid.c
> +++ b/target-i386/cpuid.c
> @@ -1081,6 +1081,27 @@ void cpu_x86_cpuid(CPUX86State *env, uint32_t index,
> uint32_t count, *ecx = 0;
>          *edx = 0;
>          break;
> +    case 0xD:
> +        /* Processor Extended State */
> +        if (!(env->cpuid_ext_features & CPUID_EXT_XSAVE)) {
> +            *eax = 0;
> +            *ebx = 0;
> +            *ecx = 0;
> +            *edx = 0;
> +            break;
> +        }
> +        if (kvm_enabled()) {
> +            *eax = kvm_arch_get_supported_cpuid(env, 0xd, count, R_EAX);
> +            *ebx = kvm_arch_get_supported_cpuid(env, 0xd, count, R_EBX);
> +            *ecx = kvm_arch_get_supported_cpuid(env, 0xd, count, R_ECX);
> +            *edx = kvm_arch_get_supported_cpuid(env, 0xd, count, R_EDX);
> +        } else {
> +            *eax = 0;
> +            *ebx = 0;
> +            *ecx = 0;
> +            *edx = 0;
> +        }
> +        break;
>      case 0x80000000:
>          *eax = env->cpuid_xlevel;
>          *ebx = env->cpuid_vendor1;