> -----Original Message-----
> From: Ying Xue [mailto:[email protected]]
> Sent: Wednesday, March 21, 2018 07:38
> To: Mohan Krishna Ghanta Krishnamurthy
> <[email protected]>; tipc-
> [email protected]; Jon Maloy <[email protected]>;
> [email protected]
> Subject: Re: [net-next 0/2] tipc: Confgiuration of MTU for media UDP
> 
> Good proposal!
> 
> Acked-by: Ying Xue <[email protected]>
> 
> The only disadvantage is that after MTU is changed by user, it cannot
> immediately take effect. Instead user has to manually have bearer down and
> up again. Sometimes user is a bit confused by this.
> 
> Of course, the behavior of changing media MTU is same with media other
> properties like tolerance, window and priority.

Unfortunately it is has more side effects. If we lower the MTU for a bearer and 
all its links a lot of packets already in the send queue may be too big for 
transmission, and will never get through, which will lead to link reset anyway.
At least that's what I thought until I started writing this mail. But I forgot 
that this is UDP, not Ethernet, so this is not a problem. The packets *will* go 
through, so there is no need to reset bearer and links at all 😉
We should change this.

///jon

> 
> On 03/19/2018 09:27 PM, GhantaKrishnamurthy MohanKrishna wrote:
> > Systematic measurements have shown that an emulated MTU of 14k for
> UDP
> > bearers is the optimal value for maximal throughput. Accordingly, the
> > default MTU of UDP bearers is changed to 14k.
> >
> > We also provide users with a fallback option from this value, by
> > providing support to configure MTU for UDP bearers.
> >
> > GhantaKrishnamurthy MohanKrishna (2):
> >   tipc: set default MTU for UDP media
> >   tipc: implement configuration of UDP media MTU
> >
> >  include/uapi/linux/tipc_config.h  |  5 +++++
> > include/uapi/linux/tipc_netlink.h |  1 +
> >  net/tipc/bearer.c                 | 13 +++++++++++++
> >  net/tipc/bearer.h                 |  3 +++
> >  net/tipc/udp_media.c              |  4 ++--
> >  net/tipc/udp_media.h              | 14 ++++++++++++++
> >  6 files changed, 38 insertions(+), 2 deletions(-)
> >
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
tipc-discussion mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tipc-discussion

Reply via email to