From: jamal <[EMAIL PROTECTED]>
Date: Mon, 25 Jul 2005 18:42:55 -0400

> Having centralized it like Dave is intending to makes it tricky on where
> to actually issue the hold/put. 

We should probably just bite the bullet and do something like:

static inline void skb_set_input_dev(struct sk_buff *skb, struct net_device 
*dev)
{
        struct net_device *orig_dev = skb->input_dev;

        if (orig_dev)
                dev_put(orig_dev);
        dev_hold(skb->input_dev = dev); 
}

and in __kfree_skb():

        if (skb->input_dev)
                dev_put(skb->input_dev);
-
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

Reply via email to