Re: [PATCH 0/1] (Was: [regression] boot failure on alpha, bisected)

From: Oleg Nesterov
Date: Sun Oct 14 2012 - 14:40:47 EST


On 10/14, Oleg Nesterov wrote:
>
> Again, this is fine. But why "else" ? It seems that
>
> if (thread_flags & _TIF_SIGPENDING) {
> do_signal(regs, r0, r19);
> r0 = 0;
> }
>
> if (thread_flags & _TIF_NOTIFY_RESUME) {
> clear_thread_flag(TIF_NOTIFY_RESUME);
> tracehook_notify_resume(regs);
> }
>
> makes a bit more sense?

Except I meant "if (_TIF_NOTIFY_RESUME)" first.

Oleg.

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