diff options
author | Ben Pfaff <blp@ovn.org> | 2016-11-30 23:02:32 -0800 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-12-01 07:45:28 -0800 |
commit | d95a73af8e97d767682e002cbaed2d705f3410a5 (patch) | |
tree | 81d46f1eb1a93e9bc9b72df4e5edf71fe4dce7f3 /ovn | |
parent | e89bfca24c12e614570592b8adbb61c444fbad3d (diff) | |
download | openvswitch-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>
Diffstat (limited to 'ovn')
-rw-r--r-- | ovn/controller/physical.c | 6 |
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); |