Commit f8d20b46 authored by Xin Long's avatar Xin Long Committed by David S. Miller

ip6_gre: only increase err_count for some certain type icmpv6 in ip6gre_err

The similar fix in patch 'ipip: only increase err_count for some
certain type icmp in ipip_err' is needed for ip6gre_err.

In Jianlin's case, udp netperf broke even when receiving a TooBig
icmpv6 packet.

Fixes: c12b395a ("gre: Support GRE over IPv6")
Reported-by: default avatarJianlin Shi <jishi@redhat.com>
Signed-off-by: default avatarXin Long <lucien.xin@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f3594f0a
...@@ -408,13 +408,16 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt, ...@@ -408,13 +408,16 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
case ICMPV6_DEST_UNREACH: case ICMPV6_DEST_UNREACH:
net_dbg_ratelimited("%s: Path to destination invalid or inactive!\n", net_dbg_ratelimited("%s: Path to destination invalid or inactive!\n",
t->parms.name); t->parms.name);
if (code != ICMPV6_PORT_UNREACH)
break; break;
return;
case ICMPV6_TIME_EXCEED: case ICMPV6_TIME_EXCEED:
if (code == ICMPV6_EXC_HOPLIMIT) { if (code == ICMPV6_EXC_HOPLIMIT) {
net_dbg_ratelimited("%s: Too small hop limit or routing loop in tunnel!\n", net_dbg_ratelimited("%s: Too small hop limit or routing loop in tunnel!\n",
t->parms.name); t->parms.name);
}
break; break;
}
return;
case ICMPV6_PARAMPROB: case ICMPV6_PARAMPROB:
teli = 0; teli = 0;
if (code == ICMPV6_HDR_FIELD) if (code == ICMPV6_HDR_FIELD)
...@@ -430,7 +433,7 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt, ...@@ -430,7 +433,7 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
net_dbg_ratelimited("%s: Recipient unable to parse tunneled packet!\n", net_dbg_ratelimited("%s: Recipient unable to parse tunneled packet!\n",
t->parms.name); t->parms.name);
} }
break; return;
case ICMPV6_PKT_TOOBIG: case ICMPV6_PKT_TOOBIG:
mtu = be32_to_cpu(info) - offset - t->tun_hlen; mtu = be32_to_cpu(info) - offset - t->tun_hlen;
if (t->dev->type == ARPHRD_ETHER) if (t->dev->type == ARPHRD_ETHER)
...@@ -438,7 +441,7 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt, ...@@ -438,7 +441,7 @@ static void ip6gre_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
if (mtu < IPV6_MIN_MTU) if (mtu < IPV6_MIN_MTU)
mtu = IPV6_MIN_MTU; mtu = IPV6_MIN_MTU;
t->dev->mtu = mtu; t->dev->mtu = mtu;
break; return;
} }
if (time_before(jiffies, t->err_time + IP6TUNNEL_ERR_TIMEO)) if (time_before(jiffies, t->err_time + IP6TUNNEL_ERR_TIMEO))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment