[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 15/22] x86: Grant AMX permission for guest
From: |
Yang Zhong |
Subject: |
Re: [PULL 15/22] x86: Grant AMX permission for guest |
Date: |
Fri, 18 Mar 2022 19:53:44 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Mar 18, 2022 at 11:13:56AM +0100, Michal Prívozník wrote:
> On 3/16/22 16:57, Peter Krempa wrote:
> > On Tue, Mar 08, 2022 at 12:34:38 +0100, Paolo Bonzini wrote:
> >> From: Yang Zhong <yang.zhong@intel.com>
> >>
> >> Kernel allocates 4K xstate buffer by default. For XSAVE features
> >> which require large state component (e.g. AMX), Linux kernel
> >> dynamically expands the xstate buffer only after the process has
> >> acquired the necessary permissions. Those are called dynamically-
> >> enabled XSAVE features (or dynamic xfeatures).
> >>
> >> There are separate permissions for native tasks and guests.
> >>
> >> Qemu should request the guest permissions for dynamic xfeatures
> >> which will be exposed to the guest. This only needs to be done
> >> once before the first vcpu is created.
> >>
> >> KVM implemented one new ARCH_GET_XCOMP_SUPP system attribute API to
> >> get host side supported_xcr0 and Qemu can decide if it can request
> >> dynamically enabled XSAVE features permission.
> >> https://lore.kernel.org/all/20220126152210.3044876-1-pbonzini@redhat.com/
> >>
> >> Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
> >> Signed-off-by: Yang Zhong <yang.zhong@intel.com>
> >> Signed-off-by: Jing Liu <jing2.liu@intel.com>
> >> Message-Id: <20220217060434.52460-4-yang.zhong@intel.com>
> >> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> >> ---
> >> target/i386/cpu.c | 7 +++++
> >> target/i386/cpu.h | 4 +++
> >> target/i386/kvm/kvm-cpu.c | 12 ++++----
> >> target/i386/kvm/kvm.c | 57 ++++++++++++++++++++++++++++++++++++++
> >> target/i386/kvm/kvm_i386.h | 1 +
> >> 5 files changed, 75 insertions(+), 6 deletions(-)
> >
> > With this commit qemu crashes for me when invoking the following
> > QMP command:
> >
> > $ ~pipo/git/qemu.git/build/qemu-system-x86_64 -S -no-user-config
> > -nodefaults -nographic -machine none,accel=kvm -qmp stdio
> > {"QMP": {"version": {"qemu": {"micro": 90, "minor": 2, "major": 6},
> > "package": "v7.0.0-rc0-8-g1d60bb4b14"}, "capabilities": ["oob"]}}
> > {'execute':'qmp_capabilities'}
> > {"return": {}}
> > {"execute":"qom-list-properties","arguments":{"typename":"max-x86_64-cpu"},"id":"libvirt-41"}
> > qemu-system-x86_64: ../target/i386/kvm/kvm-cpu.c:105: kvm_cpu_xsave_init:
> > Assertion `esa->size == eax' failed.
> > Aborted (core dumped)
> >
> > Note that the above is on a box with an 'AMD Ryzen 9 3900X'.
> >
> > Curiously on a laptop with an Intel chip (Intel(R) Core(TM) i7-10610U)
> > it seems to work.
> >
> >
>
> Not trying to beat a dead horse here, but I've just found another
> problem with this patch. On my laptop (Linux maggie
> 5.15.26-gentoo-x86_64 #1 SMP Thu Mar 10 08:55:28 CET 2022 x86_64
> Intel(R) Core(TM) i7-10610U CPU @ 1.80GHz GenuineIntel GNU/Linux), when
> I start a guest it no longer sees AVX instructions:
>
> qemu.git $ ./build/qemu-system-x86_64 -accel kvm -cpu host ...
>
Thanks Michal, this issue is caused by compatibility with older kernel
version.
The Qemu will report below logs:
emu-system-x86_64: warning: cannot get sys attribute capabilities 0
qemu-system-x86_64: warning: cannot get sys attribute capabilities 0
qemu-system-x86_64: warning: cannot get sys attribute capabilities 0
qemu-system-x86_64: warning: host doesn't support requested feature:
CPUID.0DH:EAX [bit 5]
qemu-system-x86_64: warning: host doesn't support requested feature:
CPUID.0DH:EAX [bit 6]
qemu-system-x86_64: warning: host doesn't support requested feature:
CPUID.0DH:EAX [bit 9]
......
Since the AMX changes in Qemu need read ARCH_GET_XCOMP_SUPP attribute to get
host supported_xcr0
value, and new kernel release add this new API. So the older kernel can't
report right xcr0 value.
I made one new patch to fix this issue, please try this patch. thanks!
https://lists.nongnu.org/archive/html/qemu-devel/2022-03/msg04732.html
Paolo, this patch only fix this compatibility issue, but the issue caused by
AMD cpu is still not
fixed from my side because no AMD platform can be used in my side. If you
have no time to check
this issue, maybe I need lookfor this platform from our internal. thanks!
Yang
> Michal
- [PULL 20/22] i386: Add Icelake-Server-v6 CPU model with 5-level EPT support, (continued)
- [PULL 20/22] i386: Add Icelake-Server-v6 CPU model with 5-level EPT support, Paolo Bonzini, 2022/03/08
- [PULL 15/22] x86: Grant AMX permission for guest, Paolo Bonzini, 2022/03/08
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Peter Krempa, 2022/03/16
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Daniel P . Berrangé, 2022/03/16
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Paolo Bonzini, 2022/03/16
- Re: [PULL 15/22] x86: Grant AMX permission for guest, David Edmondson, 2022/03/16
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Daniel P . Berrangé, 2022/03/16
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Maxim Levitsky, 2022/03/17
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Yang Zhong, 2022/03/17
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Michal Prívozník, 2022/03/18
- Re: [PULL 15/22] x86: Grant AMX permission for guest,
Yang Zhong <=
- Re: [PULL 15/22] x86: Grant AMX permission for guest, Yang Zhong, 2022/03/22
[PULL 11/22] target/i386: Throw a #SS when loading a non-canonical IST, Paolo Bonzini, 2022/03/08
[PULL 22/22] gitlab-ci: do not run tests with address sanitizer, Paolo Bonzini, 2022/03/08
[PULL 21/22] KVM: SVM: always set MSR_AMD64_TSC_RATIO to default value, Paolo Bonzini, 2022/03/08
[PULL 19/22] x86: Support XFD and AMX xsave data migration, Paolo Bonzini, 2022/03/08
Re: [PULL v2 00/22] QEMU changes for 7.0 soft freeze, Peter Maydell, 2022/03/10