[ovs-dev] [ovs-discuss] [ACLv2 06/19] ofproto: Add flow expiration callback.
blp at nicira.com
Fri Sep 4 22:05:44 UTC 2009
Jesse Gross <jesse at nicira.com> writes:
> This is a very good catch as there are several tricky corner cases
> here. In addition to the cases you mention, there were also similar
> conditions involving revalidation. The ACL code is not very forgiving
> of incorrect flow expiration messages: both missing and overly
> aggressive expirations will lead to incorrect stats. So if I trigger
> flow_expire_cb() in ofproto_send_packet() after it sent a packet that
> happens to match a still-active flow, then that flow will have the
> wrong stats. What I do now is pass an additional flag saying that the
> flow will not ever be installed. This causes the stats to be credited
> immediately for that packet and then there is no need to send a flow
> expiration message later.
This sounds much better, thank you.
More information about the dev