From: Justin Pettit <jpettit@nicira.com>
Date: Sat, 11 Dec 2010 04:50:58 +0000 (-0800)
Subject: ofproto: Fix problem that caused facets not to be installed into datapath.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=19739aee088aad011995f5f8e891fb65bd13aa09;p=openvswitch

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.
---

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)) {