From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44585) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d48SR-0007ls-Hb for qemu-devel@nongnu.org; Fri, 28 Apr 2017 12:09:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d48SM-0000Cl-Rl for qemu-devel@nongnu.org; Fri, 28 Apr 2017 12:09:23 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49226) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d48SM-0000CI-IE for qemu-devel@nongnu.org; Fri, 28 Apr 2017 12:09:18 -0400 Date: Fri, 28 Apr 2017 17:09:01 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20170428160901.GK2085@work-vm> References: <20170428021317.24711-1-eblake@redhat.com> <20170428021317.24711-3-eblake@redhat.com> <20170428080821.GB2085@work-vm> <3a8d9f68-f30c-1191-ca80-a93f0050f011@redhat.com> <20170428152729.GI2085@work-vm> <550a4e37-c4b3-181d-876c-64d73e96e78c@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <550a4e37-c4b3-181d-876c-64d73e96e78c@redhat.com> Subject: Re: [Qemu-devel] [PATCH v5 2/4] shutdown: Prepare for use of an enum in reset/shutdown_request List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-devel@nongnu.org, armbru@redhat.com, berrange@redhat.com, alistair.francis@xilinx.com, Stefano Stabellini , Anthony Perard , "Michael S. Tsirkin" , Paolo Bonzini , Richard Henderson , Eduardo Habkost , zhanghailiang , Juan Quintela , "open list:X86" * Eric Blake (eblake@redhat.com) wrote: > On 04/28/2017 10:27 AM, Dr. David Alan Gilbert wrote: > > >>>> +# Enumeration of various causes for shutdown. > >>>> +# > >>>> +# @host-qmp: Reaction to a QMP command, such as 'quit' > >>>> +# @host-signal: Reaction to a signal, such as SIGINT > >>>> +# @host-ui: Reaction to a UI event, such as closing the window > >>>> +# @host-replay: The host is replaying an earlier shutdown event > >>>> +# @host-error: Qemu encountered an error that prevents further use of the guest > >>>> +# @guest-shutdown: The guest requested a shutdown, such as via ACPI or > >>>> +# other hardware-specific action > >>>> +# @guest-reset: The guest requested a reset, and the command line > >>>> +# response to a reset is to instead trigger a shutdown > >>>> +# @guest-panic: The guest panicked, and the command line response to > >>>> +# a panic is to trigger a shutdown > >>> > > > At a higher level, using your tags, I'm not sure where a reset triggered > > by a fault detected by the hypervisor lives - e.g. an x86 triple fault > > where the guest screws up so badly that it just gets reset. Is > > that a guest-reset or a guest-panic or what - neither case > > was actually asked for by the guest itself. > > Wouldn't that be host-error (qemu detected an error that prevents > further execution of the guest without a reset - and a triple fault > seems to fall into the category of the guest getting itself wedged > rather than actually trying to reset)? Except patch 3 only used > SHUTDOWN_TYPE_HOST_ERROR in the xen portion of the patch. > > So if any x86 expert has an opinion on where triple-fault handling is > emulated, and what category should be used there, I'm welcome to > tweaking this series. It's pretty much on the border anyway, I don't think it matters too much; it sounds perfectly reasonable. Dave > -- > Eric Blake, Principal Software Engineer > Red Hat, Inc. +1-919-301-3266 > Virtualization: qemu.org | libvirt.org > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Dr. David Alan Gilbert" Subject: Re: [PATCH v5 2/4] shutdown: Prepare for use of an enum in reset/shutdown_request Date: Fri, 28 Apr 2017 17:09:01 +0100 Message-ID: <20170428160901.GK2085@work-vm> References: <20170428021317.24711-1-eblake@redhat.com> <20170428021317.24711-3-eblake@redhat.com> <20170428080821.GB2085@work-vm> <3a8d9f68-f30c-1191-ca80-a93f0050f011@redhat.com> <20170428152729.GI2085@work-vm> <550a4e37-c4b3-181d-876c-64d73e96e78c@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1d48SO-0001b2-Oh for xen-devel@lists.xenproject.org; Fri, 28 Apr 2017 16:09:20 +0000 Content-Disposition: inline In-Reply-To: <550a4e37-c4b3-181d-876c-64d73e96e78c@redhat.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Eric Blake Cc: Stefano Stabellini , berrange@redhat.com, Eduardo Habkost , "Michael S. Tsirkin" , armbru@redhat.com, Juan Quintela , qemu-devel@nongnu.org, alistair.francis@xilinx.com, zhanghailiang , "open list:X86" , Anthony Perard , Paolo Bonzini , Richard Henderson List-Id: xen-devel@lists.xenproject.org KiBFcmljIEJsYWtlIChlYmxha2VAcmVkaGF0LmNvbSkgd3JvdGU6Cj4gT24gMDQvMjgvMjAxNyAx MDoyNyBBTSwgRHIuIERhdmlkIEFsYW4gR2lsYmVydCB3cm90ZToKPiAKPiA+Pj4+ICsjIEVudW1l cmF0aW9uIG9mIHZhcmlvdXMgY2F1c2VzIGZvciBzaHV0ZG93bi4KPiA+Pj4+ICsjCj4gPj4+PiAr IyBAaG9zdC1xbXA6IFJlYWN0aW9uIHRvIGEgUU1QIGNvbW1hbmQsIHN1Y2ggYXMgJ3F1aXQnCj4g Pj4+PiArIyBAaG9zdC1zaWduYWw6IFJlYWN0aW9uIHRvIGEgc2lnbmFsLCBzdWNoIGFzIFNJR0lO VAo+ID4+Pj4gKyMgQGhvc3QtdWk6IFJlYWN0aW9uIHRvIGEgVUkgZXZlbnQsIHN1Y2ggYXMgY2xv c2luZyB0aGUgd2luZG93Cj4gPj4+PiArIyBAaG9zdC1yZXBsYXk6IFRoZSBob3N0IGlzIHJlcGxh eWluZyBhbiBlYXJsaWVyIHNodXRkb3duIGV2ZW50Cj4gPj4+PiArIyBAaG9zdC1lcnJvcjogUWVt dSBlbmNvdW50ZXJlZCBhbiBlcnJvciB0aGF0IHByZXZlbnRzIGZ1cnRoZXIgdXNlIG9mIHRoZSBn dWVzdAo+ID4+Pj4gKyMgQGd1ZXN0LXNodXRkb3duOiBUaGUgZ3Vlc3QgcmVxdWVzdGVkIGEgc2h1 dGRvd24sIHN1Y2ggYXMgdmlhIEFDUEkgb3IKPiA+Pj4+ICsjICAgICAgICAgICAgICAgICAgb3Ro ZXIgaGFyZHdhcmUtc3BlY2lmaWMgYWN0aW9uCj4gPj4+PiArIyBAZ3Vlc3QtcmVzZXQ6IFRoZSBn dWVzdCByZXF1ZXN0ZWQgYSByZXNldCwgYW5kIHRoZSBjb21tYW5kIGxpbmUKPiA+Pj4+ICsjICAg ICAgICAgICAgICAgcmVzcG9uc2UgdG8gYSByZXNldCBpcyB0byBpbnN0ZWFkIHRyaWdnZXIgYSBz aHV0ZG93bgo+ID4+Pj4gKyMgQGd1ZXN0LXBhbmljOiBUaGUgZ3Vlc3QgcGFuaWNrZWQsIGFuZCB0 aGUgY29tbWFuZCBsaW5lIHJlc3BvbnNlIHRvCj4gPj4+PiArIyAgICAgICAgICAgICAgIGEgcGFu aWMgaXMgdG8gdHJpZ2dlciBhIHNodXRkb3duCj4gPj4+Cj4gCj4gPiBBdCBhIGhpZ2hlciBsZXZl bCwgdXNpbmcgeW91ciB0YWdzLCBJJ20gbm90IHN1cmUgd2hlcmUgYSByZXNldCB0cmlnZ2VyZWQK PiA+IGJ5IGEgZmF1bHQgZGV0ZWN0ZWQgYnkgdGhlIGh5cGVydmlzb3IgbGl2ZXMgLSBlLmcuIGFu IHg4NiB0cmlwbGUgZmF1bHQKPiA+IHdoZXJlIHRoZSBndWVzdCBzY3Jld3MgdXAgc28gYmFkbHkg dGhhdCBpdCBqdXN0IGdldHMgcmVzZXQuICBJcwo+ID4gdGhhdCBhIGd1ZXN0LXJlc2V0IG9yIGEg Z3Vlc3QtcGFuaWMgb3Igd2hhdCAtIG5laXRoZXIgY2FzZQo+ID4gd2FzIGFjdHVhbGx5IGFza2Vk IGZvciBieSB0aGUgZ3Vlc3QgaXRzZWxmLgo+IAo+IFdvdWxkbid0IHRoYXQgYmUgaG9zdC1lcnJv ciAocWVtdSBkZXRlY3RlZCBhbiBlcnJvciB0aGF0IHByZXZlbnRzCj4gZnVydGhlciBleGVjdXRp b24gb2YgdGhlIGd1ZXN0IHdpdGhvdXQgYSByZXNldCAtIGFuZCBhIHRyaXBsZSBmYXVsdAo+IHNl ZW1zIHRvIGZhbGwgaW50byB0aGUgY2F0ZWdvcnkgb2YgdGhlIGd1ZXN0IGdldHRpbmcgaXRzZWxm IHdlZGdlZAo+IHJhdGhlciB0aGFuIGFjdHVhbGx5IHRyeWluZyB0byByZXNldCk/ICBFeGNlcHQg cGF0Y2ggMyBvbmx5IHVzZWQKPiBTSFVURE9XTl9UWVBFX0hPU1RfRVJST1IgaW4gdGhlIHhlbiBw b3J0aW9uIG9mIHRoZSBwYXRjaC4KPiAKPiBTbyBpZiBhbnkgeDg2IGV4cGVydCBoYXMgYW4gb3Bp bmlvbiBvbiB3aGVyZSB0cmlwbGUtZmF1bHQgaGFuZGxpbmcgaXMKPiBlbXVsYXRlZCwgYW5kIHdo YXQgY2F0ZWdvcnkgc2hvdWxkIGJlIHVzZWQgdGhlcmUsIEknbSB3ZWxjb21lIHRvCj4gdHdlYWtp bmcgdGhpcyBzZXJpZXMuCgpJdCdzIHByZXR0eSBtdWNoIG9uIHRoZSBib3JkZXIgYW55d2F5LCBJ IGRvbid0IHRoaW5rIGl0IG1hdHRlcnMgdG9vCm11Y2g7IGl0IHNvdW5kcyBwZXJmZWN0bHkgcmVh c29uYWJsZS4KCkRhdmUKCj4gLS0gCj4gRXJpYyBCbGFrZSwgUHJpbmNpcGFsIFNvZnR3YXJlIEVu Z2luZWVyCj4gUmVkIEhhdCwgSW5jLiAgICAgICAgICAgKzEtOTE5LTMwMS0zMjY2Cj4gVmlydHVh bGl6YXRpb246ICBxZW11Lm9yZyB8IGxpYnZpcnQub3JnCj4gCgoKCi0tCkRyLiBEYXZpZCBBbGFu IEdpbGJlcnQgLyBkZ2lsYmVydEByZWRoYXQuY29tIC8gTWFuY2hlc3RlciwgVUsKCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1kZXZlbCBtYWlsaW5n IGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbi5vcmcKaHR0cHM6Ly9saXN0cy54ZW4ub3JnL3hlbi1k ZXZlbAo=