From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoffer Dall Subject: Re: [PATCH v4 00/28] arm/arm64: KVM: Rework the hyp-stub API Date: Fri, 24 Mar 2017 15:36:08 +0100 Message-ID: <20170324143608.GF25903@cbox> References: <20170321192058.9300-1-marc.zyngier@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-arm-kernel@lists.infradead.org, kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, Russell King , Christoffer Dall , Mark Rutland , Catalin Marinas , James Morse , Ard Biesheuvel , Keerthy To: Marc Zyngier Return-path: Received: from mail-wm0-f41.google.com ([74.125.82.41]:35807 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965804AbdCXOgM (ORCPT ); Fri, 24 Mar 2017 10:36:12 -0400 Received: by mail-wm0-f41.google.com with SMTP id u132so14680063wmg.0 for ; Fri, 24 Mar 2017 07:36:10 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20170321192058.9300-1-marc.zyngier@arm.com> Sender: kvm-owner@vger.kernel.org List-ID: Hi Marc, On Tue, Mar 21, 2017 at 07:20:30PM +0000, Marc Zyngier wrote: > As noticed by RMK in this thread[1], the hyp-stub API on 32bit ARM > could do with some TLC (it cannot perform a soft-restart at HYP, and > has holes in the hyp-stub support in a number of places). In general, > it would be desirable for the 32bit behaviour to align on 64bit, if > only to ease maintenance. > > This series implements the following: > - Add HVC_[GS]ET_VECTORS and HVC_SOFT_RESTART to the 32bit code > - Add HVC_RESET_VECTORS to both arm and arm64, removing the need for > __hyp_reset_vectors > - Implement add the stub entry points in the KVM init code, which > didn't implement any so far > - Convert the HYP code to use the init code stubs directly > - Some general cleanup as a result of these changes (which includes > killing HVC_GET_VECTORS) > - Add some API documentation that covers the above > > Patches 12 to 14 would be better squashed into 10 and 11, but I've > kept them separate so that I can take the blame for everything I've > broken. > > I've tested this on arm (Cubietruck, Jetson TK1) and arm64 (Seattle), > both as host and guest. Keerthy has been kind enough to test the 32bit > code on DRA7-EVM, AM57XX-EVM and KEYSTONE-K2E-EVM. For the series, except where I had some minor questions: Acked-by: Christoffer Dall Thanks, -Christoffer From mboxrd@z Thu Jan 1 00:00:00 1970 From: cdall@linaro.org (Christoffer Dall) Date: Fri, 24 Mar 2017 15:36:08 +0100 Subject: [PATCH v4 00/28] arm/arm64: KVM: Rework the hyp-stub API In-Reply-To: <20170321192058.9300-1-marc.zyngier@arm.com> References: <20170321192058.9300-1-marc.zyngier@arm.com> Message-ID: <20170324143608.GF25903@cbox> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Marc, On Tue, Mar 21, 2017 at 07:20:30PM +0000, Marc Zyngier wrote: > As noticed by RMK in this thread[1], the hyp-stub API on 32bit ARM > could do with some TLC (it cannot perform a soft-restart at HYP, and > has holes in the hyp-stub support in a number of places). In general, > it would be desirable for the 32bit behaviour to align on 64bit, if > only to ease maintenance. > > This series implements the following: > - Add HVC_[GS]ET_VECTORS and HVC_SOFT_RESTART to the 32bit code > - Add HVC_RESET_VECTORS to both arm and arm64, removing the need for > __hyp_reset_vectors > - Implement add the stub entry points in the KVM init code, which > didn't implement any so far > - Convert the HYP code to use the init code stubs directly > - Some general cleanup as a result of these changes (which includes > killing HVC_GET_VECTORS) > - Add some API documentation that covers the above > > Patches 12 to 14 would be better squashed into 10 and 11, but I've > kept them separate so that I can take the blame for everything I've > broken. > > I've tested this on arm (Cubietruck, Jetson TK1) and arm64 (Seattle), > both as host and guest. Keerthy has been kind enough to test the 32bit > code on DRA7-EVM, AM57XX-EVM and KEYSTONE-K2E-EVM. For the series, except where I had some minor questions: Acked-by: Christoffer Dall Thanks, -Christoffer