On Wed, Jun 29, 2016 at 2:41 PM, pravin shelar <pshe...@ovn.org> wrote: > On Tue, Jun 28, 2016 at 10:28 PM, Jesse Gross <je...@kernel.org> wrote: >> Some tunnel protocols, such as Geneve, have a bit in the tunnel >> header to indicate that it is an OAM packet. This means that the >> packet should be processed as a tunnel control frame and not be >> passed onto connected links. >> >> When BFD is used inside of a tunnel it is often used in this control >> capacity, so this adds an option to enable marking the outer header >> when the output port is a tunnel that supports the OAM concept. It is >> also possible to use tunnels as point-to-point links that are simply >> carrying BFD as payload, so this is not always turned on. >> >> Conceptually, this may also apply to other types of packets locally >> generated by the switch, most obviously CFM. However, BFD seems to >> be most commonly used for this type of tunnel monitoring application >> so this only adds the option to BFD for the time being to avoid >> unnecessarily adding configuration knobs that might never get used. >> >> Signed-off-by: Jesse Gross <je...@kernel.org> >> --- > I got white space warning from git am:- > > Applying: bfd: Allow setting OAM bit when encapsulated in tunnel. > > /home/pravin/ovs/ovs/w8/.git/rebase-apply/patch:210: trailing whitespace. > > ofproto_dpif_send_packet(const struct ofport_dpif *ofport, bool oam, > > warning: 1 line adds whitespace errors. > > --- > Otherwise looks good. > > Acked-by: Pravin B Shelar <pshe...@ovn.org>
Thanks, I fixed the whitespace error and applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev