From 19739aee088aad011995f5f8e891fb65bd13aa09 Mon Sep 17 00:00:00 2001 From: Justin Pettit Date: Fri, 10 Dec 2010 20:50:58 -0800 Subject: [PATCH] ofproto: Fix problem that caused facets not to be installed into datapath. Commit cdee00f (datapath: Replace "struct odp_action" by Netlink attributes.) stopped initializing some elements in facet structures in certain cases. This caused flows to not be installed into the datapath. This commit sets that again based on the action context. --- ofproto/ofproto.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index 339b9605..255df653 100644 --- a/ofproto/ofproto.c +++ b/ofproto/ofproto.c @@ -2250,6 +2250,9 @@ facet_make_actions(struct ofproto *p, struct facet *facet, action_xlate_ctx_init(&ctx, p, &facet->flow, packet); odp_actions = xlate_actions(&ctx, rule->actions, rule->n_actions); + facet->tags = ctx.tags; + facet->may_install = ctx.may_set_up_flow; + facet->nf_flow.output_iface = ctx.nf_output_iface; if (facet->actions_len != odp_actions->size || memcmp(facet->actions, odp_actions->data, odp_actions->size)) { -- 2.30.2