Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-10 Thread liu ping fan
On Thu, May 9, 2013 at 11:26 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Thu, May 09, 2013 at 05:00:20PM +0800, liu ping fan wrote: On Thu, May 9, 2013 at 4:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-10 Thread Stefan Hajnoczi
On Fri, May 10, 2013 at 02:03:34PM +0800, liu ping fan wrote: On Thu, May 9, 2013 at 11:26 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Thu, May 09, 2013 at 05:00:20PM +0800, liu ping fan wrote: On Thu, May 9, 2013 at 4:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, May 09,

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-10 Thread liu ping fan
On Fri, May 10, 2013 at 3:12 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Fri, May 10, 2013 at 02:03:34PM +0800, liu ping fan wrote: On Thu, May 9, 2013 at 11:26 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Thu, May 09, 2013 at 05:00:20PM +0800, liu ping fan wrote: On Thu, May 9,

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread Stefan Hajnoczi
On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener priority than kvm. For currently code, take the following scenario: kvm_region_add() run earlier before

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread Michael S. Tsirkin
On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener priority than kvm. For currently code, take the following scenario: kvm_region_add() run earlier before

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread liu ping fan
On Thu, May 9, 2013 at 4:44 PM, Michael S. Tsirkin m...@redhat.com wrote: On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener priority than kvm. For currently code, take

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread liu ping fan
On Thu, May 9, 2013 at 4:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener priority than kvm. For currently code, take

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread Paolo Bonzini
Il 09/05/2013 10:54, liu ping fan ha scritto: On Thu, May 9, 2013 at 4:44 PM, Michael S. Tsirkin m...@redhat.com wrote: On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener

Re: [Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-09 Thread Stefan Hajnoczi
On Thu, May 09, 2013 at 05:00:20PM +0800, liu ping fan wrote: On Thu, May 9, 2013 at 4:30 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, May 09, 2013 at 08:40:21AM +0800, Liu Ping Fan wrote: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have

[Qemu-devel] [PATCH 1/2] Vring: vring's listener's priority should higher than kvm

2013-05-08 Thread Liu Ping Fan
From: Liu Ping Fan pingf...@linux.vnet.ibm.com Hosts threads which handle vring should have high MemoryListener priority than kvm. For currently code, take the following scenario: kvm_region_add() run earlier before vhost_region_add(), then in guest, vring's desc[i] can refer to addressX in the