[Top] [All Lists]

Re: [PATCH 1/3] signal: avoid useless test in do_signal()

To: "Ralf Baechle" <>
Subject: Re: [PATCH 1/3] signal: avoid useless test in do_signal()
From: "Franck Bui-Huu" <>
Date: Fri, 9 Feb 2007 17:50:36 +0100
Cc:,, "Franck Bui-Huu" <>
Domainkey-signature: a=rsa-sha1; c=nofws;; s=beta; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=UivTl0Rm2pXaNnmlj9W1NimNSAv/dnzWQhIkITRoqjKY/V0rHfVQLmMR5VXeAenB1DyVp8bzrSVz2y/HmwI0aMdC85mzj0FTc3pnwvE46SBUBsONh5KpYGUFfhujKxvU8mv+QyGQ0X2Wjv00cq2EL+fyKHMuWV9dw4kksUl0Tl4=
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <>
On 2/9/07, Ralf Baechle <> wrote:
Ralf Baechle wrote:
On Fri, Feb 09, 2007 at 04:07:36PM +0100, Franck Bui-Huu wrote:

-                       if (test_thread_flag(TIF_RESTORE_SIGMASK))
-                               clear_thread_flag(TIF_RESTORE_SIGMASK);

This is a microoptimization.  The assumption here is TIF_RESTORE_SIGMASK
will rarely need to be cleared and atomic operations are somewhat
expensive if as in this case we have to assume the cacheline isn't
held exclusive yet.

I missed that. You can forget this patch or maybe something like this
is more appropriate ?

        if (unlikely(test_thread_flag(TIF_RESTORE_SIGMASK)))

<Prev in Thread] Current Thread [Next in Thread>