From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH v9 00/16] KVM/ARM Implementation Date: Tue, 3 Jul 2012 16:51:18 -0300 Message-ID: <20120703195118.GB10013@amt.cnet> References: <20120703085841.27746.82730.stgit@ubuntu> <4FF2F3B7.4050701@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoffer Dall , android-virt@lists.cs.columbia.edu, kvm@vger.kernel.org, tech@virtualopensystems.com To: Avi Kivity Return-path: Received: from mx1.redhat.com ([209.132.183.28]:62962 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753188Ab2GCTva (ORCPT ); Tue, 3 Jul 2012 15:51:30 -0400 Content-Disposition: inline In-Reply-To: <4FF2F3B7.4050701@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Jul 03, 2012 at 04:29:27PM +0300, Avi Kivity wrote: > On 07/03/2012 11:59 AM, Christoffer Dall wrote: > > The following series implements KVM support for ARM processors, > > specifically on the Cortex A-15 platform. Work is done in > > collaboration between Columbia University, Virtual Open Systems and > > ARM/Linaro. > > > > The patch series applies to kvm/next, specifically commit: > > ae7a2a3fb6f8b784c2752863f4f1f20c656f76fb > > > > This is Version 9 of the patch series, but the first two versions > > were reviewed outside of the KVM mailing list. Changes can also be > > pulled from: > > git://github.com/virtualopensystems/linux-kvm-arm.git kvm-a15-v9 > > > > A non-flattened edition of the patch series can be found at: > > git://github.com/virtualopensystems/linux-kvm-arm.git kvm-a15-v9-stage > > > > The implementation is broken up into a logical set of patches, the first > > five are preparatory patches: > > 1. ARM: Add mem_type prot_pte accessor > > 2. ARM: ARM_VIRT_EXT config option > > 3. ARM: Section based HYP idmaps > > 4. KVM: Move KVM_IRQ_LINE to arch-generic code > > 5. KVM: Guard code with CONFIG_MMU_NOTIFIER (repost) > > > > KVM guys, please consider pulling the KVM generic patches as early as > > possible. Thanks. > > Those seem fine to me. Marcelo? 5. is already in, 4. has comments and the rest is ARM specific.