On Fri, Aug 12, 2016 at 12:21:28PM +0530, bscha...@redhat.com wrote: > From: Babu Shanmugam <bscha...@redhat.com> > > ovn-northd sets 'ip.dscp' to the DSCP value > > Signed-off-by: Babu Shanmugam <bscha...@redhat.com>
Thanks for the revision (and for adding a test)! This comment in ovn-northd seems like an incomplete thought (and what's a "packed"? A packet? A port?): > + /* If DSCP is enabled on this packed, add another flow at a > higher */ The big comment on build_port_security_ip() should probably describe the new behavior, as should ovn-northd(8). The documentation talks about applying DSCP to packets egressing an interface but build_port_security_ip() seems to actually apply DSCP to packets ingressing an interface. Thanks, Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
