From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44302) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UsYMA-0002rS-S8 for qemu-devel@nongnu.org; Fri, 28 Jun 2013 09:04:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UsYM5-0007eC-WF for qemu-devel@nongnu.org; Fri, 28 Jun 2013 09:04:54 -0400 Received: from cantor2.suse.de ([195.135.220.15]:37215 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UsYM5-0007e6-Mr for qemu-devel@nongnu.org; Fri, 28 Jun 2013 09:04:49 -0400 Message-ID: <51CD89EE.9080002@suse.de> Date: Fri, 28 Jun 2013 15:04:46 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <51CD7F88.2060704@virtualopensystems.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/6][RFC] AArch64 support for Versatile Express using KVM List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , m.hamayun@virtualopensystems.com Cc: VirtualOpenSystems Technical Team , qemu-devel@nongnu.org, kvmarm@lists.cs.columbia.edu Am 28.06.2013 14:44, schrieb Peter Maydell: > On 28 June 2013 13:20, Mian M. Hamayun wrote: >> This patch series implements KVM support in QEMU for the ARMv8 Cortex >> A57 CPU. >> It depends on the previously submitted AArch64 Preparation Patchset V4= , >> and uses >> as a base, the existing Versatile Express machine model and the alread= y >> available KVM in-kernel GIC support. >=20 > This rather overlaps with a bunch of work John Rigby has already done > (though not yet published to the mailing lists): >=20 > https://git.linaro.org/gitweb?p=3Dpeople/pmaydell/qemu-aarch64.git;a=3D= shortlog;h=3Drefs/heads/kvm-aarch64-wip >=20 > so now we need to reconcile the two trees. If you'd let us know > you were doing this a bit earlier we could probably have > coordinated something... I had been looking into that tree myself, and apart from John's Linux'ish "ARM: " prefixes rather than our usual "target-arm: " (part of which you forgot to fix before sending a PULL), I am really opposed to copying A15MPCore_priv for A57. I plan to post a cleanup for the A9 version the weekend, now that we have a more elaborate hw/ directory structure for headers. Also I would really prefer to have the is_aarch64() helper operate on ARMCPU rather than CPUARMState. QOM cast removal is underway to make that more efficient. Is there a 32-bit aarch64 register in ARMv8 to be accessed by TCG? If not, the field should be in ARMCPU and a bool rather than uint32_t. Regards, Andreas P.S. This cover letter got submitted separate from the actual patches so that they are not marked RFC - please use --subject-prefix=3D"RFC" or the like and suggest to indicate there against which tree it is. --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg