Re: [ovs-dev] [PATCH] datapath: An expanded table should be larger than its predecessor

2011-07-12 Thread Ben Pfaff
On Tue, Jul 12, 2011 at 03:52:44PM +0900, Simon Horman wrote: > This resolves what appears to be a think-o in tbl_expand() > > * Old Logic: Always create tables with TBL_MIN_BUCKETS buckets > * New Logic: Create tables twice as big as their predecessor > > When sending 10,000 flows through ovs-vs

Re: [ovs-dev] [PATCH] datapath: An expanded table should be larger than its predecessor

2011-07-12 Thread Jesse Gross
On Mon, Jul 11, 2011 at 11:52 PM, Simon Horman wrote: > This resolves what appears to be a think-o in tbl_expand() > > * Old Logic: Always create tables with TBL_MIN_BUCKETS buckets > * New Logic: Create tables twice as big as their predecessor > > When sending 10,000 flows through ovs-vswitchd: >

[ovs-dev] [PATCH] datapath: An expanded table should be larger than its predecessor

2011-07-11 Thread Simon Horman
This resolves what appears to be a think-o in tbl_expand() * Old Logic: Always create tables with TBL_MIN_BUCKETS buckets * New Logic: Create tables twice as big as their predecessor When sending 10,000 flows through ovs-vswitchd: * Old Logic: CPU bound in tbl_lookup(), significant packet loss *