* [PATCH] [IPV6] LL_RESERVED_SPACE
@ 2004-02-09 4:45 YOSHIFUJI Hideaki / 吉藤英明
2004-02-09 19:18 ` David S. Miller
0 siblings, 1 reply; 2+ messages in thread
From: YOSHIFUJI Hideaki / 吉藤英明 @ 2004-02-09 4:45 UTC (permalink / raw)
To: davem; +Cc: netdev, yoshfuji
D: [IPV6] Use LL_RESERVED_SPACE() where appricable.
===== net/ipv6/ip6_output.c 1.49 vs edited =====
--- 1.49/net/ipv6/ip6_output.c Thu Jan 22 15:18:21 2004
+++ edited/net/ipv6/ip6_output.c Mon Feb 9 13:25:45 2004
@@ -218,7 +218,7 @@
*/
head_room = opt->opt_nflen + opt->opt_flen;
seg_len += head_room;
- head_room += sizeof(struct ipv6hdr) + ((dst->dev->hard_header_len + 15)&~15);
+ head_room += sizeof(struct ipv6hdr) + LL_RESERVED_SPACE(dst->dev);
if (skb_headroom(skb) < head_room) {
struct sk_buff *skb2 = skb_realloc_headroom(skb, head_room);
@@ -844,7 +844,7 @@
mtu = inet->cork.fragsize;
}
- hh_len = (rt->u.dst.dev->hard_header_len&~15) + 16;
+ hh_len = LL_RESERVED_SPACE(rt->u.dst.dev);
fragheaderlen = sizeof(struct ipv6hdr) + (opt ? opt->opt_nflen : 0);
maxfraglen = ((mtu - fragheaderlen) & ~7) + fragheaderlen - sizeof(struct frag_hdr);
@@ -881,14 +881,14 @@
alloclen += sizeof(struct frag_hdr);
if (transhdrlen) {
skb = sock_alloc_send_skb(sk,
- alloclen + hh_len + 15,
+ alloclen + hh_len,
(flags & MSG_DONTWAIT), &err);
} else {
skb = NULL;
if (atomic_read(&sk->sk_wmem_alloc) <=
2 * sk->sk_sndbuf)
skb = sock_wmalloc(sk,
- alloclen + hh_len + 15, 1,
+ alloclen + hh_len, 1,
sk->sk_allocation);
if (unlikely(skb == NULL))
err = -ENOBUFS;
===== net/ipv6/mcast.c 1.50 vs edited =====
--- 1.50/net/ipv6/mcast.c Thu Jan 29 09:06:25 2004
+++ edited/net/ipv6/mcast.c Mon Feb 9 13:35:19 2004
@@ -1233,12 +1233,13 @@
IPV6_TLV_ROUTERALERT, 2, 0, 0,
IPV6_TLV_PADN, 0 };
- skb = sock_alloc_send_skb(sk, size + dev->hard_header_len+15, 1, &err);
+ /* we assume size > sizeof(ra) here */
+ skb = sock_alloc_send_skb(sk, size + LL_RESERVED_SPACE(dev), 1, &err);
if (skb == 0)
return 0;
- skb_reserve(skb, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(skb, LL_RESERVED_SPACE(dev));
if (dev->hard_header) {
unsigned char ha[MAX_ADDR_LEN];
@@ -1580,12 +1581,12 @@
payload_len = len + sizeof(ra);
full_len = sizeof(struct ipv6hdr) + payload_len;
- skb = sock_alloc_send_skb(sk, dev->hard_header_len + full_len + 15, 1, &err);
+ skb = sock_alloc_send_skb(sk, LL_RESERVED_SPACE(dev) + full_len, 1, &err);
if (skb == NULL)
return;
- skb_reserve(skb, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(skb, LL_RESERVED_SPACE(dev));
if (dev->hard_header) {
unsigned char ha[MAX_ADDR_LEN];
ndisc_mc_map(snd_addr, ha, dev, 1);
===== net/ipv6/ndisc.c 1.70 vs edited =====
--- 1.70/net/ipv6/ndisc.c Sat Feb 7 08:28:55 2004
+++ edited/net/ipv6/ndisc.c Mon Feb 9 13:25:45 2004
@@ -459,7 +459,7 @@
inc_opt = 0;
}
- skb = sock_alloc_send_skb(sk, MAX_HEADER + len + dev->hard_header_len + 15,
+ skb = sock_alloc_send_skb(sk, MAX_HEADER + len + LL_RESERVED_SPACE(dev),
1, &err);
if (skb == NULL) {
@@ -468,7 +468,7 @@
return;
}
- skb_reserve(skb, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(skb, LL_RESERVED_SPACE(dev));
ip6_nd_hdr(sk, skb, dev, src_addr, daddr, IPPROTO_ICMPV6, len);
msg = (struct nd_msg *)skb_put(skb, len);
@@ -545,7 +545,7 @@
if (send_llinfo)
len += NDISC_OPT_SPACE(dev->addr_len);
- skb = sock_alloc_send_skb(sk, MAX_HEADER + len + dev->hard_header_len + 15,
+ skb = sock_alloc_send_skb(sk, MAX_HEADER + len + LL_RESERVED_SPACE(dev),
1, &err);
if (skb == NULL) {
ND_PRINTK1("send_ns: alloc skb failed\n");
@@ -553,7 +553,7 @@
return;
}
- skb_reserve(skb, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(skb, LL_RESERVED_SPACE(dev));
ip6_nd_hdr(sk, skb, dev, saddr, daddr, IPPROTO_ICMPV6, len);
msg = (struct nd_msg *)skb_put(skb, len);
@@ -617,7 +617,7 @@
if (dev->addr_len)
len += NDISC_OPT_SPACE(dev->addr_len);
- skb = sock_alloc_send_skb(sk, MAX_HEADER + len + dev->hard_header_len + 15,
+ skb = sock_alloc_send_skb(sk, MAX_HEADER + len + LL_RESERVED_SPACE(dev),
1, &err);
if (skb == NULL) {
ND_PRINTK1("send_ns: alloc skb failed\n");
@@ -625,7 +625,7 @@
return;
}
- skb_reserve(skb, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(skb, LL_RESERVED_SPACE(dev));
ip6_nd_hdr(sk, skb, dev, saddr, daddr, IPPROTO_ICMPV6, len);
hdr = (struct icmp6hdr *)skb_put(skb, len);
@@ -1305,7 +1305,7 @@
rd_len &= ~0x7;
len += rd_len;
- buff = sock_alloc_send_skb(sk, MAX_HEADER + len + dev->hard_header_len + 15,
+ buff = sock_alloc_send_skb(sk, MAX_HEADER + len + LL_RESERVED_SPACE(dev),
1, &err);
if (buff == NULL) {
ND_PRINTK1("ndisc_send_redirect: alloc_skb failed\n");
@@ -1315,7 +1315,7 @@
hlen = 0;
- skb_reserve(buff, (dev->hard_header_len + 15) & ~15);
+ skb_reserve(buff, LL_RESERVED_SPACE(dev));
ip6_nd_hdr(sk, buff, dev, &saddr_buf, &skb->nh.ipv6h->saddr,
IPPROTO_ICMPV6, len);
--
Hideaki YOSHIFUJI @ USAGI Project <yoshfuji@linux-ipv6.org>
GPG FP: 9022 65EB 1ECF 3AD1 0BDF 80D8 4807 F894 E062 0EEA
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] [IPV6] LL_RESERVED_SPACE
2004-02-09 4:45 [PATCH] [IPV6] LL_RESERVED_SPACE YOSHIFUJI Hideaki / 吉藤英明
@ 2004-02-09 19:18 ` David S. Miller
0 siblings, 0 replies; 2+ messages in thread
From: David S. Miller @ 2004-02-09 19:18 UTC (permalink / raw)
To: yoshfuji; +Cc: netdev
I applied all of your LL_RESERVED_SPACE() patches, thanks for
doing this audit.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-02-09 19:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-02-09 4:45 [PATCH] [IPV6] LL_RESERVED_SPACE YOSHIFUJI Hideaki / 吉藤英明
2004-02-09 19:18 ` David S. Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).