Re: [PATCH 1/4] m32r: restore _BLOCKABLE

[Al Viro]

On Fri, Oct 15, 2010 at 09:16:45PM -0400, Kyle McMartin wrote:
> Commit a7f8388e accidentally removed it...
> 
> Signed-off-by: Kyle McMartin <kyle@xxxxxxxxxx>
> ---
>  arch/m32r/kernel/signal.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/m32r/kernel/signal.c b/arch/m32r/kernel/signal.c
> index 7bbe386..cc36fe1 100644
> --- a/arch/m32r/kernel/signal.c
> +++ b/arch/m32r/kernel/signal.c
> @@ -28,6 +28,8 @@
>  
>  #define DEBUG_SIG 0
>  
> +#define _BLOCKABLE (~(sigmask(SIGKILL) | sigmask(SIGSTOP)))
> +

Grrr...   Sorry, reordering breakage.  In the signals tree here I have

static inline void sig_set_blocked(struct sigset_t *set)
{
        sigdelsetmask(set, sigmask(SIGKILL) | sigmask(SIGSTOP));
        spin_lock_irq(&current->sighand->siglock);
        current->blocked = *set;
        recalc_sigpending();
        spin_unlock_irq(&current->sighand->siglock);
}

and it's used all over the place (including quite a few places where
we currently have sigprocmask(SIG_SETMASK, set, NULL), which is what
it's equivalent to).  With that done, m32r doesn't use _BLOCKABLE
anywhere, so it got removed.  And that chunk got picked when I'd been
reordering the queue to pull the arch-specific fixes in front.  Sorry.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/