From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37641) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UjkuU-0005GS-Ro for qemu-devel@nongnu.org; Tue, 04 Jun 2013 02:40:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UjkuQ-0003aj-7Q for qemu-devel@nongnu.org; Tue, 04 Jun 2013 02:39:58 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54457) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UjkuP-0003aZ-Vg for qemu-devel@nongnu.org; Tue, 04 Jun 2013 02:39:54 -0400 Date: Tue, 4 Jun 2013 09:39:45 +0300 From: Gleb Natapov Message-ID: <20130604063945.GI4725@redhat.com> References: <11353237.fLVsHzmD7c@al> <878v2rf5iy.fsf@codemonkey.ws> <51ACF872.1080007@redhat.com> <87bo7nosin.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87bo7nosin.fsf@codemonkey.ws> Subject: Re: [Qemu-devel] QEMU aborts since "kvm: support using KVM_MEM_READONLY flag for regions" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Paolo Bonzini , Peter Wu , qemu-devel@nongnu.org, Xiao Guangrong , Jordan Justen On Mon, Jun 03, 2013 at 03:20:48PM -0500, Anthony Liguori wrote: > Paolo Bonzini writes: > > > Il 03/06/2013 19:49, Anthony Liguori ha scritto: > >> Peter Wu 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.