On Tue, Jun 12, 2012 at 09:42:10PM -0700, Ben Pfaff wrote: > 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.
Ah, sorry, that had slipped my mind. I have looked over #12 but I'm not feeling particularly confident about it, partly due to its size. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
