ofproto: Add extra comments and checking for expiring a pending rule.
authorBen Pfaff <blp@nicira.com>
Tue, 3 Jul 2012 21:00:38 +0000 (14:00 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 12 Jul 2012 21:13:01 +0000 (14:13 -0700)
commite2a3d183f60b93265c095ede1379194916444822
tree531d4fecf75b5ec35cb0650c71e05a646171636a
parent080437614b40799853a42806fa29e7c71f42210d
ofproto: Add extra comments and checking for expiring a pending rule.

A given rule may only have one pending operation at a time, so when an
operation is pending we must not allow a flow expiration to be started on
that rule.

This doesn't fix a user-visible bug in ofproto-dpif because ofproto-dpif
always completes operations immediately, that is, no operations will be
pending when expiration runs.  (Technically there is a bug if the user
runs "ovs-appctl ofproto/clog", but that feature is for debugging only and
there is no reason for a user to ever run it.)

Signed-off-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif.c
ofproto/ofproto-provider.h
ofproto/ofproto.c