ofproto-dpif: Set need_revalidate when removing cfm from ofport.
authorAlex Wang <alexw@nicira.com>
Sat, 28 Mar 2015 06:19:22 +0000 (23:19 -0700)
committerAlex Wang <alexw@nicira.com>
Sat, 28 Mar 2015 21:27:42 +0000 (14:27 -0700)
commit0e45cfe43d8c3624fe599c9f5a0e68041c57d437
treeb34b9c9494757ed41f4af955a58f6d9d846b770d
parent3638617df538b3ab945fcebb10d25ca272c3a6ca
ofproto-dpif: Set need_revalidate when removing cfm from ofport.

When cfm is deleted from a port, all modules should release their
reference so that the cfm struct can be removed from the global hmap
and freed.  Therein, the reference held by xlate module can only be
released when the need_revalidate flag is set (e.g set to
REV_RECONFIGURE).  And this flag should be set while removing cfm
from ofport.  Unfortunately, this has never been done before and the
bug was hidden by another bug fixed in recent commit a190839
(netdev-vport: Do not update netdev when there is no config change.)

To fix this issue, this commit makes the code set need_revalidate
when removing cfm from ofport.

Signed-off-by: Alex Wang <alexw@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif.c