We really need to get rid of this compose_slow_path business, but for now this seems fine to me. Thanks
Acked-by: Ethan Jackson <et...@nicira.com> On Tue, Jun 11, 2013 at 11:00 PM, Justin Pettit <jpet...@nicira.com> wrote: > It's often helpful to see what the slow path actions actual are. Print > them when "ovs-appctl dpif/dump-megaflows" is called. > > Signed-off-by: Justin Pettit <jpet...@nicira.com> > --- > ofproto/ofproto-dpif.c | 15 +++++++++++++-- > 1 files changed, 13 insertions(+), 2 deletions(-) > > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c > index c0f6df0..c262e7b 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -8583,8 +8583,19 @@ ofproto_unixctl_dpif_dump_megaflows(struct > unixctl_conn *conn, > ds_put_format(&ds, "n_subfacets:%zu, ", > list_size(&facet->subfacets)); > ds_put_format(&ds, "used:%.3fs, ", (now - facet->used) / 1000.0); > ds_put_cstr(&ds, "Datapath actions: "); > - format_odp_actions(&ds, facet->xout.odp_actions.data, > - facet->xout.odp_actions.size); > + if (facet->xout.slow) { > + uint64_t slow_path_stub[128 / 8]; > + const struct nlattr *actions; > + size_t actions_len; > + > + compose_slow_path(ofproto, &facet->flow, facet->xout.slow, > + slow_path_stub, sizeof slow_path_stub, > + &actions, &actions_len); > + format_odp_actions(&ds, actions, actions_len); > + } else { > + format_odp_actions(&ds, facet->xout.odp_actions.data, > + facet->xout.odp_actions.size); > + } > ds_put_cstr(&ds, "\n"); > } > > -- > 1.7.5.4 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev