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:31:47 +0000 (14:31 -0700)
commit4e5e44e30133939b792a013a812c84f564ffa8aa
treef3e73f03e748dd67f981e1c185ad0e044715dd0d
parent4f631ccd515ffb95dbe3f5c22d74b01c64a7d9b8
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