| To: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: RFC: pskb_expand_head instead of skb_copy in skb_checksum_help |
| From: | "David S. Miller" <davem@xxxxxxxxxx> |
| Date: | Tue, 24 Aug 2004 17:17:09 -0700 |
| Cc: | jmorris@xxxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <20040824110532.GA10241@gondor.apana.org.au> |
| References: | <20040824110532.GA10241@gondor.apana.org.au> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Tue, 24 Aug 2004 21:05:32 +1000 Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote: > Here is the patch that you wanted to shoot holes at :) > > The idea is simple. None of the callers of skb_checksum are passing it > skb's which are shared. They may be cloned however. But the application > checksum is always in the skb header so there is no need to linearise it. > > Supposing all these assumptions are correct, then we can avoid the overhead > of skb_copy() and get away with pskb_expand_head(). I'm going to apply this. If your assertion is wrong, then the skb_shared() BUG() will trigger in pskb_expand_head() and we will know :-) |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [RFC/PATCH] Convert architecture dependent rtnetlink attribute to fixed sized ones, David S. Miller |
|---|---|
| Next by Date: | Re: [PATCH 2.6] deliver_skb cleanup, David S. Miller |
| Previous by Thread: | Re: RFC: pskb_expand_head instead of skb_copy in skb_checksum_help, James Morris |
| Next by Thread: | [PATCH] add SCTP to xfrm_flowi_{sport,dport}(), YOSHIFUJI Hideaki / 吉藤英明 |
| Indexes: | [Date] [Thread] [Top] [All Lists] |