[ovs-git] [openvswitch/ovs] 4e5e44: ofproto-dpif: Set need_revalidate when removing cf...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 4e5e44e30133939b792a013a812c84f564ffa8aa
      https://github.com/openvswitch/ovs/commit/4e5e44e30133939b792a013a812c84f564ffa8aa
  Author: Alex Wang <alexw at nicira.com>
  Date:   2015-03-28 (Sat, 28 Mar 2015)

  Changed paths:
    M ofproto/ofproto-dpif.c

  Log Message:
  -----------
  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 at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>