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/decnet | |
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/decnet')
-rw-r--r-- | net/decnet/dn_nsp_out.c | 6 | ||||
-rw-r--r-- | net/decnet/dn_route.c | 4 | ||||
-rw-r--r-- | net/decnet/dn_table.c | 8 | ||||
-rw-r--r-- | net/decnet/netfilter/dn_rtmsg.c | 2 |
4 files changed, 11 insertions, 9 deletions
diff --git a/net/decnet/dn_nsp_out.c b/net/decnet/dn_nsp_out.c index 84b8c5b45fef..7404653880b0 100644 --- a/net/decnet/dn_nsp_out.c +++ b/net/decnet/dn_nsp_out.c | |||
@@ -681,8 +681,10 @@ void dn_nsp_send_conninit(struct sock *sk, unsigned char msgflg) | |||
681 | if (scp->peer.sdn_objnum) | 681 | if (scp->peer.sdn_objnum) |
682 | type = 0; | 682 | type = 0; |
683 | 683 | ||
684 | skb_put(skb, dn_sockaddr2username(&scp->peer, skb->tail, type)); | 684 | skb_put(skb, dn_sockaddr2username(&scp->peer, |
685 | skb_put(skb, dn_sockaddr2username(&scp->addr, skb->tail, 2)); | 685 | skb_tail_pointer(skb), type)); |
686 | skb_put(skb, dn_sockaddr2username(&scp->addr, | ||
687 | skb_tail_pointer(skb), 2)); | ||
686 | 688 | ||
687 | menuver = DN_MENUVER_ACC | DN_MENUVER_USR; | 689 | menuver = DN_MENUVER_ACC | DN_MENUVER_USR; |
688 | if (scp->peer.sdn_flags & SDF_PROXY) | 690 | if (scp->peer.sdn_flags & SDF_PROXY) |
diff --git a/net/decnet/dn_route.c b/net/decnet/dn_route.c index bb73bf16630f..9678b096b844 100644 --- a/net/decnet/dn_route.c +++ b/net/decnet/dn_route.c | |||
@@ -1468,7 +1468,7 @@ static int dn_rt_fill_info(struct sk_buff *skb, u32 pid, u32 seq, | |||
1468 | struct dn_route *rt = (struct dn_route *)skb->dst; | 1468 | struct dn_route *rt = (struct dn_route *)skb->dst; |
1469 | struct rtmsg *r; | 1469 | struct rtmsg *r; |
1470 | struct nlmsghdr *nlh; | 1470 | struct nlmsghdr *nlh; |
1471 | unsigned char *b = skb->tail; | 1471 | unsigned char *b = skb_tail_pointer(skb); |
1472 | long expires; | 1472 | long expires; |
1473 | 1473 | ||
1474 | nlh = NLMSG_NEW(skb, pid, seq, event, sizeof(*r), flags); | 1474 | nlh = NLMSG_NEW(skb, pid, seq, event, sizeof(*r), flags); |
@@ -1509,7 +1509,7 @@ static int dn_rt_fill_info(struct sk_buff *skb, u32 pid, u32 seq, | |||
1509 | if (rt->fl.iif) | 1509 | if (rt->fl.iif) |
1510 | RTA_PUT(skb, RTA_IIF, sizeof(int), &rt->fl.iif); | 1510 | RTA_PUT(skb, RTA_IIF, sizeof(int), &rt->fl.iif); |
1511 | 1511 | ||
1512 | nlh->nlmsg_len = skb->tail - b; | 1512 | nlh->nlmsg_len = skb_tail_pointer(skb) - b; |
1513 | return skb->len; | 1513 | return skb->len; |
1514 | 1514 | ||
1515 | nlmsg_failure: | 1515 | nlmsg_failure: |
diff --git a/net/decnet/dn_table.c b/net/decnet/dn_table.c index 780a141f8342..544c45540746 100644 --- a/net/decnet/dn_table.c +++ b/net/decnet/dn_table.c | |||
@@ -295,7 +295,7 @@ static int dn_fib_dump_info(struct sk_buff *skb, u32 pid, u32 seq, int event, | |||
295 | { | 295 | { |
296 | struct rtmsg *rtm; | 296 | struct rtmsg *rtm; |
297 | struct nlmsghdr *nlh; | 297 | struct nlmsghdr *nlh; |
298 | unsigned char *b = skb->tail; | 298 | unsigned char *b = skb_tail_pointer(skb); |
299 | 299 | ||
300 | nlh = NLMSG_NEW(skb, pid, seq, event, sizeof(*rtm), flags); | 300 | nlh = NLMSG_NEW(skb, pid, seq, event, sizeof(*rtm), flags); |
301 | rtm = NLMSG_DATA(nlh); | 301 | rtm = NLMSG_DATA(nlh); |
@@ -337,13 +337,13 @@ static int dn_fib_dump_info(struct sk_buff *skb, u32 pid, u32 seq, int event, | |||
337 | nhp->rtnh_ifindex = nh->nh_oif; | 337 | nhp->rtnh_ifindex = nh->nh_oif; |
338 | if (nh->nh_gw) | 338 | if (nh->nh_gw) |
339 | RTA_PUT(skb, RTA_GATEWAY, 2, &nh->nh_gw); | 339 | RTA_PUT(skb, RTA_GATEWAY, 2, &nh->nh_gw); |
340 | nhp->rtnh_len = skb->tail - (unsigned char *)nhp; | 340 | nhp->rtnh_len = skb_tail_pointer(skb) - (unsigned char *)nhp; |
341 | } endfor_nexthops(fi); | 341 | } endfor_nexthops(fi); |
342 | mp_head->rta_type = RTA_MULTIPATH; | 342 | mp_head->rta_type = RTA_MULTIPATH; |
343 | mp_head->rta_len = skb->tail - (u8*)mp_head; | 343 | mp_head->rta_len = skb_tail_pointer(skb) - (u8 *)mp_head; |
344 | } | 344 | } |
345 | 345 | ||
346 | nlh->nlmsg_len = skb->tail - b; | 346 | nlh->nlmsg_len = skb_tail_pointer(skb) - b; |
347 | return skb->len; | 347 | return skb->len; |
348 | 348 | ||
349 | 349 | ||
diff --git a/net/decnet/netfilter/dn_rtmsg.c b/net/decnet/netfilter/dn_rtmsg.c index 0e62def05a58..ceefd9dd0c92 100644 --- a/net/decnet/netfilter/dn_rtmsg.c +++ b/net/decnet/netfilter/dn_rtmsg.c | |||
@@ -33,7 +33,7 @@ static struct sk_buff *dnrmg_build_message(struct sk_buff *rt_skb, int *errp) | |||
33 | { | 33 | { |
34 | struct sk_buff *skb = NULL; | 34 | struct sk_buff *skb = NULL; |
35 | size_t size; | 35 | size_t size; |
36 | unsigned char *old_tail; | 36 | sk_buff_data_t old_tail; |
37 | struct nlmsghdr *nlh; | 37 | struct nlmsghdr *nlh; |
38 | unsigned char *ptr; | 38 | unsigned char *ptr; |
39 | struct nf_dn_rtmsg *rtm; | 39 | struct nf_dn_rtmsg *rtm; |