diff options
author | Ben Pfaff <blp@nicira.com> | 2013-01-31 15:47:24 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2013-01-31 15:47:24 -0800 |
commit | f728af2ed60e873145019f24abfcaf92f4eccd0a (patch) | |
tree | edd1bce8491c020ba5f0c15392c474cc2edf4f87 /ofproto | |
parent | a15fce8efcd159d729ad4749a6c3a1b7dbb36571 (diff) |
ofproto-dpif: Clear revalidation flags only once, not per-ofproto.
Found by inspection.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'ofproto')
-rw-r--r-- | ofproto/ofproto-dpif.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index efd47b38..b5c3d8ef 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -867,6 +867,10 @@ type_run(const char *type) drop_key_clear(backer); } + /* Clear the revalidation flags. */ + tag_set_init(&backer->revalidate_set); + backer->need_revalidate = 0; + HMAP_FOR_EACH (ofproto, all_ofproto_dpifs_node, &all_ofproto_dpifs) { struct facet *facet; @@ -874,10 +878,6 @@ type_run(const char *type) continue; } - /* Clear the revalidation flags. */ - tag_set_init(&backer->revalidate_set); - backer->need_revalidate = 0; - HMAP_FOR_EACH (facet, hmap_node, &ofproto->facets) { if (need_revalidate || tag_set_intersects(&revalidate_set, facet->tags)) { |