[ovs-dev] [PATCH] ofp-util: Properly handle "tun_id"s in tun_id_from_cookie flows.

Ben Pfaff blp at nicira.com
Tue Apr 19 18:58:42 UTC 2011


Thanks, I pushed this.

On Mon, Apr 18, 2011 at 10:51:23AM -0700, Ethan Jackson wrote:
> Looks Good.
> 
> Ethan
> 
> On Mon, Apr 18, 2011 at 10:13 AM, Ben Pfaff <blp at nicira.com> wrote:
> > Just setting the tun_id field isn't enough--it's also necessary to set
> > the tun_id_mask. ?Otherwise the call to cls_rule_zero_wildcarded_fields()
> > at the end of ofputil_cls_rule_from_match() will zero out the tun_id again.
> >
> > This was broken by commit 8368c090cab "Implement arbitrary bitwise masks
> > for tun_id field" back in January. ?(This makes me wonder whether we can
> > drop support for tun_id_from_cookie now.)
> >
> > Reported-by: Dan Wendlandt <dan at nicira.com>



More information about the dev mailing list