[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH 3/4] public/io/netif: specify MTU override node


  • To: Jürgen Groß <jgross@xxxxxxxx>, Paul Durrant <paul@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Durrant, Paul" <pdurrant@xxxxxxxxxxxx>
  • Date: Mon, 3 Aug 2020 08:14:13 +0000
  • Accept-language: en-GB, en-US
  • Delivery-date: Mon, 03 Aug 2020 08:14:23 +0000
  • Ironport-sdr: Z6v44PorqimR+rHyqlmiktqyAKBKak0guLvXpQyvv5CfxNZMJY0L7JVjn72Cj5FDrprnyJZsYk +0+6IrMCCpHw==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHWaVQ8mFueWE86Tk2qq4H/3yFmCqkmCQzg
  • Thread-topic: [PATCH 3/4] public/io/netif: specify MTU override node

> -----Original Message-----
> From: Jürgen Groß <jgross@xxxxxxxx>
> Sent: 03 August 2020 06:09
> To: Paul Durrant <paul@xxxxxxx>; xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Durrant, Paul <pdurrant@xxxxxxxxxxxx>
> Subject: RE: [EXTERNAL] [PATCH 3/4] public/io/netif: specify MTU override node
> 
> On 30.07.20 21:48, Paul Durrant wrote:
> > From: Paul Durrant <pdurrant@xxxxxxxxxx>
> >
> > There is currently no documentation to state what MTU a frontend should
> > adertise to its network stack. It has however long been assumed that the
> > default value of 1500 is correct.
> >
> > This patch specifies a mechanism to allow the tools to set the MTU via a
> > xenstore node in the frontend area and states that the absence of that node
> > means the frontend should assume an MTU of 1500 octets.
> >
> > NOTE: The Windows PV frontend has used an MTU sampled from the xenstore
> >        node specified in this patch.
> >
> > Signed-off-by: Paul Durrant <pdurrant@xxxxxxxxxx>
> 
> Can you please update docs/misc/xenstore-paths.pandoc accordingly?

Sure. Given the path is for use by tools then it should indeed be documented 
there as well.

> With that you can have my:
> 
> Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
> 

Thanks,

  Paul

> 
> Juergen

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.