Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1341664 > unrolled thread

Re: [PATCH 00/10] x86/xsaves: Fix XSAVES known issues

Started byIngo Molnar <mingo@kernel.org>
First post2016-02-24 09:50 +0100
Last post2016-02-24 17:40 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 00/10] x86/xsaves: Fix XSAVES known issues Ingo Molnar <mingo@kernel.org> - 2016-02-24 09:50 +0100
    Re: [PATCH 00/10] x86/xsaves: Fix XSAVES known issues Yu-cheng Yu <yu-cheng.yu@intel.com> - 2016-02-24 17:40 +0100

#1341664 — Re: [PATCH 00/10] x86/xsaves: Fix XSAVES known issues

FromIngo Molnar <mingo@kernel.org>
Date2016-02-24 09:50 +0100
SubjectRe: [PATCH 00/10] x86/xsaves: Fix XSAVES known issues
Message-ID<r5DiW-1S4-31@gated-at.bofh.it>
* Yu-cheng Yu <yu-cheng.yu@intel.com> wrote:

> XSAVES is a kernel-mode instruction. It offers a compacted format and
> memory-write optimization. These patches fix known issues in the first
> implementation. They are intended for discussion and getting feedback
> before actually getting applied.  
> 
> Patch 1, 2, and 4 are for converting between kernel-mode xstate area and
> signal frames.
> 
> Patch 8 is for converting between kernel-mode xstate area and ptrace
> frames.
> 
> Patch 3 and 7 fix optimization issues introduced by XSAVES to the buffer
> init_fpstate.
> 
> Patch 5 and 6 are related to xstate component offsets.
> 
> Patch 9 fixes xstate area print out.
> 
> Patch 10 re-enables XSAVES.
> 
> Yu-cheng Yu (10):
>   x86/xsaves: Define and use user_xstate_size for xstate size in signal
>     context
>   x86/xsaves: Rename xstate_size to kernel_xstate_size to explicitly
>     distinguish xstate size in kernel from user space
>   x86/xsaves: Keep init_fpstate.xsave.header.xfeatures as zero for init
>     optimization
>   x86/xsaves: Introduce a new check that allows correct xstates copy
>     from kernel to user directly
>   x86/xsaves: Align xstate components according to CPUID
>   x86/xsaves: Supervisor state component offset
>   x86/xsaves: Fix init_fpstate.header.xcomp_bv
>   x86/xsaves: Fix PTRACE frames for XSAVES
>   x86/xsaves: Fix XSTATE component offset print out
>   x86/xsaves: Re-enable XSAVES
> 
>  arch/x86/include/asm/fpu/types.h  |   2 +
>  arch/x86/include/asm/fpu/xstate.h |   8 +-
>  arch/x86/include/asm/processor.h  |   3 +-
>  arch/x86/kernel/fpu/core.c        |   6 +-
>  arch/x86/kernel/fpu/init.c        |  32 +---
>  arch/x86/kernel/fpu/regset.c      |  56 ++++--
>  arch/x86/kernel/fpu/signal.c      |  69 ++++++-
>  arch/x86/kernel/fpu/xstate.c      | 388 +++++++++++++++++++++++++++++---------
>  8 files changed, 425 insertions(+), 139 deletions(-)

Small housekeeping request: could you please make sure your series is properly 
threaded for email clients? This submission was sent as singular patches, without 
any References header.

I don't even know how you coaxed git-send-email into doing that - did you use 
--no-thread?

The recommended flags are --thread --no-chain-reply-to.

Thanks!

	Ingo

[toc] | [next] | [standalone]


#1342189

FromYu-cheng Yu <yu-cheng.yu@intel.com>
Date2016-02-24 17:40 +0100
Message-ID<r5KDN-7ce-35@gated-at.bofh.it>
In reply to#1341664
On Wed, Feb 24, 2016 at 09:43:36AM +0100, Ingo Molnar wrote:
> 
> Small housekeeping request: could you please make sure your series is properly 
> threaded for email clients? This submission was sent as singular patches, without 
> any References header.
> 
> I don't even know how you coaxed git-send-email into doing that - did you use 
> --no-thread?
> 
> The recommended flags are --thread --no-chain-reply-to.
> 
> Thanks!
> 
> 	Ingo

I will fix it when sending out the next version.
Thanks!

-- Yu-cheng
 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web