Thanks both of you for the comments, I'll adjust accordingly,
On Wed, Nov 20, 2013 at 10:19 AM, Joe Stringer <[email protected]>wrote: > On 15 November 2013 13:58, Alex Wang <[email protected]> wrote: > > static uint64_t > > @@ -1081,6 +1088,8 @@ bfd_check_rx(struct bfd *bfd) OVS_REQUIRES(mutex) > > } > > if (bfd->forwarding_if_rx && diff > 0) { > > bfd_forwarding_if_rx_update(bfd); > > + /* Updates the forwarding flag. */ > > + bfd_forwarding__(bfd); > > } > > } > > > > There are multiple callers of bfd_forwarding_if_rx_update(); would it > be better to do bfd_forwarding__() inside that function? >
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
