From: "Liu, Jing2" <jing2.liu@intel.com> To: "Bae, Chang Seok" <chang.seok.bae@intel.com>, "bp@suse.de" <bp@suse.de>, "luto@kernel.org" <luto@kernel.org>, "tglx@linutronix.de" <tglx@linutronix.de>, "mingo@kernel.org" <mingo@kernel.org>, "x86@kernel.org" <x86@kernel.org> Cc: "Brown, Len" <len.brown@intel.com>, "Hansen, Dave" <dave.hansen@intel.com>, "Shankar, Ravi V" <ravi.v.shankar@intel.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "kvm@vger.kernel.org" <kvm@vger.kernel.org> Subject: RE: [PATCH v3 10/21] x86/fpu/xstate: Update xstate save function to support dynamic xstate Date: Thu, 7 Jan 2021 08:41:17 +0000 [thread overview] Message-ID: <BYAPR11MB3256BBBB24F9131D99CF7EF5A9AF0@BYAPR11MB3256.namprd11.prod.outlook.com> (raw) In-Reply-To: <20201223155717.19556-11-chang.seok.bae@intel.com> -----Original Message----- From: Bae, Chang Seok <chang.seok.bae@intel.com> Sent: Wednesday, December 23, 2020 11:57 PM To: bp@suse.de; luto@kernel.org; tglx@linutronix.de; mingo@kernel.org; x86@kernel.org Cc: Brown, Len <len.brown@intel.com>; Hansen, Dave <dave.hansen@intel.com>; Liu, Jing2 <jing2.liu@intel.com>; Shankar, Ravi V <ravi.v.shankar@intel.com>; linux-kernel@vger.kernel.org; Bae, Chang Seok <chang.seok.bae@intel.com>; kvm@vger.kernel.org Subject: [PATCH v3 10/21] x86/fpu/xstate: Update xstate save function to support dynamic xstate copy_xregs_to_kernel() used to save all user states in a kernel buffer. When the dynamic user state is enabled, it becomes conditional which state to be saved. fpu->state_mask can indicate which state components are reserved to be saved in XSAVE buffer. Use it as XSAVE's instruction mask to select states. KVM used to save all xstate via copy_xregs_to_kernel(). Update KVM to set a valid fpu->state_mask, which will be necessary to correctly handle dynamic state buffers. See comments together below. No functional change until the kernel supports dynamic user states. Signed-off-by: Chang S. Bae <chang.seok.bae@intel.com> Reviewed-by: Len Brown <len.brown@intel.com> Cc: x86@kernel.org Cc: linux-kernel@vger.kernel.org Cc: kvm@vger.kernel.org [...] /* * AVX512 state is tracked here because its use is diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 4aecfba04bd3..93b5bacad67a 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c @@ -9214,15 +9214,20 @@ static int complete_emulated_mmio(struct kvm_vcpu *vcpu) static void kvm_save_current_fpu(struct fpu *fpu) { + struct fpu *src_fpu = ¤t->thread.fpu; + /* * If the target FPU state is not resident in the CPU registers, just * memcpy() from current, else save CPU state directly to the target. */ - if (test_thread_flag(TIF_NEED_FPU_LOAD)) - memcpy(&fpu->state, ¤t->thread.fpu.state, + if (test_thread_flag(TIF_NEED_FPU_LOAD)) { + memcpy(&fpu->state, &src_fpu->state, fpu_kernel_xstate_min_size); For kvm, if we assume that it does not support dynamic features until this series, memcpy for only fpu->state is correct. I think this kind of assumption is reasonable and we only make original xstate work. - else + } else { + if (fpu->state_mask != src_fpu->state_mask) + fpu->state_mask = src_fpu->state_mask; Though dynamic feature is not supported in kvm now, this function still need consider more things for fpu->state_mask. I suggest that we can set it before if...else (for both cases) and not change other. Thanks, Jing copy_fpregs_to_fpstate(fpu); + } } /* Swap (qemu) user FPU context for the guest FPU context. */ -- 2.17.1
next prev parent reply other threads:[~2021-01-07 8:42 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <20201223155717.19556-1-chang.seok.bae@intel.com> 2020-12-23 15:56 ` [PATCH v3 01/21] x86/fpu/xstate: Modify initialization helper to handle both static and dynamic buffers Chang S. Bae 2021-01-15 12:40 ` Borislav Petkov 2020-12-23 15:56 ` [PATCH v3 03/21] x86/fpu/xstate: Modify address finders " Chang S. Bae 2021-01-15 13:06 ` Borislav Petkov 2020-12-23 15:57 ` [PATCH v3 04/21] x86/fpu/xstate: Modify context switch helpers " Chang S. Bae 2021-01-15 13:18 ` Borislav Petkov 2021-01-19 18:49 ` Bae, Chang Seok 2020-12-23 15:57 ` [PATCH v3 06/21] x86/fpu/xstate: Calculate and remember dynamic xstate buffer sizes Chang S. Bae 2021-01-22 11:44 ` Borislav Petkov 2021-01-27 1:23 ` Bae, Chang Seok 2021-01-27 9:38 ` Borislav Petkov 2021-02-03 2:54 ` Bae, Chang Seok 2020-12-23 15:57 ` [PATCH v3 10/21] x86/fpu/xstate: Update xstate save function to support dynamic xstate Chang S. Bae 2021-01-07 8:41 ` Liu, Jing2 [this message] 2021-01-07 18:40 ` Bae, Chang Seok 2021-01-12 2:52 ` Liu, Jing2 2021-01-15 4:59 ` Bae, Chang Seok 2021-01-15 5:45 ` Liu, Jing2 2021-02-08 12:33 ` Borislav Petkov 2021-02-09 15:48 ` Bae, Chang Seok
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=BYAPR11MB3256BBBB24F9131D99CF7EF5A9AF0@BYAPR11MB3256.namprd11.prod.outlook.com \ --to=jing2.liu@intel.com \ --cc=bp@suse.de \ --cc=chang.seok.bae@intel.com \ --cc=dave.hansen@intel.com \ --cc=kvm@vger.kernel.org \ --cc=len.brown@intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=luto@kernel.org \ --cc=mingo@kernel.org \ --cc=ravi.v.shankar@intel.com \ --cc=tglx@linutronix.de \ --cc=x86@kernel.org \ --subject='RE: [PATCH v3 10/21] x86/fpu/xstate: Update xstate save function to support dynamic xstate' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).