Re: [ovs-dev] [PATCH 1/4] ofproto/xlate: Add xlate_lookup_ofproto().

2014-08-22 Thread Jarno Rajahalme
Series pushed, thanks for the reviews! Jarno On Aug 22, 2014, at 11:15 AM, Ben Pfaff wrote: > On Fri, Aug 15, 2014 at 04:38:46PM -0700, Jarno Rajahalme wrote: >> "xlate_receive" did not tell much about what it is used for. We have >> two users of it that only want the ofproto and the OF port

Re: [ovs-dev] [PATCH 1/4] ofproto/xlate: Add xlate_lookup_ofproto().

2014-08-22 Thread Jarno Rajahalme
On Aug 22, 2014, at 11:15 AM, Ben Pfaff wrote: > On Fri, Aug 15, 2014 at 04:38:46PM -0700, Jarno Rajahalme wrote: >> "xlate_receive" did not tell much about what it is used for. We have >> two users of it that only want the ofproto and the OF port number, use the >> new xlate_lookup_ofproto() f

Re: [ovs-dev] [PATCH 1/4] ofproto/xlate: Add xlate_lookup_ofproto().

2014-08-22 Thread Ben Pfaff
On Fri, Aug 15, 2014 at 04:38:46PM -0700, Jarno Rajahalme wrote: > "xlate_receive" did not tell much about what it is used for. We have > two users of it that only want the ofproto and the OF port number, use the > new xlate_lookup_ofproto() for those. > > Fix the comments of xlate_receive() as w

[ovs-dev] [PATCH 1/4] ofproto/xlate: Add xlate_lookup_ofproto().

2014-08-15 Thread Jarno Rajahalme
"xlate_receive" did not tell much about what it is used for. We have two users of it that only want the ofproto and the OF port number, use the new xlate_lookup_ofproto() for those. Fix the comments of xlate_receive() as we no longer change the flow. Also, the helper ofproto_dpif_contains_flow()