The only thing is I didn't follow is:

"""
Do we really still need the bfd_forwarding_if_rx_update() function?
It's so short and only called in this one place, could we just move
it's code there?
"""

Since there are two other places where bfd_forwarding_if_rx_update() are
called, and I think they are necessary.

Thanks,
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to