ofproto: Fix test for installability change when revalidating.
authorBen Pfaff <blp@nicira.com>
Fri, 7 Jan 2011 19:15:41 +0000 (11:15 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 28 Jan 2011 05:08:39 +0000 (21:08 -0800)
ctx.may_set_up_flow will become the new facet->may_install, so it's that
value that we should check instead of the current facet->may_install when
deciding whether to update the datapath flow.

Reviewed by Ethan Jackson <ethan@nicira.com>.

ofproto/ofproto.c

index a0dca356dc0979c775d613040e08dcc8ae427a32..dd492258318031ccd3fb78d9cffb317bbb9e624d 100644 (file)
@@ -2499,8 +2499,8 @@ facet_revalidate(struct ofproto *ofproto, struct facet *facet)
 
     /* If the ODP actions changed or the installability changed, then we need
      * to talk to the datapath. */
-    if (actions_changed || facet->may_install != facet->installed) {
-        if (facet->may_install) {
+    if (actions_changed || ctx.may_set_up_flow != facet->installed) {
+        if (ctx.may_set_up_flow) {
             uint32_t keybuf[ODPUTIL_FLOW_KEY_U32S];
             struct dpif_flow_stats stats;
             struct ofpbuf key;