|
On Wed, Jul 16, 2014 at 12:36:38AM +0300, Julian Anastasov wrote:
>
> Hello,
>
> On Tue, 15 Jul 2014, Alex Gartrell wrote:
>
> > Previously, only the four high bits of the tclass were maintained in the
> > ipv6 case. This matches the behavior of ipv4, though whether or not we
> > should reflect ECN bits may be up for debate.
>
> Agreed, RFC 3168 9.1 has some recommendations
> and more changes may be needed, eg. by using
> INET_ECN_encapsulate as in other tunnels:
>
> ip6_flow_hdr(iph, INET_ECN_encapsulate(inner_dsfield, inner_dsfield), 0);
>
> > Signed-off-by: Alex Gartrell <agartrell@xxxxxx>
>
> So far,
>
> Acked-by: Julian Anastasov <ja@xxxxxx>
Hi Alex,
Somehow this patch did not reach my inbox.
Could you repost it with Julian's ack and me CCed?
> > ---
> > net/netfilter/ipvs/ip_vs_xmit.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/net/netfilter/ipvs/ip_vs_xmit.c
> > b/net/netfilter/ipvs/ip_vs_xmit.c
> > index c47444e..28ee592 100644
> > --- a/net/netfilter/ipvs/ip_vs_xmit.c
> > +++ b/net/netfilter/ipvs/ip_vs_xmit.c
> > @@ -967,8 +967,8 @@ ip_vs_tunnel_xmit_v6(struct sk_buff *skb, struct
> > ip_vs_conn *cp,
> > iph->nexthdr = IPPROTO_IPV6;
> > iph->payload_len = old_iph->payload_len;
> > be16_add_cpu(&iph->payload_len, sizeof(*old_iph));
> > - iph->priority = old_iph->priority;
> > memset(&iph->flow_lbl, 0, sizeof(iph->flow_lbl));
> > + ipv6_change_dsfield(iph, 0, ipv6_get_dsfield(old_iph));
> > iph->daddr = cp->daddr.in6;
> > iph->saddr = saddr;
> > iph->hop_limit = old_iph->hop_limit;
> > --
> > 1.8.1
>
> Regards
>
> --
> Julian Anastasov <ja@xxxxxx>
> --
> To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
