Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping.

2014-06-20 Thread Ryan Wilson 76511
Joe Stringer mailto:joestrin...@nicira.com>> Cc: "dev@openvswitch.org<mailto:dev@openvswitch.org>" mailto:dev@openvswitch.org>> Subject: Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping. Hey Joe, Not sure if you were looking at this patch, but I bel

Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping.

2014-06-20 Thread Ryan Wilson 76511
PM To: Joe Stringer mailto:joestrin...@nicira.com>> Cc: "dev@openvswitch.org<mailto:dev@openvswitch.org>" mailto:dev@openvswitch.org>> Subject: Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping. Hi Ryan, a few broad comments: Could you split this in

Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping.

2014-06-19 Thread Ryan Wilson
> Hi Ryan, a few broad comments: > > Could you split this into two patches, the dpif interface change and the > dpif-netdev implementation change? > > Sure will do. > I'm a little uneasy about the change to the dpif_flow_dump_next() > interface, just because the size of the output parameter was p

Re: [ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping.

2014-06-19 Thread Joe Stringer
Hi Ryan, a few broad comments: Could you split this into two patches, the dpif interface change and the dpif-netdev implementation change? I'm a little uneasy about the change to the dpif_flow_dump_next() interface, just because the size of the output parameter was previously paired with the outp

[ovs-dev] [PATCH] dpif-netdev: Implement batched flow dumping.

2014-06-18 Thread Ryan Wilson
Previously, flows were retrieved one by one when dumping flows for datapaths of type 'netdev'. This increased contention for the dump's mutex, negatively affecting revalidator performance. This patch retrieves batches of flows when dumping flows for datapaths of type 'netdev'. Signed-off-by: Ryan