From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42450) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VCD6r-0006xW-KC for qemu-devel@nongnu.org; Wed, 21 Aug 2013 14:26:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VCD6n-0003St-92 for qemu-devel@nongnu.org; Wed, 21 Aug 2013 14:26:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44745) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VCD6n-0003Sd-07 for qemu-devel@nongnu.org; Wed, 21 Aug 2013 14:26:17 -0400 Date: Wed, 21 Aug 2013 17:55:30 +0100 From: "Daniel P. Berrange" Message-ID: <20130821165529.GE10012@redhat.com> References: <1377103396-24307-1-git-send-email-pbonzini@redhat.com> <20130821164817.GC10012@redhat.com> <5214EFFF.3060804@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <5214EFFF.3060804@redhat.com> Subject: Re: [Qemu-devel] [RFC PATCH v2 0/3] Start fixing the pvpanic mess Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: pkrempa@redhat.com, marcel.a@redhat.com, libvir-list@redhat.com, lersek@redhat.com, mst@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, rhod@redhat.com, kraxel@redhat.com, anthony@codemonkey.ws, hutao@cn.fujitsu.com, lcapitulino@redhat.com, afaerber@suse.de On Wed, Aug 21, 2013 at 06:51:11PM +0200, Paolo Bonzini wrote: > Il 21/08/2013 18:48, Daniel P. Berrange ha scritto: > > No, is the right thing to be using for this from > > libvirt's pov & I don't think we should invent something new. > > The element has always been intended to represent > > handling of guest panics, not qemu internal errors. > > Actually for Xen HVM guests, it mostly traps things such as failed > vmentries. The Xen PV-on-HVM drivers do not register a panic notifier > that moves the guest to the "crashed" state. > > cannot be salvaged, in my opinion, because all domain XMLs in > the wild will have a setting that causes libvirt to add "-device > isa-pvpanic". Thus changing libvirt versions will change guest > hardware, which is _very_ bad. > > In addition, Windows XP and 2003 will show the annoying device wizard > upon a libvirt upgrade, and fixing this is what surfaced all the mess. The existance of a element should not be having any effect on what hardware we create. That is merely a lifecycle policy setting that should be completely independant of the guest device model. eg it is valid to have present in the XML at all times, even if there's no pvpanic device present. That simply means the actions will never be triggered. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|