qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [RFC v3 0/3] virtio-net: Add support to MTU feature
@ 2016-11-30 10:10 Maxime Coquelin
  2016-11-30 10:10 ` [Qemu-devel] [RFC v3 1/3] vhost-user: Add MTU protocol feature and op Maxime Coquelin
                   ` (4 more replies)
  0 siblings, 5 replies; 15+ messages in thread
From: Maxime Coquelin @ 2016-11-30 10:10 UTC (permalink / raw)
  To: mst, aconole, pbonzini, qemu-devel, fbl, berrange, mlureau,
	ktraynor
  Cc: jasowang, yuanhan.liu, Maxime Coquelin

This series implements Virtio spec update from Aaron Conole which
defines a way for the host to expose its max MTU to the guest.

This third version re-desings how MTU value is provided to QEMU.
Now, host_mtu parameter is added to provide QEMU with the MTU value,
and the backend, if supported, gets notified of the MTU value when the
MTU feature neogotiation succeeds.

Only user backend currently supports MTU notification. A new protocol
feature has been implemented for sending MTU value to the backend.
Aaron, Kevin, Flavio, do you confirm this works for OVS if DPDK vhost lib
adds needed API to get the MTU value?

For kernel backend, it is expected the management tool also configures
the tap/macvtap interface with same MTU value.
Daniel, I would be interrested about your feedback on this implementation
from management tool point of view.

Changes since RFC v2:
---------------------
 - host_mtu propoerty is now used to specify the MTU value
 - update vhost-user spec with MTU additions

Changes since RFC v1:
---------------------
 - Rebased on top of v2.8.0-rc0 (2.7.90)
 - Write MTU unconditionnaly in netcfg to avoid memory leak (Paolo)
 - Add host_mtu property to be able to disable the feature from QEMU

Maxime Coquelin (3):
  vhost-user: Add MTU protocol feature and op
  vhost-net: Notify the backend about the host MTU
  virtio-net: Add MTU feature support

 docs/specs/vhost-user.txt         | 13 +++++++++++++
 hw/net/vhost_net.c                | 22 ++++++++++++++++++++++
 hw/net/virtio-net.c               | 13 +++++++++++++
 hw/virtio/vhost-user.c            | 13 +++++++++++++
 include/hw/virtio/vhost-backend.h |  2 ++
 include/hw/virtio/virtio-net.h    |  1 +
 include/net/vhost_net.h           |  2 ++
 7 files changed, 66 insertions(+)

-- 
2.9.3

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2016-12-07  7:49 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-30 10:10 [Qemu-devel] [RFC v3 0/3] virtio-net: Add support to MTU feature Maxime Coquelin
2016-11-30 10:10 ` [Qemu-devel] [RFC v3 1/3] vhost-user: Add MTU protocol feature and op Maxime Coquelin
2016-11-30 10:25   ` Yuanhan Liu
2016-11-30 12:17     ` Maxime Coquelin
2016-11-30 10:10 ` [Qemu-devel] [RFC v3 2/3] vhost-net: Notify the backend about the host MTU Maxime Coquelin
2016-12-06 18:31   ` Aaron Conole
2016-12-07  7:49     ` Maxime Coquelin
2016-11-30 10:10 ` [Qemu-devel] [RFC v3 3/3] virtio-net: Add MTU feature support Maxime Coquelin
2016-11-30 11:24   ` Jason Wang
2016-11-30 12:24     ` Maxime Coquelin
2016-11-30 10:30 ` [Qemu-devel] [RFC v3 0/3] virtio-net: Add support to MTU feature no-reply
2016-11-30 11:23 ` Jason Wang
2016-11-30 12:16   ` Maxime Coquelin
2016-11-30 13:42     ` Michael S. Tsirkin
2016-12-05 16:11       ` Aaron Conole

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).