[ovs-dev] [PATCHv2] netdev-vport: Disable Path MTU Discovery by default

[Jesse Gross]

On Thu, Oct 25, 2012 at 3:46 PM, Ansis Atteka <aatteka at nicira.com> wrote:
> On Thu, Oct 25, 2012 at 9:24 PM, Jesse Gross <jesse at nicira.com> wrote:
>> On Thu, Oct 25, 2012 at 4:50 AM, Ansis Atteka <aatteka at nicira.com> wrote:
>>> In near future we will switch to the flow-based tunneling and drop path
>>> MTU discovery support. This patch prepares for that by changing the default
>>> PMTUD value from 'enabled' to 'disabled'.
>>>
>>> Signed-off-by: Ansis Atteka <aatteka at nicira.com>
>>
>> Since the goal of this is to warn people that the feature will be
>> going away soon for good, I think we should probably add some notes
>> indicating that to NEWS, the documentation, and maybe a warning if it
>> is enabled.
>
> Since, I already pushed this patch to 1.9 branch, then I will have to
> create another one that would:
> 1. print warning, if PMTUD is enabled,
> 2. mention this behavioural change in the man pages;
>
> In the NEWS I already mentioned that default behaviour was changed. I
> guess that what you are suggesting here is to add a little bit deeper
> explanation on why exactly this was done, right?

The main point to mention is that the default hasn't just changed but
that the feature is deprecated and will be removed in the next
release.