Re: [Qemu-devel] [BUG]: kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument

[Jordan Justen]

On Thu, May 30, 2013 at 9:08 AM, Luiz Capitulino <address@hidden> wrote:
> On Thu, 30 May 2013 18:03:04 +0200
> Paolo Bonzini <address@hidden> wrote:
>
>> Il 30/05/2013 17:46, Luiz Capitulino ha scritto:
>> > The culprit is commit:
>> >
>> > commit 235e8982ad393e5611cb892df54881c872eea9e1
>> > Author: Jordan Justen <address@hidden>
>> > Date:   Wed May 29 01:27:26 2013 -0700
>> >
>> >     kvm: support using KVM_MEM_READONLY flag for regions
>> >
>> > I'm running 3.9.2-200.fc18, btw. And, error checking is missing on the
>> > first call to kvm_vm_ioctl().

As noted in the code, the first call is for KVM commit 75d61fbc.

I'm not sure we want to fail if an error occurs when making that call.
(I'm pretty sure we don't want to in fact.)

Xiao, any thoughts?

>> Reproducer?
>
> I just try to start a VM (HEAD 87d23f7):
>
> ~/work/virt/ sudo ./qemu-qmp -drive 
> file=disks/test.img,if=virtio,cache=none,aio=native -enable-kvm -m 1G 
> -monitor stdio -cpu host -snapshot
> QEMU 1.5.50 monitor - type 'help' for more information
> (qemu) kvm_set_phys_mem: error unregistering overlapping slot: Invalid 
> argument
> ~/work/virt/

Sorry. I am working with Linux 3.8.0, and I don't see this. I'll try
to update my kernel.

Does the firmware behave as a ROM for you?

-Jordan