Great, thanks.
On Thu, Jun 21, 2012 at 09:19:01PM -0700, Mehak Mahajan wrote: > Hey Ben, > > Thanks for the review and for retesting this. > I pushed this to master in the morning with changes to the commit message. > > thanx! > mehak > > On Thu, Jun 21, 2012 at 11:30 AM, Mehak Mahajan <mmaha...@nicira.com> wrote: > > > sure will do that.. > > > > > > On Thu, Jun 21, 2012 at 10:12 AM, Ben Pfaff <b...@nicira.com> wrote: > > > >> On Wed, Jun 20, 2012 at 09:24:34PM -0700, Mehak Mahajan wrote: > >> > This patch reapplies the changes that were reverted with the commit > >> 59efa47 > >> > (Revert DSCP update changes.). It also addresses the problem introduced > >> by > >> > the original commits, cd8fca2 ((jsonrpc: Correctly setting the dscp > >> value > >> > before reconnect.) and b2e18d (No need to restart DB / OVS on changing > >> > dscp value.), that caused numerous unit test failures on some systems > >> (as > >> > diagnosed by valgrind). > >> > > >> > Signed-off-by: Mehak Mahajan <mmaha...@nicira.com> > >> > >> I tested it and it worked for me. Thank you. > >> > >> Please update the commit message, though, to include everything from > >> the original commit message, rather than just referring to it. > >> > >> Thanks, > >> > >> Ben. > >> > > > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
