ofproto: Fix problem that caused facets not to be installed into datapath.
authorJustin Pettit <jpettit@nicira.com>
Sat, 11 Dec 2010 04:50:58 +0000 (20:50 -0800)
committerJustin Pettit <jpettit@nicira.com>
Sat, 11 Dec 2010 06:20:14 +0000 (22:20 -0800)
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

index 339b9605dbf4f3198a834f2f283934e8ff0c01cb..255df6534ce7d8b033793526ea7ccb989532f3aa 100644 (file)
@@ -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)) {