lkml.org 
[lkml]   [2016]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v6 01/13] x86/xsaves: Define and use fpu_user_xstate_size
Date
> From: Borislav Petkov [mailto:bp@suse.de]
> Sent: Wednesday, May 11, 2016 10:21 AM
> To: Yu, Yu-cheng <yu-cheng.yu@intel.com>
> Cc: linux-kernel@vger.kernel.org; x86@kernel.org; H. Peter Anvin
> <hpa@zytor.com>; Thomas Gleixner <tglx@linutronix.de>; Ingo Molnar
> <mingo@redhat.com>; Dave Hansen <dave.hansen@linux.intel.com>; Andy
> Lutomirski <luto@kernel.org>; Prakhya, Sai Praneeth
> <sai.praneeth.prakhya@intel.com>; Shankar, Ravi V
> <ravi.v.shankar@intel.com>; Yu, Fenghua <fenghua.yu@intel.com>
> Subject: Re: [PATCH v6 01/13] x86/xsaves: Define and use
> fpu_user_xstate_size
>
> On Tue, May 10, 2016 at 04:29:53PM -0700, Yu-cheng Yu wrote:
> > The XSAVE area of kernel can be in standard or compacted format;
>
> "The kernel xstate area... "
>
> and can we call it the xstate area as there are a bunch of XSAVE* insns
> touching it. The file which deals with it is even called that:
> arch/x86/kernel/fpu/xstate.c
>
> > it is always in standard format for user mode. When XSAVES is enabled,
> > the kernel uses the compacted format and it is necessary to use a
> > separate fpu_user_xstate_size for signal/ptrace frames.
> >
> > Based on an earlier patch from Fenghua Yu <fenghua.yu@intel.com>
> >
> > Signed-off-by: Fenghua Yu <fenghua.yu@intel.com>
> > [yu-cheng.yu@intel.com: rebase to current, rename to
> > fpu_user_xstate_size]
> > Signed-off-by: Yu-cheng Yu <yu-cheng.yu@intel.com>
> > Reviewed-by: Dave Hansen <dave.hansen@intel.com>
>
> Maybe I wasn't as clear as I hoped to be. Let me be more specific:
>
> So you either need to do:
>
> ---
> From: Fenghua
>
> ...
>
> Signed-off-by: Fenghua
> Signed-off-by: You
> ...
> ---
>
> or
>
> ---
>
> Based on an earlier patch from Fenghua Yu <fenghua.yu@intel.com>.
>
> Signed-off-by: You
>
> ---
>
> with the second variant making you the author implicitly because you're the
> sender.
>
> Makes more sense this way?

Is this possible to have the third one?

From: Yu-cheng

Signed-off-by: Yu-cheng
Signed-off-by: Fenghua

Thanks.

-Fenghua
\
 
 \ /
  Last update: 2016-05-11 19:41    [W:0.107 / U:0.920 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site