From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51727) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d4AOU-0000lV-Pm for qemu-devel@nongnu.org; Fri, 28 Apr 2017 14:13:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d4AOR-0006kn-Gc for qemu-devel@nongnu.org; Fri, 28 Apr 2017 14:13:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54600) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d4AOR-0006kL-70 for qemu-devel@nongnu.org; Fri, 28 Apr 2017 14:13:23 -0400 Date: Fri, 28 Apr 2017 19:13:14 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20170428181314.GJ3276@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> <20170428160901.GK2085@work-vm> <2b7417be-559c-b1e5-eaf4-e195d611b4d6@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2b7417be-559c-b1e5-eaf4-e195d611b4d6@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 11:09 AM, Dr. David Alan Gilbert wrote: > > >>> 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. > > Actually, reading > https://blogs.msdn.microsoft.com/larryosterman/2005/02/08/faster-syscall-trap-redux/ > makes it sound like the triple-fault = reset is exploited by existing OS > (dating back to days of targetting 286 machines), so it is bare-metal > behavior that we have to faithfully emulate as a guest-triggered reset, > and not something where the guest has wedged itself to the point where > qemu can no longer execute the guest. The point is it's both :-) A lot of x86 reset code tries four or five different ways to invoke a reset and if all else fails they triple fault. 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 19:13:14 +0100 Message-ID: <20170428181314.GJ3276@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> <20170428160901.GK2085@work-vm> <2b7417be-559c-b1e5-eaf4-e195d611b4d6@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 1d4AOT-0001aZ-BF for xen-devel@lists.xenproject.org; Fri, 28 Apr 2017 18:13:25 +0000 Content-Disposition: inline In-Reply-To: <2b7417be-559c-b1e5-eaf4-e195d611b4d6@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 MTowOSBBTSwgRHIuIERhdmlkIEFsYW4gR2lsYmVydCB3cm90ZToKPiAKPiA+Pj4gQXQgYSBoaWdo ZXIgbGV2ZWwsIHVzaW5nIHlvdXIgdGFncywgSSdtIG5vdCBzdXJlIHdoZXJlIGEgcmVzZXQgdHJp Z2dlcmVkCj4gPj4+IGJ5IGEgZmF1bHQgZGV0ZWN0ZWQgYnkgdGhlIGh5cGVydmlzb3IgbGl2ZXMg LSBlLmcuIGFuIHg4NiB0cmlwbGUgZmF1bHQKPiA+Pj4gd2hlcmUgdGhlIGd1ZXN0IHNjcmV3cyB1 cCBzbyBiYWRseSB0aGF0IGl0IGp1c3QgZ2V0cyByZXNldC4gIElzCj4gPj4+IHRoYXQgYSBndWVz dC1yZXNldCBvciBhIGd1ZXN0LXBhbmljIG9yIHdoYXQgLSBuZWl0aGVyIGNhc2UKPiA+Pj4gd2Fz IGFjdHVhbGx5IGFza2VkIGZvciBieSB0aGUgZ3Vlc3QgaXRzZWxmLgo+ID4+Cj4gPj4gV291bGRu J3QgdGhhdCBiZSBob3N0LWVycm9yIChxZW11IGRldGVjdGVkIGFuIGVycm9yIHRoYXQgcHJldmVu dHMKPiA+PiBmdXJ0aGVyIGV4ZWN1dGlvbiBvZiB0aGUgZ3Vlc3Qgd2l0aG91dCBhIHJlc2V0IC0g YW5kIGEgdHJpcGxlIGZhdWx0Cj4gPj4gc2VlbXMgdG8gZmFsbCBpbnRvIHRoZSBjYXRlZ29yeSBv ZiB0aGUgZ3Vlc3QgZ2V0dGluZyBpdHNlbGYgd2VkZ2VkCj4gPj4gcmF0aGVyIHRoYW4gYWN0dWFs bHkgdHJ5aW5nIHRvIHJlc2V0KT8gIEV4Y2VwdCBwYXRjaCAzIG9ubHkgdXNlZAo+ID4+IFNIVVRE T1dOX1RZUEVfSE9TVF9FUlJPUiBpbiB0aGUgeGVuIHBvcnRpb24gb2YgdGhlIHBhdGNoLgo+ID4+ Cj4gPj4gU28gaWYgYW55IHg4NiBleHBlcnQgaGFzIGFuIG9waW5pb24gb24gd2hlcmUgdHJpcGxl LWZhdWx0IGhhbmRsaW5nIGlzCj4gPj4gZW11bGF0ZWQsIGFuZCB3aGF0IGNhdGVnb3J5IHNob3Vs ZCBiZSB1c2VkIHRoZXJlLCBJJ20gd2VsY29tZSB0bwo+ID4+IHR3ZWFraW5nIHRoaXMgc2VyaWVz Lgo+ID4gCj4gPiBJdCdzIHByZXR0eSBtdWNoIG9uIHRoZSBib3JkZXIgYW55d2F5LCBJIGRvbid0 IHRoaW5rIGl0IG1hdHRlcnMgdG9vCj4gPiBtdWNoOyBpdCBzb3VuZHMgcGVyZmVjdGx5IHJlYXNv bmFibGUuCj4gCj4gQWN0dWFsbHksIHJlYWRpbmcKPiBodHRwczovL2Jsb2dzLm1zZG4ubWljcm9z b2Z0LmNvbS9sYXJyeW9zdGVybWFuLzIwMDUvMDIvMDgvZmFzdGVyLXN5c2NhbGwtdHJhcC1yZWR1 eC8KPiBtYWtlcyBpdCBzb3VuZCBsaWtlIHRoZSB0cmlwbGUtZmF1bHQgPSByZXNldCBpcyBleHBs b2l0ZWQgYnkgZXhpc3RpbmcgT1MKPiAoZGF0aW5nIGJhY2sgdG8gZGF5cyBvZiB0YXJnZXR0aW5n IDI4NiBtYWNoaW5lcyksIHNvIGl0IGlzIGJhcmUtbWV0YWwKPiBiZWhhdmlvciB0aGF0IHdlIGhh dmUgdG8gZmFpdGhmdWxseSBlbXVsYXRlIGFzIGEgZ3Vlc3QtdHJpZ2dlcmVkIHJlc2V0LAo+IGFu ZCBub3Qgc29tZXRoaW5nIHdoZXJlIHRoZSBndWVzdCBoYXMgd2VkZ2VkIGl0c2VsZiB0byB0aGUg cG9pbnQgd2hlcmUKPiBxZW11IGNhbiBubyBsb25nZXIgZXhlY3V0ZSB0aGUgZ3Vlc3QuCgpUaGUg cG9pbnQgaXMgaXQncyBib3RoIDotKQpBIGxvdCBvZiB4ODYgcmVzZXQgY29kZSB0cmllcyBmb3Vy IG9yIGZpdmUgZGlmZmVyZW50IHdheXMgdG8gaW52b2tlCmEgcmVzZXQgYW5kIGlmIGFsbCBlbHNl IGZhaWxzIHRoZXkgdHJpcGxlIGZhdWx0LgoKRGF2ZQoKPiAtLSAKPiBFcmljIEJsYWtlLCBQcmlu Y2lwYWwgU29mdHdhcmUgRW5naW5lZXIKPiBSZWQgSGF0LCBJbmMuICAgICAgICAgICArMS05MTkt MzAxLTMyNjYKPiBWaXJ0dWFsaXphdGlvbjogIHFlbXUub3JnIHwgbGlidmlydC5vcmcKPiAKCgoK LS0KRHIuIERhdmlkIEFsYW4gR2lsYmVydCAvIGRnaWxiZXJ0QHJlZGhhdC5jb20gLyBNYW5jaGVz dGVyLCBVSwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K WGVuLWRldmVsIG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xp c3RzLnhlbi5vcmcveGVuLWRldmVsCg==