lkml.org 
[lkml]   [2012]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] xtensa: fix build failure in xtensa/kernel/signal.c
    From
    Date
    On Wed, 2012-04-18 at 16:50 -0400, Paul Gortmaker wrote:
    > Caused by commit 3785006ac3c8941feb63097c416de92114a6bc39
    >
    > "xtensa: don't mask signals if we fail to setup signal stack"
    >
    > It assigns a return value to "ret", but there is no such variable
    > anywhere in scope. Create one.
    >
    > Cc: Matt Fleming <matt.fleming@intel.com>
    > Cc: Oleg Nesterov <oleg@redhat.com>
    > Cc: Chris Zankel <chris@zankel.net>
    > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
    >
    > diff --git a/arch/xtensa/kernel/signal.c b/arch/xtensa/kernel/signal.c
    > index b69b000..d78869a 100644
    > --- a/arch/xtensa/kernel/signal.c
    > +++ b/arch/xtensa/kernel/signal.c
    > @@ -496,6 +496,7 @@ int do_signal(struct pt_regs *regs, sigset_t *oldset)
    > signr = get_signal_to_deliver(&info, &ka, regs, NULL);
    >
    > if (signr > 0) {
    > + int ret;
    >
    > /* Are we from a system call? */
    >

    Whoops! Thanks Paul, that's my bad. Looks like I went code-blind after
    making the same kinds of changes to lots of different architectures.

    I've added Andrew to the Cc list since he picked up the last set of
    xtensa signal patches.



    \
     
     \ /
      Last update: 2012-04-19 09:47    [W:0.026 / U:0.672 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site