From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id C265A8E7E for ; Tue, 3 Nov 2015 09:03:04 +0100 (CET) Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id E9315341307; Tue, 3 Nov 2015 08:03:03 +0000 (UTC) Received: from [10.36.6.185] (vpn1-6-185.ams2.redhat.com [10.36.6.185]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tA3831FL026822; Tue, 3 Nov 2015 03:03:02 -0500 To: "Xu, Qian Q" , Thomas Monjalon References: <1444907319-26348-1-git-send-email-marcel@redhat.com> <20151015161150-mutt-send-email-mst@redhat.com> <2518261.T4ICXTenZs@xps13> <82F45D86ADE5454A95A89742C8D1410E0317151D@shsmsx102.ccr.corp.intel.com> <82F45D86ADE5454A95A89742C8D1410E03171555@shsmsx102.ccr.corp.intel.com> From: Marcel Apfelbaum Message-ID: <56386A34.2000004@redhat.com> Date: Tue, 3 Nov 2015 10:03:00 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <82F45D86ADE5454A95A89742C8D1410E03171555@shsmsx102.ccr.corp.intel.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.23 Cc: "dev@dpdk.org" , "Michael S. Tsirkin" Subject: Re: [dpdk-dev] [PATCH] vhost-user: enable virtio 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Nov 2015 08:03:05 -0000 On 11/03/2015 05:49 AM, Xu, Qian Q wrote: > Sorry, correct the kernel info, my kernel version is 4.1.8-100.fc21.x86_64. Hi, This is weird, VIRTIO_F_VERSION_1 is defined in 4.0 (I think), and for sure in 4.1 . You can see commit 4ec22fae (virtio: add virtio 1.0 feature bit) from 2014-10-22 in kernel git. Can you please check a mismatch in header files of your development machine? Thanks, Marcel > > Thanks > Qian > > > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Xu, Qian Q > Sent: Tuesday, November 03, 2015 11:45 AM > To: Thomas Monjalon; Marcel Apfelbaum > Cc: dev@dpdk.org; Michael S. Tsirkin > Subject: Re: [dpdk-dev] [PATCH] vhost-user: enable virtio 1.0 > > DPDK GCC 64bit build on kernel 3.18 will be failed, could you help check? > > == Build lib/librte_pipeline > /home/qxu10/virtio-opt-test/dpdk/lib/librte_vhost/virtio-net.c:81:106: error: 'VIRTIO_F_VERSION_1' undeclared here (not in a function) static uint64_t VHOST_FEATURES = VHOST_SUPPORTED_FEATURES; > ^ > /home/qxu10/virtio-opt-test/dpdk/mk/internal/rte.compile-pre.mk:126: recipe for target 'virtio-net.o' failed > make[5]: *** [virtio-net.o] Error 1 > /home/qxu10/virtio-opt-test/dpdk/mk/rte.subdir.mk:61: recipe for target 'librte_vhost' failed > make[4]: *** [librte_vhost] Error 2 > make[4]: *** Waiting for unfinished jobs.... > SYMLINK-FILE include/rte_pipeline.h > CC rte_pipeline.o > AR librte_pipeline.a > INSTALL-LIB librte_pipeline.a > /home/qxu10/virtio-opt-test/dpdk/mk/rte.sdkbuild.mk:93: recipe for target 'lib' failed > make[3]: *** [lib] Error 2 > /home/qxu10/virtio-opt-test/dpdk/mk/rte.sdkroot.mk:124: recipe for target 'all' failed > make[2]: *** [all] Error 2 > /home/qxu10/virtio-opt-test/dpdk/mk/rte.sdkinstall.mk:58: recipe for target 'x86_64-native-linuxapp-gcc_install' failed > make[1]: *** [x86_64-native-linuxapp-gcc_install] Error 2 > /home/qxu10/virtio-opt-test/dpdk/mk/rte.sdkroot.mk:102: recipe for target 'install' failed > make: *** [install] Error 2 > > Thanks > Qian > > > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Tuesday, November 03, 2015 6:14 AM > To: Marcel Apfelbaum > Cc: dev@dpdk.org; Michael S. Tsirkin > Subject: Re: [dpdk-dev] [PATCH] vhost-user: enable virtio 1.0 > >>> Make vhost-user virtio 1.0 compatible by adding it to the supported >>> features and keeping the header length the same as for mergeable RX >>> buffers. >>> >>> Signed-off-by: Marcel Apfelbaum >> >> Looks good to me >> >> Acked-by: Michael S. Tsirkin > > Applied, thanks >