From: Gleb Natapov <gleb@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Peter Wu <lekensteyn@gmail.com>,
qemu-devel@nongnu.org,
Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com>,
Jordan Justen <jordan.l.justen@intel.com>
Subject: Re: [Qemu-devel] QEMU aborts since "kvm: support using KVM_MEM_READONLY flag for regions"
Date: Tue, 4 Jun 2013 09:39:45 +0300 [thread overview]
Message-ID: <20130604063945.GI4725@redhat.com> (raw)
In-Reply-To: <87bo7nosin.fsf@codemonkey.ws>
On Mon, Jun 03, 2013 at 03:20:48PM -0500, Anthony Liguori wrote:
> Paolo Bonzini <pbonzini@redhat.com> writes:
>
> > Il 03/06/2013 19:49, Anthony Liguori ha scritto:
> >> Peter Wu <lekensteyn@gmail.com> writes:
> >>
> >>> Hi,
> >>>
> >>> With the current git source (f10acc8b38d65a66ffa0588a036489d7fa6a593e),
> >>> `qemu-system-x86_64 -enable-kvm` fails to run with the following error:
> >>>
> >>> kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument
> >>>
> >>> After this message, qemu aborts. Reverting the following commit on top of
> >>> master makes the bug go away. Removing the `-enable-kvm` option also
> >>> allows QEMU to operate, but without KVM.
> >>
> >> I also noted this on F19 this weekend.
> >>
> >> Is anyone else seeing this on more recent kernels?
> >
> > Yes, it's caused by 3.9. Gleb and I decided to consider this (raising
> > an error when removing a non-existent slot) a feature since it never
> > happened as far as we know on released versions of QEMU/nlkt/whatever.
> > Gleb queued a patch in uq/master, I think he'll be sending a pull
> > request soon.
>
> Since I suspect Gleb is done for the day, I will apply directly since
> this is a pretty major breakage.
>
Heh, since patch was on the list for 3 days before I picked it (was traveling
before that) I thought I can spend one day more for testing before
sending pull request :) The patch is already queued on uq/master and I'd
rather do not rebase this branch, so I'll push it to you anyway. git
will know how to handle this.
--
Gleb.
prev parent reply other threads:[~2013-06-04 6:40 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-03 17:23 [Qemu-devel] QEMU aborts since "kvm: support using KVM_MEM_READONLY flag for regions" Peter Wu
2013-06-03 17:35 ` Jordan Justen
2013-06-03 18:02 ` Peter Wu
2013-06-03 17:49 ` Anthony Liguori
2013-06-03 20:11 ` Paolo Bonzini
2013-06-03 20:20 ` Anthony Liguori
2013-06-04 6:39 ` Gleb Natapov [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130604063945.GI4725@redhat.com \
--to=gleb@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=jordan.l.justen@intel.com \
--cc=lekensteyn@gmail.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=xiaoguangrong@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.