Search Linux Wireless

Re: [PATCH 4/7] mac80211: make ieee80211_get_hdrlen_from_skb return unsigned

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> unlikely() isn't some magic make-me-faster function, it just moves
> code
> to the end of the function to get it out of the icache and jumps to it
> in the unlikely case it is taken.  When all there is is a return, I
> don't think it even makes any difference.  In both these cases a
> comment
> is probably more appropriate.

That's not true on ppc where you can actually tell the processor about
the jump likelyhood for its branch prediction.

> On X86-32 at least the generated code is exactly the same with or
> without the unlikely()s.  I'll put them back in a follow-on if people
> are really attached to them.

Please keep them there, I think they make sense for normal operation.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux