[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, including >>> some other comments addressing.

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

2015-09-23 Thread Yuanhan Liu
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, including > >some other comments addressing. Please do more test then :) > > It's unlikely that I will

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

2015-09-22 Thread Yuanhan Liu
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 Liu wrote: > >>>On Tue, Sep 22, 2015 at 11:10:13AM +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 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 Liu wrote: > On Tue, Sep 22,

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

2015-09-22 Thread Yuanhan Liu
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 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 Yuanhan Liu
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: > [...] > >>> > >>>Hi, > >>> > >>>I have made 4 cleanup patches few weeks before, including the patch > >>>to define

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

2015-09-22 Thread Yuanhan Liu
On Mon, Sep 21, 2015 at 08:56:30PM +0300, Marcel Apfelbaum wrote: > 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, >

[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 Liu wrote: > On Mon, Sep 21,

[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: >> [...] > > Hi, > > I have made 4 cleanup patches few

[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 type, and they have already got >>> the ACK from

[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 vring_call message is received >>>

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

2015-09-21 Thread Michael S. Tsirkin
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 vring_call message is received > >first time for a specific queue pair. > > > >This

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

2015-09-21 Thread Yuanhan Liu
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 vring_call message is received > >first time for a specific queue pair. > > > >This

[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 03/12] vhost: vring queue setup for multiple queue support

2015-09-19 Thread Yuanhan Liu
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 break anything as it does no functional changes: we