[PATCH v3 0/8] tools: propogate MTU to vif frontends

Paul Durrant posted 8 patches 3 years, 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20200811080202.31163-1-paul@xen.org
Maintainers: Jan Beulich <jbeulich@suse.com>, Andrew Cooper <andrew.cooper3@citrix.com>, Juergen Gross <jgross@suse.com>, George Dunlap <george.dunlap@citrix.com>, Anthony PERARD <anthony.perard@citrix.com>, Julien Grall <julien@xen.org>, Stefano Stabellini <sstabellini@kernel.org>, Wei Liu <wl@xen.org>, Ian Jackson <ian.jackson@eu.citrix.com>
docs/man/xl-network-configuration.5.pod   |  6 +++
docs/misc/xenstore-paths.pandoc           |  3 ++
tools/hotplug/Linux/Makefile              |  1 -
tools/hotplug/Linux/vif-bridge            | 20 +++----
tools/hotplug/Linux/vif2                  | 54 -------------------
tools/hotplug/Linux/xen-network-common.sh | 65 +++++++++++++++++++----
tools/libxl/libxl_nic.c                   | 27 +++++++++-
tools/xl/xl_cmdtable.c                    |  2 +-
tools/xl/xl_parse.c                       |  2 +
xen/include/public/io/netif.h             | 12 +++++
10 files changed, 110 insertions(+), 82 deletions(-)
delete mode 100644 tools/hotplug/Linux/vif2
[PATCH v3 0/8] tools: propogate MTU to vif frontends
Posted by Paul Durrant 3 years, 7 months ago
From: Paul Durrant <pdurrant@amazon.com>

This is an expansion from v2 of the series to include the facility to set
the MTU in the vif config.

There is also one cleanup patch to remove the defunct 'vif2' script.

Paul Durrant (8):
  public/io/netif: specify MTU override node
  tools/hotplug/Linux: re-factor add_to_bridge() in
    xen-network-common.sh
  tools/hotplug/Linux: add remove_from_bridge()
  tools/hotplug/Linux: remove code duplication in vif-bridge
  libxl: wire the libxl_device_nic 'mtu' value into xenstore
  tools/hotplug/Linux: modify set_mtu() to optionally use a configured
    value...
  xl: add 'mtu' option to network configuration
  remove netchannel2 hotplug script... ancient history

 docs/man/xl-network-configuration.5.pod   |  6 +++
 docs/misc/xenstore-paths.pandoc           |  3 ++
 tools/hotplug/Linux/Makefile              |  1 -
 tools/hotplug/Linux/vif-bridge            | 20 +++----
 tools/hotplug/Linux/vif2                  | 54 -------------------
 tools/hotplug/Linux/xen-network-common.sh | 65 +++++++++++++++++++----
 tools/libxl/libxl_nic.c                   | 27 +++++++++-
 tools/xl/xl_cmdtable.c                    |  2 +-
 tools/xl/xl_parse.c                       |  2 +
 xen/include/public/io/netif.h             | 12 +++++
 10 files changed, 110 insertions(+), 82 deletions(-)
 delete mode 100644 tools/hotplug/Linux/vif2
---
Cc: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: Anthony PERARD <anthony.perard@citrix.com>
Cc: George Dunlap <george.dunlap@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Jan Beulich <jbeulich@suse.com>
Cc: Julien Grall <julien@xen.org>
Cc: Stefano Stabellini <sstabellini@kernel.org>
Cc: Wei Liu <wl@xen.org>
-- 
2.20.1


Re: [PATCH v3 0/8] tools: propogate MTU to vif frontends
Posted by Wei Liu 3 years, 7 months ago
On Tue, Aug 11, 2020 at 09:01:54AM +0100, Paul Durrant wrote:
> From: Paul Durrant <pdurrant@amazon.com>
> 
> This is an expansion from v2 of the series to include the facility to set
> the MTU in the vif config.
> 
> There is also one cleanup patch to remove the defunct 'vif2' script.
> 
> Paul Durrant (8):
>   public/io/netif: specify MTU override node
>   tools/hotplug/Linux: re-factor add_to_bridge() in
>     xen-network-common.sh
>   tools/hotplug/Linux: add remove_from_bridge()
>   tools/hotplug/Linux: remove code duplication in vif-bridge
>   libxl: wire the libxl_device_nic 'mtu' value into xenstore
>   tools/hotplug/Linux: modify set_mtu() to optionally use a configured
>     value...
>   xl: add 'mtu' option to network configuration
>   remove netchannel2 hotplug script... ancient history
> 

Patches 2 - 8:
Acked-by: Wei Liu <wl@xen.org>

(I already acked patch 1)

RE: [PATCH v3 0/8] tools: propogate MTU to vif frontends
Posted by Durrant, Paul 3 years, 7 months ago
> -----Original Message-----
> From: Wei Liu <wl@xen.org>
> Sent: 27 August 2020 10:58
> To: Paul Durrant <paul@xen.org>
> Cc: xen-devel@lists.xenproject.org; Durrant, Paul <pdurrant@amazon.co.uk>; Andrew Cooper
> <andrew.cooper3@citrix.com>; Anthony PERARD <anthony.perard@citrix.com>; George Dunlap
> <george.dunlap@citrix.com>; Ian Jackson <ian.jackson@eu.citrix.com>; Jan Beulich <jbeulich@suse.com>;
> Julien Grall <julien@xen.org>; Stefano Stabellini <sstabellini@kernel.org>; Wei Liu <wl@xen.org>
> Subject: RE: [EXTERNAL] [PATCH v3 0/8] tools: propogate MTU to vif frontends
> 
> CAUTION: This email originated from outside of the organization. Do not click links or open
> attachments unless you can confirm the sender and know the content is safe.
> 
> 
> 
> On Tue, Aug 11, 2020 at 09:01:54AM +0100, Paul Durrant wrote:
> > From: Paul Durrant <pdurrant@amazon.com>
> >
> > This is an expansion from v2 of the series to include the facility to set
> > the MTU in the vif config.
> >
> > There is also one cleanup patch to remove the defunct 'vif2' script.
> >
> > Paul Durrant (8):
> >   public/io/netif: specify MTU override node
> >   tools/hotplug/Linux: re-factor add_to_bridge() in
> >     xen-network-common.sh
> >   tools/hotplug/Linux: add remove_from_bridge()
> >   tools/hotplug/Linux: remove code duplication in vif-bridge
> >   libxl: wire the libxl_device_nic 'mtu' value into xenstore
> >   tools/hotplug/Linux: modify set_mtu() to optionally use a configured
> >     value...
> >   xl: add 'mtu' option to network configuration
> >   remove netchannel2 hotplug script... ancient history
> >
> 
> Patches 2 - 8:
> Acked-by: Wei Liu <wl@xen.org>
> 
> (I already acked patch 1)

Thanks Wei.

Ian... ping?

  Paul