From 5a046d3b09a158c295a6734a0fe7fac1eb21cc02 Mon Sep 17 00:00:00 2001 From: Yi-Hung Wei Date: Wed, 14 Mar 2018 11:16:41 -0700 Subject: [PATCH] ofproto-dpif-xlate: Report ct fields changes in ofproto/trace With commit f6fabcc6 ("ofproto-dpif: Mark packets as "untracked" after call to ct()", after the ct() action, the packet conntrack state is set to an untracked state, and all the conntrack fields are cleared. This patch updates ofproto/trace report to reflect this change, so that it would be easier to debug OpenFlow pipeline with conntrack. Signed-off-by: Yi-Hung Wei Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif-xlate.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index a00bed70480..42ac1186fb3 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -5754,6 +5754,8 @@ compose_conntrack_action(struct xlate_ctx *ctx, struct ofpact_conntrack *ofc, /* The ct_* fields are only available in the scope of the 'recirc_table' * call chain. */ flow_clear_conntrack(&ctx->xin->flow); + xlate_report(ctx, OFT_DETAIL, "Sets the packet to an untracked state, " + "and clears all the conntrack fields."); ctx->conntracked = false; }