Re: [Qemu-devel] [PATCH] vhost-user: pass message as a pointer to process_message_reply()

2017-05-24 Thread Jens Freimann
On Wed, May 24, 2017 at 10:42:30AM +0200, Maxime Coquelin wrote: > > > On 05/24/2017 10:40 AM, Marc-André Lureau wrote: > > > > > > On Wed, May 24, 2017 at 11:35 AM Maxime Coquelin > > mailto:maxime.coque...@redhat.com>> wrote: > > > > process_message_reply() was recently updated to get fu

Re: [Qemu-devel] [PATCH] vhost-user: pass message as a pointer to process_message_reply()

2017-05-24 Thread Yang, Zhiyong
Hi, Maxime: > -Original Message- > From: Maxime Coquelin [mailto:maxime.coque...@redhat.com] > Sent: Wednesday, May 24, 2017 4:35 PM > To: Yang, Zhiyong ; m...@redhat.com; qemu- > de...@nongnu.org; jfrei...@redhat.com; marcandre.lur...@redhat.com > Cc: Maxime Coquelin > Subject: [PATCH] v

Re: [Qemu-devel] [PATCH] vhost-user: pass message as a pointer to process_message_reply()

2017-05-24 Thread Maxime Coquelin
On 05/24/2017 10:40 AM, Marc-André Lureau wrote: On Wed, May 24, 2017 at 11:35 AM Maxime Coquelin mailto:maxime.coque...@redhat.com>> wrote: process_message_reply() was recently updated to get full message content instead of only its request field. There is no need to copy all

Re: [Qemu-devel] [PATCH] vhost-user: pass message as a pointer to process_message_reply()

2017-05-24 Thread Marc-André Lureau
On Wed, May 24, 2017 at 11:35 AM Maxime Coquelin wrote: > process_message_reply() was recently updated to get full message > content instead of only its request field. > > There is no need to copy all the struct content into the stack, > so just pass its pointer. > > Cc: Zhiyong Yang > Fixes: 60

[Qemu-devel] [PATCH] vhost-user: pass message as a pointer to process_message_reply()

2017-05-24 Thread Maxime Coquelin
process_message_reply() was recently updated to get full message content instead of only its request field. There is no need to copy all the struct content into the stack, so just pass its pointer. Cc: Zhiyong Yang Fixes: 60cd11024f41 ("hw/virtio: fix vhost user fails to startup when MQ") Signed