Attachment 309841 Details for Bug 451196 – patch to fix rebinding IPIP,GRE,SIT for RHEL5

[patch] patch to fix rebinding IPIP,GRE,SIT for RHEL5

rhel5.patch (text/plain), 9.25 KB, created by Michal Schmidt on 2008-06-19 10:24:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michal Schmidt

Created: 2008-06-19 10:24:55 UTC

Size: 9.25 KB

patch

obsolete

>diff --git a/net/ipv4/ip_gre.c b/net/ipv4/ip_gre.c
>index 0f9b3a3..cfd1b09 100644
>--- a/net/ipv4/ip_gre.c
>+++ b/net/ipv4/ip_gre.c
>@@ -894,6 +894,59 @@ tx_error:
> 	return 0;
> }
> 
>+static void ipgre_tunnel_bind_dev(struct net_device *dev)
>+{
>+	struct net_device *tdev = NULL;
>+	struct ip_tunnel *tunnel;
>+	struct iphdr *iph;
>+	int hlen = LL_MAX_HEADER;
>+	int mtu = ETH_DATA_LEN;
>+	int addend = sizeof(struct iphdr) + 4;
>+
>+	tunnel = netdev_priv(dev);
>+	iph = &tunnel->parms.iph;
>+
>+	/* Guess output device to choose reasonable mtu and hard_header_len */
>+
>+	if (iph->daddr) {
>+		struct flowi fl = { .oif = tunnel->parms.link,
>+				    .nl_u = { .ip4_u =
>+					      { .daddr = iph->daddr,
>+						.saddr = iph->saddr,
>+						.tos = RT_TOS(iph->tos) } },
>+				    .proto = IPPROTO_GRE };
>+		struct rtable *rt;
>+		if (!ip_route_output_key(&rt, &fl)) {
>+			tdev = rt->u.dst.dev;
>+			ip_rt_put(rt);
>+		}
>+		dev->flags |= IFF_POINTOPOINT;
>+	}
>+
>+	if (!tdev && tunnel->parms.link)
>+		tdev = __dev_get_by_index(tunnel->parms.link);
>+
>+	if (tdev) {
>+		hlen = tdev->hard_header_len;
>+		mtu = tdev->mtu;
>+	}
>+	dev->iflink = tunnel->parms.link;
>+
>+	/* Precalculate GRE options length */
>+	if (tunnel->parms.o_flags&(GRE_CSUM|GRE_KEY|GRE_SEQ)) {
>+		if (tunnel->parms.o_flags&GRE_CSUM)
>+			addend += 4;
>+		if (tunnel->parms.o_flags&GRE_KEY)
>+			addend += 4;
>+		if (tunnel->parms.o_flags&GRE_SEQ)
>+			addend += 4;
>+	}
>+	dev->hard_header_len = hlen + addend;
>+	dev->mtu = mtu - addend;
>+	tunnel->hlen = addend;
>+
>+}
>+
> static int
> ipgre_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> {
>@@ -981,6 +1034,11 @@ ipgre_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> 				t->parms.iph.ttl = p.iph.ttl;
> 				t->parms.iph.tos = p.iph.tos;
> 				t->parms.iph.frag_off = p.iph.frag_off;
>+				if (t->parms.link != p.link) {
>+					t->parms.link = p.link;
>+					ipgre_tunnel_bind_dev(dev);
>+					netdev_state_change(dev);
>+				}
> 			}
> 			if (copy_to_user(ifr->ifr_ifru.ifru_data, &t->parms, sizeof(p)))
> 				err = -EFAULT;
>@@ -1147,12 +1205,8 @@ static void ipgre_tunnel_setup(struct net_device *dev)
> 
> static int ipgre_tunnel_init(struct net_device *dev)
> {
>-	struct net_device *tdev = NULL;
> 	struct ip_tunnel *tunnel;
> 	struct iphdr *iph;
>-	int hlen = LL_MAX_HEADER;
>-	int mtu = ETH_DATA_LEN;
>-	int addend = sizeof(struct iphdr) + 4;
> 
> 	tunnel = netdev_priv(dev);
> 	iph = &tunnel->parms.iph;
>@@ -1163,23 +1217,9 @@ static int ipgre_tunnel_init(struct net_device *dev)
> 	memcpy(dev->dev_addr, &tunnel->parms.iph.saddr, 4);
> 	memcpy(dev->broadcast, &tunnel->parms.iph.daddr, 4);
> 
>-	/* Guess output device to choose reasonable mtu and hard_header_len */
>+	ipgre_tunnel_bind_dev(dev);
> 
> 	if (iph->daddr) {
>-		struct flowi fl = { .oif = tunnel->parms.link,
>-				    .nl_u = { .ip4_u =
>-					      { .daddr = iph->daddr,
>-						.saddr = iph->saddr,
>-						.tos = RT_TOS(iph->tos) } },
>-				    .proto = IPPROTO_GRE };
>-		struct rtable *rt;
>-		if (!ip_route_output_key(&rt, &fl)) {
>-			tdev = rt->u.dst.dev;
>-			ip_rt_put(rt);
>-		}
>-
>-		dev->flags |= IFF_POINTOPOINT;
>-
> #ifdef CONFIG_NET_IPGRE_BROADCAST
> 		if (MULTICAST(iph->daddr)) {
> 			if (!iph->saddr)
>@@ -1192,27 +1232,6 @@ static int ipgre_tunnel_init(struct net_device *dev)
> #endif
> 	}
> 
>-	if (!tdev && tunnel->parms.link)
>-		tdev = __dev_get_by_index(tunnel->parms.link);
>-
>-	if (tdev) {
>-		hlen = tdev->hard_header_len;
>-		mtu = tdev->mtu;
>-	}
>-	dev->iflink = tunnel->parms.link;
>-
>-	/* Precalculate GRE options length */
>-	if (tunnel->parms.o_flags&(GRE_CSUM|GRE_KEY|GRE_SEQ)) {
>-		if (tunnel->parms.o_flags&GRE_CSUM)
>-			addend += 4;
>-		if (tunnel->parms.o_flags&GRE_KEY)
>-			addend += 4;
>-		if (tunnel->parms.o_flags&GRE_SEQ)
>-			addend += 4;
>-	}
>-	dev->hard_header_len = hlen + addend;
>-	dev->mtu = mtu - addend;
>-	tunnel->hlen = addend;
> 	return 0;
> }
> 
>diff --git a/net/ipv4/ipip.c b/net/ipv4/ipip.c
>index 76ab50b..4e7a6bf 100644
>--- a/net/ipv4/ipip.c
>+++ b/net/ipv4/ipip.c
>@@ -656,6 +656,40 @@ tx_error:
> 	return 0;
> }
> 
>+static void ipip_tunnel_bind_dev(struct net_device *dev)
>+{
>+	struct net_device *tdev = NULL;
>+	struct ip_tunnel *tunnel;
>+	struct iphdr *iph;
>+
>+	tunnel = netdev_priv(dev);
>+	iph = &tunnel->parms.iph;
>+
>+	if (iph->daddr) {
>+		struct flowi fl = { .oif = tunnel->parms.link,
>+				    .nl_u = { .ip4_u =
>+					      { .daddr = iph->daddr,
>+						.saddr = iph->saddr,
>+						.tos = RT_TOS(iph->tos) } },
>+				    .proto = IPPROTO_IPIP };
>+		struct rtable *rt;
>+		if (!ip_route_output_key(&rt, &fl)) {
>+			tdev = rt->u.dst.dev;
>+			ip_rt_put(rt);
>+		}
>+		dev->flags |= IFF_POINTOPOINT;
>+	}
>+
>+	if (!tdev && tunnel->parms.link)
>+		tdev = __dev_get_by_index(tunnel->parms.link);
>+
>+	if (tdev) {
>+		dev->hard_header_len = tdev->hard_header_len + sizeof(struct iphdr);
>+		dev->mtu = tdev->mtu - sizeof(struct iphdr);
>+	}
>+	dev->iflink = tunnel->parms.link;
>+}
>+
> static int
> ipip_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> {
>@@ -728,6 +762,11 @@ ipip_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> 				t->parms.iph.ttl = p.iph.ttl;
> 				t->parms.iph.tos = p.iph.tos;
> 				t->parms.iph.frag_off = p.iph.frag_off;
>+				if (t->parms.link != p.link) {
>+					t->parms.link = p.link;
>+					ipip_tunnel_bind_dev(dev);
>+					netdev_state_change(dev);
>+				}
> 			}
> 			if (copy_to_user(ifr->ifr_ifru.ifru_data, &t->parms, sizeof(p)))
> 				err = -EFAULT;
>@@ -796,12 +835,9 @@ static void ipip_tunnel_setup(struct net_device *dev)
> 
> static int ipip_tunnel_init(struct net_device *dev)
> {
>-	struct net_device *tdev = NULL;
> 	struct ip_tunnel *tunnel;
>-	struct iphdr *iph;
> 
> 	tunnel = netdev_priv(dev);
>-	iph = &tunnel->parms.iph;
> 
> 	tunnel->dev = dev;
> 	strcpy(tunnel->parms.name, dev->name);
>@@ -809,29 +845,7 @@ static int ipip_tunnel_init(struct net_device *dev)
> 	memcpy(dev->dev_addr, &tunnel->parms.iph.saddr, 4);
> 	memcpy(dev->broadcast, &tunnel->parms.iph.daddr, 4);
> 
>-	if (iph->daddr) {
>-		struct flowi fl = { .oif = tunnel->parms.link,
>-				    .nl_u = { .ip4_u =
>-					      { .daddr = iph->daddr,
>-						.saddr = iph->saddr,
>-						.tos = RT_TOS(iph->tos) } },
>-				    .proto = IPPROTO_IPIP };
>-		struct rtable *rt;
>-		if (!ip_route_output_key(&rt, &fl)) {
>-			tdev = rt->u.dst.dev;
>-			ip_rt_put(rt);
>-		}
>-		dev->flags |= IFF_POINTOPOINT;
>-	}
>-
>-	if (!tdev && tunnel->parms.link)
>-		tdev = __dev_get_by_index(tunnel->parms.link);
>-
>-	if (tdev) {
>-		dev->hard_header_len = tdev->hard_header_len + sizeof(struct iphdr);
>-		dev->mtu = tdev->mtu - sizeof(struct iphdr);
>-	}
>-	dev->iflink = tunnel->parms.link;
>+	ipip_tunnel_bind_dev(dev);
> 
> 	return 0;
> }
>diff --git a/net/ipv6/sit.c b/net/ipv6/sit.c
>index 836eecd..d9b2169 100644
>--- a/net/ipv6/sit.c
>+++ b/net/ipv6/sit.c
>@@ -591,6 +591,42 @@ tx_error:
> 	return 0;
> }
> 
>+static void ipip6_tunnel_bind_dev(struct net_device *dev)
>+{
>+	struct net_device *tdev = NULL;
>+	struct ip_tunnel *tunnel;
>+	struct iphdr *iph;
>+
>+	tunnel = netdev_priv(dev);
>+	iph = &tunnel->parms.iph;
>+
>+	if (iph->daddr) {
>+		struct flowi fl = { .nl_u = { .ip4_u =
>+					      { .daddr = iph->daddr,
>+						.saddr = iph->saddr,
>+						.tos = RT_TOS(iph->tos) } },
>+				    .oif = tunnel->parms.link,
>+				    .proto = IPPROTO_IPV6 };
>+		struct rtable *rt;
>+		if (!ip_route_output_key(&rt, &fl)) {
>+			tdev = rt->u.dst.dev;
>+			ip_rt_put(rt);
>+		}
>+		dev->flags |= IFF_POINTOPOINT;
>+	}
>+
>+	if (!tdev && tunnel->parms.link)
>+		tdev = __dev_get_by_index(tunnel->parms.link);
>+
>+	if (tdev) {
>+		dev->hard_header_len = tdev->hard_header_len + sizeof(struct iphdr);
>+		dev->mtu = tdev->mtu - sizeof(struct iphdr);
>+		if (dev->mtu < IPV6_MIN_MTU)
>+			dev->mtu = IPV6_MIN_MTU;
>+	}
>+	dev->iflink = tunnel->parms.link;
>+}
>+
> static int
> ipip6_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> {
>@@ -662,6 +698,11 @@ ipip6_tunnel_ioctl (struct net_device *dev, struct ifreq *ifr, int cmd)
> 			if (cmd == SIOCCHGTUNNEL) {
> 				t->parms.iph.ttl = p.iph.ttl;
> 				t->parms.iph.tos = p.iph.tos;
>+				if (t->parms.link != p.link) {
>+					t->parms.link = p.link;
>+					ipip6_tunnel_bind_dev(dev);
>+					netdev_state_change(dev);
>+				}
> 			}
> 			if (copy_to_user(ifr->ifr_ifru.ifru_data, &t->parms, sizeof(p)))
> 				err = -EFAULT;
>@@ -730,12 +771,9 @@ static void ipip6_tunnel_setup(struct net_device *dev)
> 
> static int ipip6_tunnel_init(struct net_device *dev)
> {
>-	struct net_device *tdev = NULL;
> 	struct ip_tunnel *tunnel;
>-	struct iphdr *iph;
> 
> 	tunnel = netdev_priv(dev);
>-	iph = &tunnel->parms.iph;
> 
> 	tunnel->dev = dev;
> 	strcpy(tunnel->parms.name, dev->name);
>@@ -743,31 +781,7 @@ static int ipip6_tunnel_init(struct net_device *dev)
> 	memcpy(dev->dev_addr, &tunnel->parms.iph.saddr, 4);
> 	memcpy(dev->broadcast, &tunnel->parms.iph.daddr, 4);
> 
>-	if (iph->daddr) {
>-		struct flowi fl = { .nl_u = { .ip4_u =
>-					      { .daddr = iph->daddr,
>-						.saddr = iph->saddr,
>-						.tos = RT_TOS(iph->tos) } },
>-				    .oif = tunnel->parms.link,
>-				    .proto = IPPROTO_IPV6 };
>-		struct rtable *rt;
>-		if (!ip_route_output_key(&rt, &fl)) {
>-			tdev = rt->u.dst.dev;
>-			ip_rt_put(rt);
>-		}
>-		dev->flags |= IFF_POINTOPOINT;
>-	}
>-
>-	if (!tdev && tunnel->parms.link)
>-		tdev = __dev_get_by_index(tunnel->parms.link);
>-
>-	if (tdev) {
>-		dev->hard_header_len = tdev->hard_header_len + sizeof(struct iphdr);
>-		dev->mtu = tdev->mtu - sizeof(struct iphdr);
>-		if (dev->mtu < IPV6_MIN_MTU)
>-			dev->mtu = IPV6_MIN_MTU;
>-	}
>-	dev->iflink = tunnel->parms.link;
>+	ipip6_tunnel_bind_dev(dev);
> 
> 	return 0;
> }

Actions: View | Diff

Attachments on bug 451196: 309841