[ovs-dev] [PATCH] tunnel: Add to nw_tos bits instead of replacing them in tnl_port_send().
Ben Pfaff
blp at nicira.com
Thu Oct 23 15:29:41 UTC 2014
Thanks, applied.
On Wed, Oct 22, 2014 at 03:03:54PM -0700, Justin Pettit wrote:
> Acked-by: Justin Pettit <jpettit at nicira.com>
>
> --Justin
>
>
> > On Oct 22, 2014, at 2:58 PM, Ben Pfaff <blp at nicira.com> wrote:
> >
> > We normally only add 1-bits to wc->masks for datapath flow matching
> > purposes, never removing them. In this case, the bits that get set to
> > zero will be set back to 1 later on in the function, so this does not fix
> > any actual bug, but the principle of only setting to 1, not to 0, seems
> > sound to me.
> >
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > ofproto/tunnel.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/ofproto/tunnel.c b/ofproto/tunnel.c
> > index 46b0719..a9eecff 100644
> > --- a/ofproto/tunnel.c
> > +++ b/ofproto/tunnel.c
> > @@ -405,7 +405,7 @@ tnl_port_send(const struct ofport_dpif *ofport, struct flow *flow,
> > }
> >
> > if (cfg->tos_inherit && is_ip_any(flow)) {
> > - wc->masks.nw_tos = IP_DSCP_MASK;
> > + wc->masks.nw_tos |= IP_DSCP_MASK;
> > flow->tunnel.ip_tos = flow->nw_tos & IP_DSCP_MASK;
> > } else {
> > flow->tunnel.ip_tos = cfg->tos;
> > --
> > 1.7.10.4
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
>
More information about the dev
mailing list