[dpdk-dev] [Qemu-devel] [PATCH] vhost: fix compilation issue caused by virtio 1.0 support

2015-11-03 Thread Marcel Apfelbaum
On 11/03/2015 01:28 PM, Marcel Apfelbaum wrote: > Commit 15e9ee6982a4822ce395fd597dd500a61ceafa7c (vhost: enable virtio 1.0) > uses the VIRTIO_F_VERSION_1 macro existing only in newer kernels. > > Fixed it by manually defining it for older kernels. I added to CC the involved develop

[dpdk-dev] [Qemu-devel] [PATCH] vhost: fix compilation issue caused by virtio 1.0 support

2015-11-03 Thread Marcel Apfelbaum
Commit 15e9ee6982a4822ce395fd597dd500a61ceafa7c (vhost: enable virtio 1.0) uses the VIRTIO_F_VERSION_1 macro existing only in newer kernels. Fixed it by manually defining it for older kernels. Reported-by: Xu, Qian Q Signed-off-by: Marcel Apfelbaum --- Hi, I reproduced the issue with 3.9.5

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-11-03 Thread Marcel Apfelbaum
On 11/03/2015 10:16 AM, Xie, Huawei wrote: > On 11/3/2015 4:03 PM, Marcel Apfelbaum wrote: >> 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 w

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-11-03 Thread Marcel Apfelbaum
On 11/03/2015 10:26 AM, Thomas Monjalon wrote: > 2015-11-03 08:16, Xie, Huawei: >> On 11/3/2015 4:03 PM, Marcel Apfelbaum wrote: >>> 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

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-11-03 Thread Marcel Apfelbaum
x27;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 > > >

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-11-01 Thread Marcel Apfelbaum
On 11/01/2015 11:53 AM, Thomas Monjalon wrote: > 2015-11-01 11:00, Marcel Apfelbaum: >> On 10/30/2015 07:48 PM, Thomas Monjalon wrote: >>> 2015-10-18 10:04, Michael S. Tsirkin: >>>> This will require more work then as 1.0 is a different >>>> endian-ne

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-11-01 Thread Marcel Apfelbaum
On 10/30/2015 07:48 PM, Thomas Monjalon wrote: > 2015-10-18 10:04, Michael S. Tsirkin: >> On Fri, Oct 16, 2015 at 02:52:30PM +0100, Bruce Richardson wrote: >>> On Thu, Oct 15, 2015 at 04:18:59PM +0300, Michael S. Tsirkin wrote: >>>> On Thu, Oct 15, 2015 at 02:08:39PM

[dpdk-dev] [PATCH v6 00/13] vhost-user multiple queues enabling

2015-10-20 Thread Marcel Apfelbaum
On 10/09/2015 08:45 AM, Yuanhan Liu wrote: > This patch set enables vhost-user multiple queues. > > Overview > > > It depends on some QEMU patches that has already been merged to upstream. > Those qemu patches introduce some new vhost-user messages, for vhost-user > mq enabling negotiation

[dpdk-dev] [PATCH] vhost-user: enable virtio 1.0

2015-10-15 Thread Marcel Apfelbaum
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 --- To be applied on top of: [dpdk-dev] [PATCH v6 00/13] vhost-user multiple queues enabling Thanks, Marcel lib

[dpdk-dev] [PATCH v6 00/13] vhost-user multiple queues enabling

2015-10-09 Thread Marcel Apfelbaum
On 10/09/2015 08:45 AM, Yuanhan Liu wrote: > This patch set enables vhost-user multiple queues. > > Overview > > > It depends on some QEMU patches that has already been merged to upstream. > Those qemu patches introduce some new vhost-user messages, for vhost-user > mq enabling negotiation

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-24 Thread Marcel Apfelbaum
On 09/23/2015 06:46 AM, Yuanhan Liu wrote: > On Tue, Sep 22, 2015 at 05:51:02PM +0300, Marcel Apfelbaum wrote: [...] >>> It's proved to work after the fix (at least in my testing), but >>> it's late here and I'm gonna send a new version tomorrow, includin

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-22 Thread Marcel Apfelbaum
On 09/22/2015 05:22 PM, Yuanhan Liu wrote: > On Tue, Sep 22, 2015 at 01:06:17PM +0300, Marcel Apfelbaum wrote: >> On 09/22/2015 12:21 PM, Yuanhan Liu wrote: >>> On Tue, Sep 22, 2015 at 11:47:34AM +0300, Marcel Apfelbaum wrote: >>>> On 09/22/2015 11:34 AM, Yuanhan L

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-22 Thread Marcel Apfelbaum
On 09/22/2015 12:21 PM, Yuanhan Liu wrote: > On Tue, Sep 22, 2015 at 11:47:34AM +0300, Marcel Apfelbaum wrote: >> On 09/22/2015 11:34 AM, Yuanhan Liu wrote: >>> On Tue, Sep 22, 2015 at 11:10:13AM +0300, Marcel Apfelbaum wrote: >>>> On 09/22/2015 10:31 AM, Yuanhan L

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-22 Thread Marcel Apfelbaum
On 09/22/2015 11:34 AM, Yuanhan Liu wrote: > On Tue, Sep 22, 2015 at 11:10:13AM +0300, Marcel Apfelbaum wrote: >> On 09/22/2015 10:31 AM, Yuanhan Liu wrote: >>> On Mon, Sep 21, 2015 at 08:56:30PM +0300, Marcel Apfelbaum wrote: >> [...] >>>>> >>>>

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-22 Thread Marcel Apfelbaum
On 09/22/2015 10:31 AM, Yuanhan Liu wrote: > On Mon, Sep 21, 2015 at 08:56:30PM +0300, Marcel Apfelbaum wrote: [...] >>> >>> Hi, >>> >>> I have made 4 cleanup patches few weeks before, including the patch >>> to define kickfd and callfd as int

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-21 Thread Marcel Apfelbaum
On 09/21/2015 05:06 AM, Yuanhan Liu wrote: > On Sun, Sep 20, 2015 at 04:58:42PM +0300, Marcel Apfelbaum wrote: >> On 09/18/2015 06:10 PM, Yuanhan Liu wrote: >>> All queue pairs, including the default (the first) queue pair, >>> are allocated dynamically, when a vri

[dpdk-dev] [PATCH v5 resend 03/12] vhost: vring queue setup for multiple queue support

2015-09-20 Thread Marcel Apfelbaum
On 09/18/2015 06:10 PM, Yuanhan Liu wrote: > All queue pairs, including the default (the first) queue pair, > are allocated dynamically, when a vring_call message is received > first time for a specific queue pair. > > This is a refactor work for enabling vhost-user multiple queue; > it should not

[dpdk-dev] [PATCH v5 resend 00/12] vhost-user multiple queues enabling

2015-09-20 Thread Marcel Apfelbaum
On 09/18/2015 06:10 PM, Yuanhan Liu wrote: > This patch set enables vhost-user multiple queues. It seems the series does not apply on master, did you use a different branch or tag? Thanks, Marcel > > Overview > > > It depends on some QEMU patches that, hopefully, will be merged soon. >