Can you provide a reference to something that helps me understand why the new value is the "correct" one? This is not my area of expertise and all I learn from the commit message is an assertion that the new value is better.
Thanks, Ben. On Mon, Dec 25, 2017 at 04:59:22PM +0000, Venkatesan Pradeep wrote: > Send BFD packets with TOS value equivalent to DSCP CS6 so that the network > can apply the right QoS for those packets. This can help avoid BFD flaps due > to network congestion. > > Signed-off-by: Venkatesan Pradeep <[email protected]> > --- > lib/bfd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/bfd.c b/lib/bfd.c > index 40cd0be..8d291bb 100644 > --- a/lib/bfd.c > +++ b/lib/bfd.c > @@ -612,7 +612,7 @@ bfd_put_packet(struct bfd *bfd, struct dp_packet *p, > ip->ip_ihl_ver = IP_IHL_VER(5, 4); > ip->ip_tot_len = htons(sizeof *ip + sizeof *udp + sizeof *msg); > ip->ip_ttl = MAXTTL; > - ip->ip_tos = IPTOS_LOWDELAY | IPTOS_THROUGHPUT; > + ip->ip_tos = IPTOS_PREC_INTERNETCONTROL; > ip->ip_proto = IPPROTO_UDP; > put_16aligned_be32(&ip->ip_src, bfd->ip_src); > put_16aligned_be32(&ip->ip_dst, bfd->ip_dst); > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
