ofproto-dpif: Revalidate tagged packets, too, in facet_lookup_valid().
authorBen Pfaff <blp@nicira.com>
Tue, 27 Sep 2011 22:32:18 +0000 (15:32 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 14 Oct 2011 21:08:43 +0000 (14:08 -0700)
I don't see a reason that we should only revalidate facets if we're
revalidating every facet.

ofproto/ofproto-dpif.c

index 9c3d6705b105b88f74c43ad93b33c036d350043a..0c50bfb179f4fd4c44c11ccdf148cb689ea52297 100644 (file)
@@ -2547,7 +2547,8 @@ facet_lookup_valid(struct ofproto_dpif *ofproto, const struct flow *flow)
     /* The facet we found might not be valid, since we could be in need of
      * revalidation.  If it is not valid, don't return it. */
     if (facet
-        && ofproto->need_revalidate
+        && (ofproto->need_revalidate
+            || tag_set_intersects(&ofproto->revalidate_set, facet->tags))
         && !facet_revalidate(ofproto, facet)) {
         COVERAGE_INC(facet_invalidated);
         return NULL;