Looks good, Ethan
On Tue, Dec 27, 2011 at 13:23, Ben Pfaff <[email protected]> wrote: > Without logging of operation groups, it becomes more difficult to debug > problems related to flow setups, since those go through operation groups. > > Signed-off-by: Ben Pfaff <[email protected]> > --- > lib/dpif.c | 14 ++++++++++++++ > 1 files changed, 14 insertions(+), 0 deletions(-) > > diff --git a/lib/dpif.c b/lib/dpif.c > index 3ec83a2..54a5d4e 100644 > --- a/lib/dpif.c > +++ b/lib/dpif.c > @@ -991,6 +991,20 @@ dpif_operate(struct dpif *dpif, struct dpif_op **ops, > size_t n_ops) > > if (dpif->dpif_class->operate) { > dpif->dpif_class->operate(dpif, ops, n_ops); > + > + for (i = 0; i < n_ops; i++) { > + struct dpif_op *op = ops[i]; > + > + switch (op->type) { > + case DPIF_OP_FLOW_PUT: > + log_flow_put_message(dpif, &op->u.flow_put, op->error); > + break; > + > + case DPIF_OP_EXECUTE: > + log_execute_message(dpif, &op->u.execute, op->error); > + break; > + } > + } > return; > } > > -- > 1.7.2.5 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
