From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:35700) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RqPwA-0008Tx-PY for qemu-devel@nongnu.org; Thu, 26 Jan 2012 09:04:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RqPw5-0005Qv-6d for qemu-devel@nongnu.org; Thu, 26 Jan 2012 09:04:26 -0500 Received: from mx1.redhat.com ([209.132.183.28]:28772) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RqPw4-0005Qd-VM for qemu-devel@nongnu.org; Thu, 26 Jan 2012 09:04:21 -0500 Date: Thu, 26 Jan 2012 16:05:43 +0200 From: "Michael S. Tsirkin" Message-ID: <20120126140543.GC17198@redhat.com> References: <4F1F971B.4020309@endace.com> <20120125153415.GA10267@redhat.com> <1327517961.26484.124.camel@bling.home> <4F20C651.2010108@endace.com> <4F215A4A.1000400@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4F215A4A.1000400@redhat.com> Subject: Re: [Qemu-devel] [RFC/PATCH] Fix guest OS panic when 64bit BAR is present List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Alexey Korolev , sfd@endace.com, Alex Williamson , Kevin O'Connor , "qemu-devel@nongnu.org" On Thu, Jan 26, 2012 at 03:51:06PM +0200, Avi Kivity wrote: > > Please look at HPET lines. HPET is mapped to 0xfed00000. > > Size of ivshmem is 32MB. During pci enumeration ivshmem will corrupt the range from 0xfe000000 - 0xffffffff. > > It overlaps HPET memory. When Linux does late_hpet init, it finds garbage and this is causing panic. > > > > Let me see if I get this right: during BAR sizing, the guest sets the > BAR to ~1, which means 4GB-32MB -> 4GB, which overlaps the HPET. If so, > that's expected behaviour. Yes BAR sizing temporarily sets the BAR to an invalid value then restores it. What I don't understand is how come something accesses the HPET range in between. > If the guest doesn't want this memory there, > it should disable mmio. Recent kernels do this for most devices, but not for platform devices. > -- > error compiling committee.c: too many arguments to function