diff options
| author | Ben Pfaff <blp@nicira.com> | 2012-01-06 15:03:07 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2012-01-06 17:02:13 -0800 |
| commit | b87dd2dcf4a2af434d1b92f6efcb32f54fee03b2 (patch) | |
| tree | a4a3cd23208761e64738a6a35de72faead2f9e78 /AUTHORS | |
| parent | 2716b6379dc86331330d78b496985e908320ddfa (diff) | |
ofproto-dpif: Keep subfacets longer to avoid assert-fail in facet_account().
If a subfacet expired when its facet still had statistics that had not
yet been pushed into the rule, and the facet either used the "normal"
action or the bridge contained a bond port, then facet_account() would
be called after the last subfacet was removed from its facet's list of
subfacets, triggering an assertion failure in list_front().
This fixes the problem by always running facet_flush_stats() (which calls
facet_account()) before deleting the last subfacet from a facet.
This problem took a while to surface because subfacets usually expire only
long after their statistics have been pushed into the rule.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Reported-by: Mike Kruze <mkruze@nicira.com>
Bug #9074.
Diffstat (limited to 'AUTHORS')
| -rw-r--r-- | AUTHORS | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -93,6 +93,7 @@ Luiz Henrique Ozaki luiz.ozaki@gmail.com Michael A. Collins mike.a.collins@ark-net.org Michael Hu mhu@nicira.com Michael Mao mmao@nicira.com +Mike Kruze mkruze@nicira.com Murphy McCauley murphy.mccauley@gmail.com Mikael Doverhag mdoverhag@nicira.com Niklas Andersson nandersson@nicira.com |