On 15 November 2013 13:58, Alex Wang <al...@nicira.com> 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 dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev