[ovs-dev] [PATCHv2] ofproto: Add support for Openflow group and bucket stats.
Ryan Wilson 76511
wryan at vmware.com
Mon May 19 20:42:22 UTC 2014
We should use Andy's patch for the tests since I didn't write the tests:
I'll split the group RCU and the crediting stats parts of this commit on my next version.
----- Original Message -----
From: "Alex Wang" <alexw at nicira.com>
To: "Ryan Wilson" <wryan at nicira.com>
Cc: dev at openvswitch.org
Sent: Monday, May 19, 2014 12:48:07 PM
Subject: Re: [ovs-dev] [PATCHv2] ofproto: Add support for Openflow group and bucket stats.
Still high level comments first,
Could you break the patch into three? (your change about crediting the group stats, Andy's test, and use RCU for group configuration)
The group stats crediting part of code looks good to me.
The test makes sense.
I haven't reviewed the "rcu for group configuration", and will wait after your new post.
dev mailing list
dev at openvswitch.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the dev