From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36676) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c7oHB-0002Lk-86 for qemu-devel@nongnu.org; Fri, 18 Nov 2016 13:52:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c7oH6-0000tL-E2 for qemu-devel@nongnu.org; Fri, 18 Nov 2016 13:52:41 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57810) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c7oH6-0000tD-77 for qemu-devel@nongnu.org; Fri, 18 Nov 2016 13:52:36 -0500 References: <1479419887-10515-1-git-send-email-maxime.coquelin@redhat.com> From: Maxime Coquelin Message-ID: <0105e7e3-6003-04c8-483d-30ed1208e5fc@redhat.com> Date: Fri, 18 Nov 2016 19:52:30 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC v2 0/3] virtio-net: Add support to MTU feature List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aaron Conole Cc: mst@redhat.com, pbonzini@redhat.com, qemu-devel@nongnu.org, jasowang@redhat.com, yuanhan.liu@linux.intel.com On 11/18/2016 07:15 PM, Aaron Conole wrote: > Maxime Coquelin writes: > >> This series implements Virtio spec update from Aaron Conole which >> defines a way for the host to expose its max MTU to the guest. >> >> 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 new protocol feature MTU >> vhost-net: Add new MTU feature support >> virtio-net: Add MTU feature support >> >> hw/net/vhost_net.c | 11 +++++++++++ >> hw/net/virtio-net.c | 14 ++++++++++++++ >> hw/virtio/vhost-user.c | 11 +++++++++++ >> include/hw/virtio/vhost.h | 1 + >> include/hw/virtio/virtio-net.h | 1 + >> include/net/vhost_net.h | 2 ++ >> 6 files changed, 40 insertions(+) > > I ran this with a VM, but it seems the offered maximum MTU was of value > 0 - is this expected with this version? How can I change the offered > value? Sorry, I'm not as familiar with QEMU/libvirt side of the world. They way I implemented it, the MTU value is to be provided by vhost-user process (e.g. OVS/DPDK). I added a Vhost protocol feature for this. The sequence is: 1. Qemu send VHOST_USER_GET_PROTOCOL_FEATURES request 2. DPDK replies with providing supported features 3. If DPDK supports VHOST_USER_PROTOCOL_F_MTU, Qemu send VHOST_USER_GET_MTU resuest 4. DPDK replies with MTU value Does that make sense? Another possibility would be that we could directly pass the MTU value to Qemu. It may be easier to implement, and to handle migration. Problem is that if we do this, this is not the vSwitch that decides the MTU to set. Regards, Maxime