Re: [ovs-dev] [PATCH 2/2] bfd: Allow users to set local/remote src/dst MAC address.

2014-08-10 Thread Ethan Jackson
For the local eth_src and eth_dst, instead of having the various is set booleans, wouldn't it be simpler to just copy the default value into the struct bfd in bfd_configure() instead of doing it in bfd_put_packet()? I'm not sure we should have the rmt_eth_src flag. We should probably accept bfd p

[ovs-dev] [PATCH 2/2] bfd: Allow users to set local/remote src/dst MAC address.

2014-08-08 Thread Alex Wang
This commit adds four options for configuring the MAC addresses in BFD state machine. Therein, the "bfd_local_src_mac" and "bfd_local_dst_mac" configure the MAC address of sent BFD packets. The "bfd_remote_src_mac" and "bfd_remote_dst_mac" configure the matching of MAC address on recevied BFD pac