Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 03:31:05PM +, Felipe Franciosi wrote: > > > On 10 Oct 2016, at 16:24, Michael S. Tsirkin wrote: > > > > On Mon, Oct 10, 2016 at 03:17:36PM +, Felipe Franciosi wrote: > >> > >>> On 9 Oct 2016, at 23:33, Michael S. Tsirkin wrote:

Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-10-10 Thread Felipe Franciosi
> On 9 Oct 2016, at 23:33, Michael S. Tsirkin wrote: > > On Thu, Sep 22, 2016 at 01:13:41PM +0100, Felipe Franciosi wrote: >> Vhost-user requires an early GET_FEATURES call to determine if the >> slave supports protocol feature negotiation. An extra GET_FEATURES >> call is made

Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-10-10 Thread Felipe Franciosi
> On 10 Oct 2016, at 16:24, Michael S. Tsirkin wrote: > > On Mon, Oct 10, 2016 at 03:17:36PM +, Felipe Franciosi wrote: >> >>> On 9 Oct 2016, at 23:33, Michael S. Tsirkin wrote: >>> >>> On Thu, Sep 22, 2016 at 01:13:41PM +0100, Felipe Franciosi wrote:

Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-10-10 Thread Michael S. Tsirkin
On Mon, Oct 10, 2016 at 03:17:36PM +, Felipe Franciosi wrote: > > > On 9 Oct 2016, at 23:33, Michael S. Tsirkin wrote: > > > > On Thu, Sep 22, 2016 at 01:13:41PM +0100, Felipe Franciosi wrote: > >> Vhost-user requires an early GET_FEATURES call to determine if the > >>

Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-10-09 Thread Michael S. Tsirkin
On Thu, Sep 22, 2016 at 01:13:41PM +0100, Felipe Franciosi wrote: > Vhost-user requires an early GET_FEATURES call to determine if the > slave supports protocol feature negotiation. An extra GET_FEATURES > call is made after vhost_backend_init() to actually set the device > features. > > This

Re: [Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-09-23 Thread Michael S. Tsirkin
On Thu, Sep 22, 2016 at 01:13:41PM +0100, Felipe Franciosi wrote: > Vhost-user requires an early GET_FEATURES call to determine if the > slave supports protocol feature negotiation. An extra GET_FEATURES > call is made after vhost_backend_init() to actually set the device > features. > > This

[Qemu-devel] [PATCH v2] Avoid additional GET_FEATURES call on vhost-user

2016-09-22 Thread Felipe Franciosi
Vhost-user requires an early GET_FEATURES call to determine if the slave supports protocol feature negotiation. An extra GET_FEATURES call is made after vhost_backend_init() to actually set the device features. This patch moves the actual setting of the device features to both implementations