diff options
author | Ben Pfaff <blp@nicira.com> | 2010-11-05 09:27:27 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2010-11-05 09:27:27 -0700 |
commit | 6d6c72594d019672f1fc45ba9ca95b622ac3e969 (patch) | |
tree | 446a8ec89e18453cc1c5191c856313b8f1914a8c | |
parent | 7a0efeb5ad32fc685640b3815da4e20a7f11e3c5 (diff) |
ofproto: Improve comment in send_flow_removed().
-rw-r--r-- | ofproto/ofproto.c | 12 |
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); } } |