Skip to content

Commit

Permalink
vti6: Remove caching of flow informations.
Browse files Browse the repository at this point in the history
Unlike ip6_tunnel, vti6 does not use the the tunnel
endpoint addresses to do route and xfrm lookups.
So no need to cache the flow informations. It also
does not make sense to calculate the mtu based on
such flow informations, so remove this too.

Signed-off-by: Steffen Klassert <[email protected]>
  • Loading branch information
klassert committed Mar 14, 2014
1 parent 7c85258 commit 7cf9fdb
Showing 1 changed file with 0 additions and 31 deletions.
31 changes: 0 additions & 31 deletions net/ipv6/ip6_vti.c
Original file line number Diff line number Diff line change
Expand Up @@ -443,19 +443,10 @@ static void vti6_link_config(struct ip6_tnl *t)
struct dst_entry *dst;
struct net_device *dev = t->dev;
struct __ip6_tnl_parm *p = &t->parms;
struct flowi6 *fl6 = &t->fl.u.ip6;

memcpy(dev->dev_addr, &p->laddr, sizeof(struct in6_addr));
memcpy(dev->broadcast, &p->raddr, sizeof(struct in6_addr));

/* Set up flowi template */
fl6->saddr = p->laddr;
fl6->daddr = p->raddr;
fl6->flowi6_oif = p->link;
fl6->flowi6_mark = be32_to_cpu(p->i_key);
fl6->flowi6_proto = p->proto;
fl6->flowlabel = 0;

p->flags &= ~(IP6_TNL_F_CAP_XMIT | IP6_TNL_F_CAP_RCV |
IP6_TNL_F_CAP_PER_PACKET);
p->flags |= ip6_tnl_get_cap(t, &p->laddr, &p->raddr);
Expand All @@ -466,28 +457,6 @@ static void vti6_link_config(struct ip6_tnl *t)
dev->flags &= ~IFF_POINTOPOINT;

dev->iflink = p->link;

if (p->flags & IP6_TNL_F_CAP_XMIT) {

dst = ip6_route_output(dev_net(dev), NULL, fl6);
if (dst->error)
return;

dst = xfrm_lookup(dev_net(dev), dst, flowi6_to_flowi(fl6),
NULL, 0);
if (IS_ERR(dst))
return;

if (dst->dev) {
dev->hard_header_len = dst->dev->hard_header_len;

dev->mtu = dst_mtu(dst);

if (dev->mtu < IPV6_MIN_MTU)
dev->mtu = IPV6_MIN_MTU;
}
dst_release(dst);
}
}

/**
Expand Down

0 comments on commit 7cf9fdb

Please sign in to comment.