[ovs-dev] [RFC PATCH 3/3] tunneling: Allow tunnel fragmentation by default.

[Jesse Gross]

On Mon, Mar 25, 2013 at 12:03 PM, Jarno Rajahalme
<jarno.rajahalme at nsn.com> wrote:
> Changes the default tunnel dont_fragment from "true" (don't
> fragment) to "false" (allow fragmentation).  Tunnel outer headers
> will not have the DF bit set by default, and if "df=true" option is
> given for a tunnel, also local fragmentation will be disabled.
> The name of the option is changed from "df_default" to "df" to be in
> line with the rest of the tunneling code.
>
> Signed-off-by: Jarno Rajahalme <jarno.rajahalme at nsn.com>

I can see the desire to make these two settings consistent, although
it really seems preferable to me to have DF on in most situations to
avoid possible repeated fragmentation.  I also don't know that there's
much benefit to turning local_df off since the alternative is to
simply drop the packet (it will also generate an ICMP message but in
the case of tunnels, the sender will never get it).