summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Pfaff <blp@ovn.org>2016-11-30 23:02:32 -0800
committerBen Pfaff <blp@ovn.org>2016-12-01 07:45:28 -0800
commitd95a73af8e97d767682e002cbaed2d705f3410a5 (patch)
tree81d46f1eb1a93e9bc9b72df4e5edf71fe4dce7f3
parente89bfca24c12e614570592b8adbb61c444fbad3d (diff)
downloadopenvswitch-d95a73af8e97d767682e002cbaed2d705f3410a5.tar.gz
physical: Remove obsolete comments.
These comments were added above code that removed flows from the flow table. Now that that code was deleted, the comments no longer make sense. Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Russell Bryant <russell@ovn.org>
-rw-r--r--ovn/controller/physical.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c
index c5866b4c6..48adb78d1 100644
--- a/ovn/controller/physical.c
+++ b/ovn/controller/physical.c
@@ -781,9 +781,6 @@ physical_run(struct controller_ctx *ctx, enum mf_field_id mff_ovn_geneve,
* 64 for logical-to-physical translation. */
const struct sbrec_port_binding *binding;
SBREC_PORT_BINDING_FOR_EACH (binding, ctx->ovnsb_idl) {
- /* Because it is possible in the above code to enter this
- * for loop without having cleared the flow table first, we
- * should clear the old flows to avoid collisions. */
consider_port_binding(mff_ovn_geneve, ct_zones, local_datapaths,
patched_datapaths, binding, &ofpacts,
flow_table);
@@ -794,9 +791,6 @@ physical_run(struct controller_ctx *ctx, enum mf_field_id mff_ovn_geneve,
struct ofpbuf remote_ofpacts;
ofpbuf_init(&remote_ofpacts, 0);
SBREC_MULTICAST_GROUP_FOR_EACH (mc, ctx->ovnsb_idl) {
- /* As multicast groups are always reprocessed each time,
- * the first step is to clean the old flows for the group
- * so that we avoid warning messages on collisions. */
consider_mc_group(mff_ovn_geneve, ct_zones,
local_datapaths, mc, &ofpacts, &remote_ofpacts,
flow_table);