On Wed, Jun 13, 2012 at 01:40:19PM +0900, Simon Horman wrote: > On Tue, Jun 12, 2012 at 09:27:51PM -0700, Ben Pfaff wrote: > > On Wed, Jun 13, 2012 at 09:25:19AM +0900, Simon Horman wrote: > > > On Tue, Jun 12, 2012 at 12:32:14AM -0700, Ben Pfaff wrote: > > > > "enum ofperr" is clearer than "int". > > > > > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > > > > > Reviewed-by: Simon Horman <ho...@verge.net.au> > > > > Thanks for all the reviews. I pushed patches 1 through 10 to master. > > I'll wait for more reviews (and possibly do some more work on the > > remaining commits) before I push more patches. > > Thanks. I am reasonably comfortable with 1-15 at this point, > but I'm happy with a more cautious approach too.
In particular I didn't see a followup to #12, so I'm not sure whether you read it. That's also the biggest change and the most likely to be wrong, so I'll probably seek a second review on it in any case. Thanks, Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
