Changed.
On Sat, Oct 13, 2012 at 4:29 PM, Ben Pfaff <[email protected]> wrote: > Thanks. As a nit, the first period in that looks like it should be a > comma. > > On Sat, Oct 13, 2012 at 02:26:49PM -0700, Ethan Jackson wrote: >> Good idea, I've added the following paragraph >> >> <p> >> When operating over tunnels which have no <code>in_key</code>, or >> an >> <code>in_key</code> of <code>flow</code>. CFM will only accept >> CCMs >> with a tunnel key of zero. >> </p> >> >> Ethan >> >> On Fri, Oct 12, 2012 at 8:12 PM, Ben Pfaff <[email protected]> wrote: >> > On Fri, Oct 12, 2012 at 07:40:22PM -0700, Ethan Jackson wrote: >> >> Before this patch, when a tunnel is configured with key=flow, CFM >> >> didn't verify that incoming packets had the appropriate key of >> >> zero. This could cause the CFM module to consume packets which >> >> weren't actually intended for it. >> >> >> >> Bug #13542. >> >> Signed-off-by: Ethan Jackson <[email protected]> >> > >> > This looks good to me. >> > >> > It might be good to add some words about this verification in the >> > documentation for CFM in vswitch.xml. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
