Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2018-01-03 Thread Zhoujian (jay)
i (Arei) > ; Zhoujian (jay) > Subject: Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots > number > > On Fri, 22 Dec 2017 23:15:09 +0200 > "Michael S. Tsirkin" wrote: > > > On Fri, Dec 22, 2017 at 07:48:55PM +0100, Igor Mammedov wrote: >

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-28 Thread Igor Mammedov
On Fri, 22 Dec 2017 23:15:09 +0200 "Michael S. Tsirkin" wrote: > On Fri, Dec 22, 2017 at 07:48:55PM +0100, Igor Mammedov wrote: > > On Fri, 15 Dec 2017 16:45:55 +0800 > > Jay Zhou wrote: > > > > > If the VM already has N(N>8) available memory slots for vhost user, > > > the VM will be crashed i

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-28 Thread Igor Mammedov
On Sat, 23 Dec 2017 08:27:25 + "Zhoujian (jay)" wrote: > > > > -Original Message- > > From: Igor Mammedov [mailto:imamm...@redhat.com] > > Sent: Saturday, December 23, 2017 2:49 AM > > To: Zhoujian (jay) > > Cc: qemu-devel@nongnu.org; m...@redhat.com; Huangweidong (C) > > ; Gonglei

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-23 Thread Zhoujian (jay)
[...] > > --- > > hw/virtio/vhost.c | 27 +++ > > 1 file changed, 23 insertions(+), 4 deletions(-) > > > > diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c index > > 59a32e9..e45f5e2 100644 > > --- a/hw/virtio/vhost.c > > +++ b/hw/virtio/vhost.c > > @@ -1234,6 +1234,18

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-23 Thread Zhoujian (jay)
> -Original Message- > From: Igor Mammedov [mailto:imamm...@redhat.com] > Sent: Saturday, December 23, 2017 2:49 AM > To: Zhoujian (jay) > Cc: qemu-devel@nongnu.org; m...@redhat.com; Huangweidong (C) > ; Gonglei (Arei) ; > wangxin (U) ; Liuzhe (Cloud Open Labs, NFV) > ; dgilb...@redhat.c

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-22 Thread Michael S. Tsirkin
On Fri, Dec 22, 2017 at 07:48:55PM +0100, Igor Mammedov wrote: > On Fri, 15 Dec 2017 16:45:55 +0800 > Jay Zhou wrote: > > > If the VM already has N(N>8) available memory slots for vhost user, > > the VM will be crashed in vhost_user_set_mem_table if we try to > > hotplug the first vhost user NIC.

Re: [Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-22 Thread Igor Mammedov
On Fri, 15 Dec 2017 16:45:55 +0800 Jay Zhou wrote: > If the VM already has N(N>8) available memory slots for vhost user, > the VM will be crashed in vhost_user_set_mem_table if we try to > hotplug the first vhost user NIC. > This patch checks if memslots number exceeded or not after updating > vh

[Qemu-devel] [PATCH v2 2/2] vhost: double check used memslots number

2017-12-15 Thread Jay Zhou
If the VM already has N(N>8) available memory slots for vhost user, the VM will be crashed in vhost_user_set_mem_table if we try to hotplug the first vhost user NIC. This patch checks if memslots number exceeded or not after updating vhost_user_used_memslots. Signed-off-by: Jay Zhou --- hw/virti