From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH v3 0/9] HyperV equivalent of pvpanic driver Date: Wed, 1 Jul 2015 16:09:13 +0200 Message-ID: <5593F489.3040208@redhat.com> References: <1435664007-4965-1-git-send-email-den@openvz.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org, Andrey Smetanin , Gleb Natapov To: "Denis V. Lunev" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:34285 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753024AbbGAOJT (ORCPT ); Wed, 1 Jul 2015 10:09:19 -0400 In-Reply-To: <1435664007-4965-1-git-send-email-den@openvz.org> Sender: kvm-owner@vger.kernel.org List-ID: On 30/06/2015 13:33, Denis V. Lunev wrote: > Windows 2012 guests can notify hypervisor about occurred guest crash > (Windows bugcheck(BSOD)) by writing specific Hyper-V msrs. This patch does > handling of this MSR's by KVM and sending notification to user space that > allows to gather Windows guest crash dump by QEMU/LIBVIRT. > > The idea is to provide functionality equal to pvpanic device without > QEMU guest agent for Windows. > > The idea is borrowed from Linux HyperV bus driver and validated against > Windows 2k12. > > Changes from v2: > * forbid modification crash ctl msr by guest > * qemu_system_guest_panicked usage in pvpanic and s390x > * hyper-v crash handler move from generic kvm to i386 > * hyper-v crash handler: skip fetching crash msrs just mark crash occured > * sync with linux-next 20150629 > * patch 11 squashed to patch 10 > * patch 9 squashed to patch 7 > > Changes from v1: > * hyperv code move to hyperv.c > * added read handlers of crash data msrs > * added per vm and per cpu hyperv context structures > * added saving crash msrs inside qemu cpu state > * added qemu fetch and update of crash msrs > * added qemu crash msrs store in cpu state and it's migration > > Signed-off-by: Andrey Smetanin > Signed-off-by: Denis V. Lunev > CC: Gleb Natapov > CC: Paolo Bonzini The patches look good, thanks. I'll queue them as soon as I start merging 4.3 features. Paolo