Re: [ovs-dev] [PATCH] ofproto-dpif: More efficient implementation of get_odp_port().

2013-01-09 Thread Ben Pfaff
On Wed, Jan 09, 2013 at 10:25:14PM +0200, Jarno Rajahalme wrote: > On Jan 9, 2013, at 20:30 , ext Ben Pfaff wrote: > > > On Wed, Jan 09, 2013 at 02:21:47PM +0200, Jarno Rajahalme wrote: > >> Avoids unnecessary hash bucket traversal (ofproto_get_port()), since we > >> have the ofport_dpif already a

Re: [ovs-dev] [PATCH] ofproto-dpif: More efficient implementation of get_odp_port().

2013-01-09 Thread Jarno Rajahalme
On Jan 9, 2013, at 20:30 , ext Ben Pfaff wrote: > On Wed, Jan 09, 2013 at 02:21:47PM +0200, Jarno Rajahalme wrote: >> Avoids unnecessary hash bucket traversal (ofproto_get_port()), since we >> have the ofport_dpif already after odp_port_to_ofport(). >> >> Signed-off-by: Jarno Rajahalme > > Than

Re: [ovs-dev] [PATCH] ofproto-dpif: More efficient implementation of get_odp_port().

2013-01-09 Thread Ben Pfaff
On Wed, Jan 09, 2013 at 02:21:47PM +0200, Jarno Rajahalme wrote: > Avoids unnecessary hash bucket traversal (ofproto_get_port()), since we > have the ofport_dpif already after odp_port_to_ofport(). > > Signed-off-by: Jarno Rajahalme Thanks! I was able to simplify the test to eliminate the ofpro

[ovs-dev] [PATCH] ofproto-dpif: More efficient implementation of get_odp_port().

2013-01-09 Thread Jarno Rajahalme
Avoids unnecessary hash bucket traversal (ofproto_get_port()), since we have the ofport_dpif already after odp_port_to_ofport(). Signed-off-by: Jarno Rajahalme --- ofproto/ofproto-dpif.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto