Re: [PATCH] small skbuff.[ch] tweaks

[Mitchell Blank Jr <mitch@sfgoth.com>]

Andi Kleen wrote:
> > > Both unlikely(!ptr) and likely(ptr) are not needed because gcc assumes this
> > > by default
> > 
> > Is there any disadvantage to stating it explicitly?
> 
> It makes the code much uglier.

Well I guess it's a matter of taste then.  Personally I like unlikely()/
likely() a lot, even from just a readability standpoint.  I think it provides
a nice hint to the structure of code while reading it ("ok, we're just
handling an error case here, the meat of the code is below")  If anything they
help comment the code.

I certainly think they're preferable to "this is an unlikely error condition but
we shouldn't mark it as such because of some gcc trivia lets us save a few
characters of typing"

However if the consensus is that those unlikely()'s should be removed I'll be
happy to spin a new patch with those removed.

-Mitch