> On Sep 13, 2015, at 1:58 PM, Alex Wang <[email protected]> wrote: > > --- a/ovn/controller-vtep/gateway.c > +++ b/ovn/controller-vtep/gateway.c > @@ -189,7 +189,8 @@ gateway_run(struct controller_vtep_ctx *ctx) > } > > /* Destroys the chassis table entries for vtep physical switches. > - * Returns true when all done. */ > + * Returns true when done (i.e. there is no change made to 'ovnsb_idl'), > + * otherwise returns false. */
My only comment is that I'd use "ctx->ovnsb_idl". I think it's ready to merge. Acked-by: Justin Pettit <[email protected]> --Justin _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
