From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48510) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V9HEV-0007mD-4g for qemu-devel@nongnu.org; Tue, 13 Aug 2013 12:14:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V9HEN-00032a-BI for qemu-devel@nongnu.org; Tue, 13 Aug 2013 12:14:07 -0400 Received: from e32.co.us.ibm.com ([32.97.110.150]:33186) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V9HEN-00031f-2o for qemu-devel@nongnu.org; Tue, 13 Aug 2013 12:13:59 -0400 Received: from /spool/local by e32.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 13 Aug 2013 10:13:58 -0600 Received: from d03relay03.boulder.ibm.com (d03relay03.boulder.ibm.com [9.17.195.228]) by d03dlp03.boulder.ibm.com (Postfix) with ESMTP id AED9919D8041 for ; Tue, 13 Aug 2013 10:12:53 -0600 (MDT) Received: from d03av01.boulder.ibm.com (d03av01.boulder.ibm.com [9.17.195.167]) by d03relay03.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r7DGCs4w180846 for ; Tue, 13 Aug 2013 10:12:55 -0600 Received: from d03av01.boulder.ibm.com (loopback [127.0.0.1]) by d03av01.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r7DGCpbR028050 for ; Tue, 13 Aug 2013 10:12:53 -0600 From: Anthony Liguori In-Reply-To: <520A575F.7040302@suse.de> References: <20130813125349.GA28317@redhat.com> <87a9klllyz.fsf@codemonkey.ws> <520A48C0.5000205@redhat.com> <520A4C8C.7090202@suse.de> <520A5059.10009@redhat.com> <520A575F.7040302@suse.de> Date: Tue, 13 Aug 2013 11:12:46 -0500 Message-ID: <87bo517eo1.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH for-1.6] pc: fix up pc initialization List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?utf-8?Q?F=C3=A4rber?= , Paolo Bonzini Cc: marcel.a@redhat.com, "Michael S. Tsirkin" , qemu-devel@nongnu.org, Michael Roth , Gerd Hoffmann , Igor Mammedov Andreas F=C3=A4rber writes: > Am 13.08.2013 17:27, schrieb Paolo Bonzini: >> Il 13/08/2013 17:11, Andreas F=C3=A4rber ha scritto: >>> >>> mst suggested to patch stable-1.5 to disable it there, too. I am not >>> against but have doubts as to how well that works with migration, since >>> 1.5.3 is still a bit off and I would expect 1.5.2 -> 1.6.0 migration to >>> work without guest-visible changes... We could argue that having to use >>> -M pc-i440fx-1.5 we can also expect users to add -device pvpanic; >>> question would be how to convey that knowledge of >>> if-you-use-pc-x.y-then-you-also-need-to-do-Z to users, which >>> compat_props usually handle under the hood. We could misuse >>> pvpanic.ioport=3D0 for that purpose until we have a better solution. >>=20 >> pvpanic has no vmstate though, has it? So it won't break migration >> (needs testing of course). > > Just checked, you're right, it doesn't. I somehow assumed it would. > >> All that should happen is that after migration you will not get panic >> notifications on the destination. > > Well, how does the Linux driver cope with pvpanic device present on boot > but not present on panic? ISA PIO is not usually expected to be > hot-unplugged. ;) Yeah, disappearing doesn't seem acceptable to me. We made a mistake with 1.5, now we have to live with it. Regards, Anthony Liguori > > Andreas > > --=20 > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3= =BCrnberg