> On Jun 23, 2015, at 11:46 AM, Ben Pfaff <b...@nicira.com> wrote: > > On Tue, Jun 23, 2015 at 12:02:44AM +0000, Nithin Raju wrote: >>> On Jun 22, 2015, at 10:58 AM, Sorin Vinturis >>> <svintu...@cloudbasesolutions.com> wrote: >>> >>> Hi Nithin, >>> >>> I didn't thought about your idea. I have added instrumentation code in >>> order to force the EEXIST error and test the patch and the code performs as >>> expected. The extension is able to be activated, in case of both the >>> sublayer and callout are already present, and everything works as fine. >> >> Sounds good. I guess when we cannot replicate real failures, we need to get >> creative to test the code added :) > > This sounded like a positive review, so I applied this to master and > branch-2.4.
Thanks Ben. For some reason, I thought this was already applied and hence didn’t ACK. Yes, we are good to go with this patch. -- Nithin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
