Thanks for Peter's reply. Since it is better to pull the patch by x86
maintainers, could any maintainer help to merge it, @Paolo Bonzini or @Stefan
Hajnoczi? The original patch is attached below. Thanks a lot.
Best Regards,
Wenchao
---------------------------------
From b1789f2523d06798b8883664bfa9a9df797bfccf Mon Sep 17 00:00:00 2001
From: Wenchao Wang <wenchao.w...@intel.com>
Date: Fri, 25 Nov 2022 18:37:34 +0800
Subject: [PATCH] target/i386/hax: Add XCR0 support
Introduce extended control register XCR0 to support XSAVE feature set.
Note: This change requires at least HAXM v7.8.0 to support.
Reviewed-by: Hang Yuan <hang.y...@intel.com>
Signed-off-by: Wenchao Wang <wenchao.w...@intel.com>
---
target/i386/hax/hax-interface.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/i386/hax/hax-interface.h b/target/i386/hax/hax-interface.h
index 537ae084e9..1d13bb2380 100644
--- a/target/i386/hax/hax-interface.h
+++ b/target/i386/hax/hax-interface.h
@@ -201,6 +201,8 @@ struct vcpu_state_t {
uint64_t _cr3;
uint64_t _cr4;
+ uint64_t _xcr0;
+
uint64_t _dr0;
uint64_t _dr1;
uint64_t _dr2;
--
2.17.1
-----Original Message-----
From: Peter Maydell <peter.mayd...@linaro.org>
Sent: Thursday, December 15, 2022 18:27
To: Wang, Wenchao <wenchao.w...@intel.com>
Cc: Thomas Huth <th...@redhat.com>; Paolo Bonzini <pbonz...@redhat.com>; Stefan
Hajnoczi <stefa...@redhat.com>; qemu-devel@nongnu.org; Richard Henderson
<richard.hender...@linaro.org>
Subject: Re: [PATCH] target/i386/hax: Add XCR0 support
On Thu, 15 Dec 2022 at 09:45, Wang, Wenchao <wenchao.w...@intel.com> wrote:
>
> Hi, Thomas,
>
> Thanks for your reply. I have attempted to follow you suggestions but it
> always failed on tagging a GPG-signed tag before submitting the pull request.
> I have used GPG 2.2.4 to generate a RSA4096 GPG secret key and pasted the
> public key on GitHub successfully.
>
> $ git tag -s pull-request-hax -m 'target/i386/hax: Add XCR0 support'
> error: gpg failed to sign the data
> error: unable to sign the tag
>
> Meanwhile, could @Paolo Bonzini or @Stefan Hajnoczi help to pick the patch up
> as there is only one-line change for HAX and we have verified it for all
> guest launching? Thanks a lot.
Yes, please. For a single trivial patch I strongly prefer that some existing
(in this case x86) maintainer takes it in their pullreq, rather than my having
to deal with a pullreq submission from a new-to-the-process person.
(It's extra work to check submissions from new people, which is fine if they're
going to be doing them a lot in future, but for a one-off it's a waste of their
time and mine.)
thanks
-- PMM
