Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-11 Thread Kevin O'Connor
On Tue, Sep 11, 2012 at 11:15:50AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > On 2012-09-11 05:02, Kevin O'Connor wrote: > >> The easiest way to fix this is to change QEMU to boot with the area > >> read-write. There's no real gain in booting with the memory read-only > >> as the firs

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-11 Thread Anthony Liguori
Jan Kiszka writes: > On 2012-09-11 05:02, Kevin O'Connor wrote: >> On Mon, Sep 10, 2012 at 11:25:38AM +0200, Jan Kiszka wrote: >>> On 2012-09-09 17:45, Avi Kivity wrote: On 09/07/2012 11:50 AM, Jan Kiszka wrote: > >> +} else { >> +cpu_physical_memory_r

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-11 Thread Jan Kiszka
On 2012-09-11 18:15, Anthony Liguori wrote: > Jan Kiszka writes: > >> On 2012-09-11 05:02, Kevin O'Connor wrote: >>> On Mon, Sep 10, 2012 at 11:25:38AM +0200, Jan Kiszka wrote: On 2012-09-09 17:45, Avi Kivity wrote: > On 09/07/2012 11:50 AM, Jan Kiszka wrote: >> >>> +

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-11 Thread Jan Kiszka
On 2012-09-11 05:02, Kevin O'Connor wrote: > On Mon, Sep 10, 2012 at 11:25:38AM +0200, Jan Kiszka wrote: >> On 2012-09-09 17:45, Avi Kivity wrote: >>> On 09/07/2012 11:50 AM, Jan Kiszka wrote: > +} else { > +cpu_physical_memory_rw(run->mmio.phys_addr, >

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-10 Thread Kevin O'Connor
On Mon, Sep 10, 2012 at 11:25:38AM +0200, Jan Kiszka wrote: > On 2012-09-09 17:45, Avi Kivity wrote: > > On 09/07/2012 11:50 AM, Jan Kiszka wrote: > >> > >>> +} else { > >>> +cpu_physical_memory_rw(run->mmio.phys_addr, > >>> + run->m

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-10 Thread Jan Kiszka
On 2012-09-09 17:45, Avi Kivity wrote: > On 09/07/2012 11:50 AM, Jan Kiszka wrote: >> >>> +} else { >>> +cpu_physical_memory_rw(run->mmio.phys_addr, >>> + run->mmio.data, >>> + run->mmio.len, >>>

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-09 Thread Avi Kivity
On 09/07/2012 11:50 AM, Jan Kiszka wrote: > >> +} else { >> +cpu_physical_memory_rw(run->mmio.phys_addr, >> + run->mmio.data, >> + run->mmio.len, >> + run->

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-09 Thread Avi Kivity
On 09/07/2012 11:26 AM, Liu Sheng wrote: > as readonly memory is support in kvm, this patch supports this feature > in qemu, mainly pluging the memory region with KVM_MEM_READONLY flag > to kvm. > > @@ -1607,10 +1623,19 @@ int kvm_cpu_exec(CPUArchState *env) > break; > case K

Re: [Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-07 Thread Jan Kiszka
On 2012-09-07 10:26, Liu Sheng wrote: > as readonly memory is support in kvm, this patch supports this feature > in qemu, mainly pluging the memory region with KVM_MEM_READONLY flag > to kvm. > > this can be tested by a kernel module from the author of kvm readonly > memory slot: > > static int r

[Qemu-devel] [PATCH 3/3] support readonly memory feature in qemu

2012-09-07 Thread Liu Sheng
as readonly memory is support in kvm, this patch supports this feature in qemu, mainly pluging the memory region with KVM_MEM_READONLY flag to kvm. this can be tested by a kernel module from the author of kvm readonly memory slot: static int rom_tester_probe(struct pci_dev *dev, const struct pci_