From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MwydF-0006Hb-Pz for qemu-devel@nongnu.org; Sun, 11 Oct 2009 09:38:41 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MwydA-0006AX-Lt for qemu-devel@nongnu.org; Sun, 11 Oct 2009 09:38:40 -0400 Received: from [199.232.76.173] (port=38635 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MwydA-0006AM-E1 for qemu-devel@nongnu.org; Sun, 11 Oct 2009 09:38:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:31673) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Mwyd9-0004Mw-UF for qemu-devel@nongnu.org; Sun, 11 Oct 2009 09:38:36 -0400 Date: Sun, 11 Oct 2009 15:36:26 +0200 From: "Michael S. Tsirkin" Subject: Re: [Qemu-devel] Re: [PATCH 2/3] qemu: make cirrus init value pci spec compliant Message-ID: <20091011133626.GA10229@redhat.com> References: <20091008145241.GC12275@redhat.com> <4ACE0559.5040206@redhat.com> <20091008160623.GA13710@redhat.com> <4ACE0FC6.3050908@redhat.com> <20091008184011.GA6576@shareable.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091008184011.GA6576@shareable.org> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jamie Lokier Cc: gleb@redhat.com, Anthony Liguori , Avi Kivity , kvm-devel , "qemu-devel@nongnu.org" On Thu, Oct 08, 2009 at 07:40:12PM +0100, Jamie Lokier wrote: > Avi Kivity wrote: > > On 10/08/2009 06:06 PM, Michael S. Tsirkin wrote: > > >On Thu, Oct 08, 2009 at 05:29:29PM +0200, Avi Kivity wrote: > > > > > >>On 10/08/2009 04:52 PM, Michael S. Tsirkin wrote: > > >> > > >>>PCI memory should be disabled at reset, otherwise > > >>>we might claim transactions at address 0. > > >>>I/O should also be disabled, although for cirrus > > >>>it is harmless to enable it as we do not > > >>>have I/O bar. > > >>> > > >>>Note: need bios fix for this patch to work: > > >>>currently pc-bios incorrently assumes that it does not > > >>>need to enable i/o unless device has i/o bar. > > >>> > > >>>Signed-off-by: Michael S. Tsirkin > > >> > > >>This needs to be conditional on the machine type. Old machines must > > >>retain this code for live migration to work (we need to live migrate the > > >>bios, so we can't assume the bios fix is in during live migration from > > >>older qemus). > > >> > > >No, if you migrate from older qemu you will be fine as command > > >is enabled there on init. > > > > > > > Right. > > No, I think Avi was right the first time. > > Migrating from an older qemu will be fine at first, but at the next > reset _following_ migration, it'll be running the old BIOS on a new > qemu and fail. > > -- Jamie So after discussion with Gleb, it seems that what will happen after migration to new qemu, old BIOS is shadowed, but memory is already enabled. After reset, new BIOS will be shadowed and it will enable memory. Makes sense? -- MST