Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-10 Thread David Gibson
On Wed, Aug 10, 2011 at 12:01:04PM +0300, Avi Kivity wrote: > On 08/10/2011 08:10 AM, David Gibson wrote: > >On Mon, Aug 08, 2011 at 11:24:09AM +0300, Avi Kivity wrote: > >> On 08/08/2011 09:03 AM, David Gibson wrote: [snip] > >This would seem to be a genuine bug in the hugepage code, which has >

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-10 Thread Avi Kivity
On 08/10/2011 08:10 AM, David Gibson wrote: On Mon, Aug 08, 2011 at 11:24:09AM +0300, Avi Kivity wrote: > On 08/08/2011 09:03 AM, David Gibson wrote: > >Second, if userspace qemu passing hugepages to kvm can cause (host) > >kernel memory corruption, that is clearly a host kernel bug. So am I

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-09 Thread David Gibson
On Mon, Aug 08, 2011 at 11:24:09AM +0300, Avi Kivity wrote: > On 08/08/2011 09:03 AM, David Gibson wrote: > >Second, if userspace qemu passing hugepages to kvm can cause (host) > >kernel memory corruption, that is clearly a host kernel bug. So am I > >correct in thinking this is basically just a s

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-08 Thread Avi Kivity
On 08/08/2011 09:03 AM, David Gibson wrote: Second, if userspace qemu passing hugepages to kvm can cause (host) kernel memory corruption, that is clearly a host kernel bug. So am I correct in thinking this is basically just a safety feature if qemu is run on a buggy kernel. Seems so, yes. 2.6

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-07 Thread David Gibson
On Fri, Aug 05, 2011 at 12:30:53PM -0300, Marcelo Tosatti wrote: > On Fri, Aug 05, 2011 at 08:16:42AM +0200, Jan Kiszka wrote: > > On 2011-08-05 06:02, David Gibson wrote: > > > At present, an explicit test disallows use of -mem-path when kvm is > > > enabled > > > but KVM_CAP_SYNC_MMU is not set.

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-05 Thread Marcelo Tosatti
On Fri, Aug 05, 2011 at 08:16:42AM +0200, Jan Kiszka wrote: > On 2011-08-05 06:02, David Gibson wrote: > > At present, an explicit test disallows use of -mem-path when kvm is enabled > > but KVM_CAP_SYNC_MMU is not set. In particular, this prevents the user > > from using hugetlbfs to back the gue

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-04 Thread Jan Kiszka
On 2011-08-05 06:02, David Gibson wrote: > At present, an explicit test disallows use of -mem-path when kvm is enabled > but KVM_CAP_SYNC_MMU is not set. In particular, this prevents the user > from using hugetlbfs to back the guest memory. > > I can see no reason for this check, and when I asked

[Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-04 Thread David Gibson
At present, an explicit test disallows use of -mem-path when kvm is enabled but KVM_CAP_SYNC_MMU is not set. In particular, this prevents the user from using hugetlbfs to back the guest memory. I can see no reason for this check, and when I asked about it previously, the only theory offered was t