Re: [dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-04-01 Thread Yuanhan Liu
On Sat, Apr 01, 2017 at 04:32:58PM +0800, Yuanhan Liu wrote: > On Wed, Mar 22, 2017 at 04:56:12PM +0800, Yuanhan Liu wrote: > > On Wed, Mar 22, 2017 at 09:41:07AM +0100, Maxime Coquelin wrote: > > > > > > > > > On 03/02/2017 07:16 AM, Yuanhan Liu wrote: > > > >We used to allocate queues based on

Re: [dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-04-01 Thread Yuanhan Liu
On Wed, Mar 22, 2017 at 04:56:12PM +0800, Yuanhan Liu wrote: > On Wed, Mar 22, 2017 at 09:41:07AM +0100, Maxime Coquelin wrote: > > > > > > On 03/02/2017 07:16 AM, Yuanhan Liu wrote: > > >We used to allocate queues based on the index from SET_VRING_CALL > > >request: if corresponding queue hasn't

Re: [dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-03-22 Thread Yuanhan Liu
On Wed, Mar 22, 2017 at 09:41:07AM +0100, Maxime Coquelin wrote: > > > On 03/02/2017 07:16 AM, Yuanhan Liu wrote: > >We used to allocate queues based on the index from SET_VRING_CALL > >request: if corresponding queue hasn't been allocated, allocate it. > > > >Though it's pratically right (it's t

Re: [dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-03-22 Thread Maxime Coquelin
On 03/02/2017 07:16 AM, Yuanhan Liu wrote: We used to allocate queues based on the index from SET_VRING_CALL request: if corresponding queue hasn't been allocated, allocate it. Though it's pratically right (it's the first per-vring request we will get from QEMU for vhost-user negotiation), but

[dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-03-01 Thread Yuanhan Liu
We used to allocate queues based on the index from SET_VRING_CALL request: if corresponding queue hasn't been allocated, allocate it. Though it's pratically right (it's the first per-vring request we will get from QEMU for vhost-user negotiation), but it's not technically right: it's not documente

[dpdk-dev] [PATCH] vhost: remove a hack on queue allocation

2017-03-01 Thread Yuanhan Liu
We used to allocate queues based on the index from SET_VRING_CALL request: if corresponding queue hasn't been allocated, allocate it. Though it's pratically right (it's the first per-vring request we will get from QEMU for vhost-user negotiation), but it's not technically right: it's not documente