From: Christoph Hellwig <[EMAIL PROTECTED]> Date: Thu, 14 Dec 2006 22:30:09 +0000
> Maybe you should only move the slowpath out of line ala: > > static inline void dev_kfree_skb_irq(struct sk_buff *skb) > { > if (atomic_dec_and_test(&skb->users)) > __dev_kfree_skb_irq(skb); > } The atomic operation all by itself is either a function call or a 6-7 instruction sequence, so the inlining doesn't make sense even in this case. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html