[ovs-dev] [MASTER 2/3] ofproto: Add separate functions for checking bfd/cfm status change.

Ben Pfaff blp at nicira.com
Fri Jun 13 19:51:39 UTC 2014


On Tue, Jun 10, 2014 at 10:03:51AM -0700, Alex Wang wrote:
> Currently, ofproto_port_get_bfd/cfm_status() is used to check the
> bfd/cfm status change and query the status change.  Users decide
> what to do with the filled status struct based on the return value
> of the funciton.  Such design is confusing and makes the caller
> code hard to read.
> 
> This commit breaks the function into a status change check function
> and a status query function, so that they become easier to read and
> use.
> 
> Signed-off-by: Alex Wang <alexw at nicira.com>

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list