Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753345AbbETJRo (ORCPT ); Wed, 20 May 2015 05:17:44 -0400 Received: from foss.arm.com ([217.140.101.70]:37351 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752474AbbETJRl (ORCPT ); Wed, 20 May 2015 05:17:41 -0400 Date: Wed, 20 May 2015 10:17:35 +0100 From: Will Deacon To: Alex =?iso-8859-1?Q?Benn=E9e?= Cc: "kvm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "kvmarm@lists.cs.columbia.edu" , "christoffer.dall@linaro.org" , Marc Zyngier , "peter.maydell@linaro.org" , "agraf@suse.de" , "drjones@redhat.com" , "pbonzini@redhat.com" , "zhichao.huang@linaro.org" , "jan.kiszka@siemens.com" , "dahi@linux.vnet.ibm.com" , "r65777@freescale.com" , "bp@suse.de" , Gleb Natapov , Jonathan Corbet , Russell King , Catalin Marinas , "open list:DOCUMENTATION" , open list Subject: Re: [PATCH v4 06/12] KVM: arm64: guest debug, add SW break point support Message-ID: <20150520091735.GB8986@arm.com> References: <1431700035-23479-1-git-send-email-alex.bennee@linaro.org> <1431700035-23479-7-git-send-email-alex.bennee@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1431700035-23479-7-git-send-email-alex.bennee@linaro.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2007 Lines: 53 Hi Alex, On Fri, May 15, 2015 at 03:27:09PM +0100, Alex Benn?e wrote: > This adds support for SW breakpoints inserted by userspace. > > We do this by trapping all guest software debug exceptions to the > hypervisor (MDCR_EL2.TDE). The exit handler sets an exit reason of > KVM_EXIT_DEBUG with the kvm_debug_exit_arch structure holding the > exception syndrome information. > > It will be up to userspace to extract the PC (via GET_ONE_REG) and > determine if the debug event was for a breakpoint it inserted. If not > userspace will need to re-inject the correct exception restart the > hypervisor to deliver the debug exception to the guest. > > Any other guest software debug exception (e.g. single step or HW > assisted breakpoints) will cause an error and the VM to be killed. This > is addressed by later patches which add support for the other debug > types. > > Signed-off-by: Alex Benn?e > Reviewed-by: Christoffer Dall > > --- > v2 > - update to use new exit struct > - tweak for C setup > - do our setup in debug_setup/clear code > - fixed up comments > v3: > - fix spacing in KVM_GUESTDBG_VALID_MASK > - fix and clarify wording on kvm_handle_guest_debug > - handle error case in kvm_handle_guest_debug > - re-word the commit message > v4 > - rm else leg > - add r-b-tag > > diff --git a/Documentation/virtual/kvm/api.txt b/Documentation/virtual/kvm/api.txt > index ba635c7..33c8143 100644 > --- a/Documentation/virtual/kvm/api.txt > +++ b/Documentation/virtual/kvm/api.txt Not sure why, but your patches seem to drop the diffstat which makes it slightly more onerous for reviewers trying to figure out which bits touch their trees. Are you removing it manually? Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/