aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Pfaff <blp@nicira.com>2010-11-05 09:27:27 -0700
committerBen Pfaff <blp@nicira.com>2010-11-05 09:27:27 -0700
commit6d6c72594d019672f1fc45ba9ca95b622ac3e969 (patch)
tree446a8ec89e18453cc1c5191c856313b8f1914a8c
parent7a0efeb5ad32fc685640b3815da4e20a7f11e3c5 (diff)
ofproto: Improve comment in send_flow_removed().
-rw-r--r--ofproto/ofproto.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index b35139cb..42260443 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -4548,12 +4548,6 @@ send_flow_removed(struct ofproto *p, struct rule *rule,
return;
}
- /* We limit the maximum number of queued flow expirations it by accounting
- * them under the counter for replies. That works because preventing
- * OpenFlow requests from being processed also prevents new flows from
- * being added (and expiring). (It also prevents processing OpenFlow
- * requests that would not add new flows, so it is imperfect.) */
-
LIST_FOR_EACH (ofconn, node, &p->all_conns) {
struct ofpbuf *msg;
@@ -4563,6 +4557,12 @@ send_flow_removed(struct ofproto *p, struct rule *rule,
}
msg = compose_flow_removed(p, rule, now, reason);
+
+ /* Account flow expirations under ofconn->reply_counter, the counter
+ * for replies to OpenFlow requests. That works because preventing
+ * OpenFlow requests from being processed also prevents new flows from
+ * being added (and expiring). (It also prevents processing OpenFlow
+ * requests that would not add new flows, so it is imperfect.) */
queue_tx(msg, ofconn, ofconn->reply_counter);
}
}