Re: [PATCH V9 08/21] csky: Process management and Signal

From: Guo Ren
Date: Wed Oct 17 2018 - 22:37:41 EST


On Wed, Oct 17, 2018 at 05:11:00PM +0200, Arnd Bergmann wrote:
> On Tue, Oct 16, 2018 at 5:01 AM Guo Ren <ren_guo@xxxxxxxxx> wrote:
> >
> > This patch adds files related to task_switch, sigcontext, signal,
> > fpu context switch.
> >
> > Signed-off-by: Guo Ren <ren_guo@xxxxxxxxx>
> > Cc: Arnd Bergmann <arnd@xxxxxxxx>
> > ---
> > Changelog:
> > - abiv2/fpu.c: Userspace should never be sent NSIGXXX as a si_code.
> > Use FPE_FLTUNK instead.
> > - abiv2/fpu.c: Use force_sig_fault instead.
>
> I'm hesitant to provide a Reviewed-by: tag for this one, as I don't
> understand the signal handling enough myself, and it's something
> that should be reviewed properly because of the ABI implications.
>
> Maybe someone else can have a closer look at this?
NSIGXXX and force_sig_fault modification is from Eric W. Biederman
in PATCH V6 and I forget add:
Cc: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>

Best Regards
Guo Ren