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
