Thanks both of you for the comments,
I'll adjust accordingly,
On Wed, Nov 20, 2013 at 10:19 AM, Joe Stringer <joestrin...@nicira.com>wrote:
> 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
