Re: [PATCHv3 net-next 2/3] sunvnet: allow admin to set sunvnet MTU

[Sowmini Varadhan <sowmini.varadhan@oracle.com>]

On 09/13/2014 10:15 PM, David L Stevens wrote:

> I wouldn't say I like it, either, but the problem is that without it, we
> are tied to the least common denominator. Anything that doesn't support
> v1.6 of the VIO protocol is stuck at the low MTU and low throughput, and

To put things in perspective, in practice its only legacy linux today 
that will do the v1.0, and administrators are likely to want to upgrade
to the later version, so encumbering the code with legacy version 
support may end up becoming hard-to-maintain code?

> I think of it as an Ethernet connected to a virtual switch, and the ICMP
> errors are for PMTUD are analogous to IGMP snooping. This is not an Ethernet
> device alone-- those don't negotiate per-destination link MTUs. But nothing forces
> anyone to mix MTUs; the ICMP errors simply allow it.

As I understand it, this method of sending ICMP from the driver will not
work for L2 (non-IP) packets, and it will not even work for IP packets 
that are coming to us, from, say, openvswitch, right? So in practice it
actually has limited usability?

--Sowmini