[libvirt] [PATCH v2 0/4] Support setting MTU in libvirt networks

Laine Stump posted 4 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20170203173526.2494-1-laine@laine.org
docs/formatnetwork.html.in          | 19 ++++++++++++++++++-
docs/news.xml                       |  5 +++--
docs/schemas/network.rng            |  5 +++++
src/bhyve/bhyve_command.c           |  1 +
src/conf/network_conf.c             | 31 ++++++++++++++++++++++++++++++-
src/conf/network_conf.h             |  1 +
src/network/bridge_driver.c         |  1 +
src/qemu/qemu_command.c             | 32 ++++++++++++++++++++++----------
src/qemu/qemu_command.h             |  3 ++-
src/qemu/qemu_hotplug.c             |  5 +++--
src/qemu/qemu_interface.c           |  4 +++-
src/qemu/qemu_interface.h           |  3 ++-
src/uml/uml_conf.c                  |  1 +
src/util/virnetdevtap.c             | 32 +++++++++++++++++++++++++++-----
src/util/virnetdevtap.h             |  2 ++
tests/bhyvexml2argvmock.c           |  2 ++
tests/networkxml2xmlin/set-mtu.xml  | 12 ++++++++++++
tests/networkxml2xmlout/set-mtu.xml | 12 ++++++++++++
tests/networkxml2xmltest.c          |  1 +
19 files changed, 148 insertions(+), 24 deletions(-)
create mode 100644 tests/networkxml2xmlin/set-mtu.xml
create mode 100644 tests/networkxml2xmlout/set-mtu.xml
[libvirt] [PATCH v2 0/4] Support setting MTU in libvirt networks
Posted by Laine Stump 7 years, 1 month ago
Similar to V1, but changing the XML to match Michal's patches, and
adding a patch that plumbs network MTU back through to the qemu
host_mtu option.

Laine Stump (4):
  util: add MTU arg to virNetDevTapCreateInBridgePort()
  conf: support configuring mtu size in a virtual network
  network: honor mtu setting when creating network
  qemu: propagate bridge MTU into qemu "host_mtu" option

 docs/formatnetwork.html.in          | 19 ++++++++++++++++++-
 docs/news.xml                       |  5 +++--
 docs/schemas/network.rng            |  5 +++++
 src/bhyve/bhyve_command.c           |  1 +
 src/conf/network_conf.c             | 31 ++++++++++++++++++++++++++++++-
 src/conf/network_conf.h             |  1 +
 src/network/bridge_driver.c         |  1 +
 src/qemu/qemu_command.c             | 32 ++++++++++++++++++++++----------
 src/qemu/qemu_command.h             |  3 ++-
 src/qemu/qemu_hotplug.c             |  5 +++--
 src/qemu/qemu_interface.c           |  4 +++-
 src/qemu/qemu_interface.h           |  3 ++-
 src/uml/uml_conf.c                  |  1 +
 src/util/virnetdevtap.c             | 32 +++++++++++++++++++++++++++-----
 src/util/virnetdevtap.h             |  2 ++
 tests/bhyvexml2argvmock.c           |  2 ++
 tests/networkxml2xmlin/set-mtu.xml  | 12 ++++++++++++
 tests/networkxml2xmlout/set-mtu.xml | 12 ++++++++++++
 tests/networkxml2xmltest.c          |  1 +
 19 files changed, 148 insertions(+), 24 deletions(-)
 create mode 100644 tests/networkxml2xmlin/set-mtu.xml
 create mode 100644 tests/networkxml2xmlout/set-mtu.xml

-- 
2.9.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 0/4] Support setting MTU in libvirt networks
Posted by Michal Privoznik 7 years, 1 month ago
On 03.02.2017 18:35, Laine Stump wrote:
> Similar to V1, but changing the XML to match Michal's patches, and
> adding a patch that plumbs network MTU back through to the qemu
> host_mtu option.
> 
> Laine Stump (4):
>   util: add MTU arg to virNetDevTapCreateInBridgePort()
>   conf: support configuring mtu size in a virtual network
>   network: honor mtu setting when creating network
>   qemu: propagate bridge MTU into qemu "host_mtu" option
> 
>  docs/formatnetwork.html.in          | 19 ++++++++++++++++++-
>  docs/news.xml                       |  5 +++--
>  docs/schemas/network.rng            |  5 +++++
>  src/bhyve/bhyve_command.c           |  1 +
>  src/conf/network_conf.c             | 31 ++++++++++++++++++++++++++++++-
>  src/conf/network_conf.h             |  1 +
>  src/network/bridge_driver.c         |  1 +
>  src/qemu/qemu_command.c             | 32 ++++++++++++++++++++++----------
>  src/qemu/qemu_command.h             |  3 ++-
>  src/qemu/qemu_hotplug.c             |  5 +++--
>  src/qemu/qemu_interface.c           |  4 +++-
>  src/qemu/qemu_interface.h           |  3 ++-
>  src/uml/uml_conf.c                  |  1 +
>  src/util/virnetdevtap.c             | 32 +++++++++++++++++++++++++++-----
>  src/util/virnetdevtap.h             |  2 ++
>  tests/bhyvexml2argvmock.c           |  2 ++
>  tests/networkxml2xmlin/set-mtu.xml  | 12 ++++++++++++
>  tests/networkxml2xmlout/set-mtu.xml | 12 ++++++++++++
>  tests/networkxml2xmltest.c          |  1 +
>  19 files changed, 148 insertions(+), 24 deletions(-)
>  create mode 100644 tests/networkxml2xmlin/set-mtu.xml
>  create mode 100644 tests/networkxml2xmlout/set-mtu.xml
> 

ACK series. Regarding the migration issue - that is pre-existing and
thus does not hold back pushing of these patches.

Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list