diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2007-04-19 23:29:13 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-04-26 01:26:28 -0400 |
commit | 27a884dc3cb63b93c2b3b643f5b31eed5f8a4d26 (patch) | |
tree | 5a267e40f9b94014be38dad5de0a52b6628834e0 /net/sched/cls_tcindex.c | |
parent | be8bd86321fa7f06359d866ef61fb4d2f3e9dce9 (diff) |
[SK_BUFF]: Convert skb->tail to sk_buff_data_t
So that it is also an offset from skb->head, reduces its size from 8 to 4 bytes
on 64bit architectures, allowing us to combine the 4 bytes hole left by the
layer headers conversion, reducing struct sk_buff size to 256 bytes, i.e. 4
64byte cachelines, and since the sk_buff slab cache is SLAB_HWCACHE_ALIGN...
:-)
Many calculations that previously required that skb->{transport,network,
mac}_header be first converted to a pointer now can be done directly, being
meaningful as offsets or pointers.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/cls_tcindex.c')
-rw-r--r-- | net/sched/cls_tcindex.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/sched/cls_tcindex.c b/net/sched/cls_tcindex.c index 7563fdcef4b7..0537d6066b43 100644 --- a/net/sched/cls_tcindex.c +++ b/net/sched/cls_tcindex.c | |||
@@ -448,7 +448,7 @@ static int tcindex_dump(struct tcf_proto *tp, unsigned long fh, | |||
448 | { | 448 | { |
449 | struct tcindex_data *p = PRIV(tp); | 449 | struct tcindex_data *p = PRIV(tp); |
450 | struct tcindex_filter_result *r = (struct tcindex_filter_result *) fh; | 450 | struct tcindex_filter_result *r = (struct tcindex_filter_result *) fh; |
451 | unsigned char *b = skb->tail; | 451 | unsigned char *b = skb_tail_pointer(skb); |
452 | struct rtattr *rta; | 452 | struct rtattr *rta; |
453 | 453 | ||
454 | DPRINTK("tcindex_dump(tp %p,fh 0x%lx,skb %p,t %p),p %p,r %p,b %p\n", | 454 | DPRINTK("tcindex_dump(tp %p,fh 0x%lx,skb %p,t %p),p %p,r %p,b %p\n", |
@@ -463,7 +463,7 @@ static int tcindex_dump(struct tcf_proto *tp, unsigned long fh, | |||
463 | RTA_PUT(skb,TCA_TCINDEX_SHIFT,sizeof(p->shift),&p->shift); | 463 | RTA_PUT(skb,TCA_TCINDEX_SHIFT,sizeof(p->shift),&p->shift); |
464 | RTA_PUT(skb,TCA_TCINDEX_FALL_THROUGH,sizeof(p->fall_through), | 464 | RTA_PUT(skb,TCA_TCINDEX_FALL_THROUGH,sizeof(p->fall_through), |
465 | &p->fall_through); | 465 | &p->fall_through); |
466 | rta->rta_len = skb->tail-b; | 466 | rta->rta_len = skb_tail_pointer(skb) - b; |
467 | } else { | 467 | } else { |
468 | if (p->perfect) { | 468 | if (p->perfect) { |
469 | t->tcm_handle = r-p->perfect; | 469 | t->tcm_handle = r-p->perfect; |
@@ -486,7 +486,7 @@ static int tcindex_dump(struct tcf_proto *tp, unsigned long fh, | |||
486 | 486 | ||
487 | if (tcf_exts_dump(skb, &r->exts, &tcindex_ext_map) < 0) | 487 | if (tcf_exts_dump(skb, &r->exts, &tcindex_ext_map) < 0) |
488 | goto rtattr_failure; | 488 | goto rtattr_failure; |
489 | rta->rta_len = skb->tail-b; | 489 | rta->rta_len = skb_tail_pointer(skb) - b; |
490 | 490 | ||
491 | if (tcf_exts_dump_stats(skb, &r->exts, &tcindex_ext_map) < 0) | 491 | if (tcf_exts_dump_stats(skb, &r->exts, &tcindex_ext_map) < 0) |
492 | goto rtattr_failure; | 492 | goto rtattr_failure; |