From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55638) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VC9Yt-0006Cl-3S for qemu-devel@nongnu.org; Wed, 21 Aug 2013 10:39:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VC9Yn-0002fQ-4r for qemu-devel@nongnu.org; Wed, 21 Aug 2013 10:39:03 -0400 Message-ID: <5214D0C4.2010503@redhat.com> Date: Wed, 21 Aug 2013 16:37:56 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1377086477-19553-1-git-send-email-pbonzini@redhat.com> <5214B5BE.1030906@redhat.com> <5214B5DF.5040100@redhat.com> <20130821101749.5b95b2e4@redhat.com> <20130821143030.GA10645@redhat.com> In-Reply-To: <20130821143030.GA10645@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] vl: allow "cont" from panicked state List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: qemu-devel@nongnu.org, pkrempa@redhat.com, marcel.a@redhat.com, libvir-list@redhat.com, hutao@cn.fujitsu.com, qemu-stable@nongnu.org, Luiz Capitulino , rhod@redhat.com, kraxel@redhat.com, anthony@codemonkey.ws, Laszlo Ersek , afaerber@suse.de Il 21/08/2013 16:30, Michael S. Tsirkin ha scritto: >> I think the same reasoning went behind the PANICKED state, and for most >> cases it's going to be disastrous to put the guest to run again, > > Why will it? It will most likely just call halt a bit later. I agree. >> but I can understand that this is up user/mngt to decide this, not QEMU. > > I don't have a problem with this patch as such, so > > Acked-by: Michael S. Tsirkin > > though I'm still not really sure why do we > want to block guest immediately on panic. > Why not let it call halt a bit later? To make sure the panic is detected, and action taken, in the host even if management has crashed at the time. For example, even if you have reboot-on-panic active, management has time to take a core dump of the paused guest _before_ the reboot. Paolo