[ovs-dev] [PATCH 4/8] ofproto-dpif: Make bfd and cfm notify ofproto-dpif when the status changes.

[Alex Wang]

On Wed, Apr 9, 2014 at 4:00 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Fri, Apr 04, 2014 at 03:00:39PM -0700, Alex Wang wrote:
> > This commit adds call-back function to the tunnel monitoring protocal
> > bfd/cfm to allow the session to notify the ofproto-dpif whenever its
> > status changes.  This helps eliminate the overhead of updating unchanged
> > session status to the database.
> >
> > Signed-off-by: Alex Wang <alexw at nicira.com>
>
> I prefer to avoid callbacks, when I can.  Is it possible to avoid them
> here?  One possible way would be to add a sequence number to cfm and
> bfd objects; another way is to add a "changed" flag to cfm and bfd
> flags and provide an interface to read it and reset it.
>


Yes, I think a boolean flag is good enough.  I'll change accordingly,
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140409/b50e5862/attachment-0005.html>