Re: [ovs-dev] [xlate 1.11 9/9] ofproto-dpif: Revamp xlate_actions() interface.

2013-05-30 Thread Ethan Jackson
> > orig_tunnel_ip_dst is not used in 1.11 yet, so it should be deleted from this > patch. > There is only one use of it later in the patch, which should also be removed. > ... That's why I wanted someone to peek at it =). Thanks Jarno, I'll delete it. Ethan >> >> - >> -/* Translates the 'ofpa

Re: [ovs-dev] [xlate 1.11 9/9] ofproto-dpif: Revamp xlate_actions() interface.

2013-05-30 Thread Rajahalme, Jarno (NSN - FI/Espoo)
On May 30, 2013, at 0:24 , ext Ethan Jackson wrote: > +/* Context used by xlate_actions() and its callees. */ > +struct xlate_ctx { > +struct xlate_in *xin; > +struct xlate_out *xout; > > -struct ofpbuf *odp_actions; /* Datapath actions. */ > -tag_type tags; /* Tags

Re: [ovs-dev] [xlate 1.11 9/9] ofproto-dpif: Revamp xlate_actions() interface.

2013-05-29 Thread Ben Pfaff
On Wed, May 29, 2013 at 02:24:41PM -0700, Ethan Jackson wrote: > This patch implements a new interface to xlate_actions which, aside > from being simpler and more intuitive, achieves several goals. It > pulls all of xlate_actions() results into a single structure which > will be easier to cache an