Message-ID: <49809C2E.3000005@redhat.com>
Date: Wed, 28 Jan 2009 12:55:58 -0500
From: Chris Snook <csnook@redhat.com>
Organization: Red Hat
User-Agent: Thunderbird 2.0.0.19 (X11/20090105)
MIME-Version: 1.0
To: Davide Libenzi <davidel@xmailserver.org>
CC: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: if (unlikely(...)) == unnecessary?
References: <alpine.DEB.1.10.0901272346480.20262@alien.or.mcafeemobile.com>
In-Reply-To: <alpine.DEB.1.10.0901272346480.20262@alien.or.mcafeemobile.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1169
Lines: 22

Davide Libenzi wrote:
> I noticed that GCC >= 3.3 (not tried the ones before) automatically 
> branches out the "if" code (and follow-through the "else" code, if there). 
> Is that a coincidence or a rule we can rely on going forward?

That's the default behavior, but there are lots of things that can cause it to 
behave differently.  Also, not all branch predictors behave the same way, and 
some architectures use things like conditional instructions to fill their 
pipeline bubbles, so it's still generally useful to have a real compiler hint in 
fast-path code, even if it ends up being a no-op most of the time.

Most kernel code isn't so clock-cycle-critical that it needs these annotations. 
  If you're working on code that already has them, that's a good indication you 
should probably use them too, but otherwise you don't need to worry about it 
unless your code starts chewing up a lot of CPU time.

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