From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
To: Greg Bellows <greg.bellows@linaro.org>
Cc: peter.maydell@linaro.org, alex.bennee@linaro.org,
qemu-devel@nongnu.org, christoffer.dall@linaro.org,
a.spyridakis@virtualopensystems.com
Subject: Re: [Qemu-devel] [PATCH v7 0/4] target-arm: ARM64: Adding EL1 AARCH32 guest support
Date: Thu, 12 Feb 2015 19:10:51 +1000 [thread overview]
Message-ID: <20150212091051.GA29368@toto> (raw)
In-Reply-To: <1423723750-2544-1-git-send-email-greg.bellows@linaro.org>
On Thu, Feb 12, 2015 at 02:49:06PM +0800, Greg Bellows wrote:
> Added support for running an AArch32 guest on a AArch64 KVM host. Support has
> only been added to the QEMU machvirt machine. The addition of CPU properties
> specifiable from the command line were added to allow disablement of AArch64
> execution state hereby forcing EL1 to be AArch32. The new CPU command line
> property is "aarch64=on/off" that is specified as follows:
>
> aarch64-softmmu/qemu-system-aarch64 -M virt -cpu cortex-a57,aarch64=off ...
Thanks for working on this Greg!
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
on the whole series.
Cheers,
Edgar
>
> ---
>
> v6 -> v7
> - Fix comment issues
>
> v5 -> v6
> - Reorder conditionals in 64_to_32 for readability.
>
> v3 -> v4
> - Move and fix sync functions to properly handle current EL/PL
> - Replace use of strtok
> - Add disablement of aarch64 option if KVM disabled
> - Revert and update AArch64 check on vcpu init to not check for AARch64 feature
> but rather AArch64 family type.
> - Relocate register sync on get
> - add missing env->aarch64 refresh after pstate fetch
>
> v2 -> v3
> - Fix KVM64/AArch64 hang by conditionalizing register sync
> - Conditionalize 64-bit interrupt handler setting of aarch64
>
> v1 -> v2
> - Replaced custom property parsing with use of generic CPU property parser
> - Added CPU property registration
> - Fixed mulitple property handling in virt.c
> - Removed unnecessary kernel load changes
>
> Greg Bellows (4):
> target-arm: Add CPU property to disable AArch64
> target-arm: Add feature parsing to virt
> target-arm: Add 32/64-bit register sync
> target-arm: Add AArch32 guest support to KVM64
>
> hw/arm/virt.c | 20 ++++-
> target-arm/cpu.c | 5 +-
> target-arm/cpu.h | 2 +
> target-arm/cpu64.c | 39 +++++++++
> target-arm/helper-a64.c | 5 +-
> target-arm/helper.c | 211 ++++++++++++++++++++++++++++++++++++++++++++++++
> target-arm/kvm64.c | 36 ++++++++-
> target-arm/op_helper.c | 6 +-
> 8 files changed, 309 insertions(+), 15 deletions(-)
>
> --
> 1.8.3.2
>
next prev parent reply other threads:[~2015-02-12 9:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-12 6:49 [Qemu-devel] [PATCH v7 0/4] target-arm: ARM64: Adding EL1 AARCH32 guest support Greg Bellows
2015-02-12 6:49 ` [Qemu-devel] [PATCH v7 1/4] target-arm: Add CPU property to disable AArch64 Greg Bellows
2015-02-12 6:49 ` [Qemu-devel] [PATCH v7 2/4] target-arm: Add feature parsing to virt Greg Bellows
2015-02-12 6:49 ` [Qemu-devel] [PATCH v7 3/4] target-arm: Add 32/64-bit register sync Greg Bellows
2015-02-12 6:49 ` [Qemu-devel] [PATCH v7 4/4] target-arm: Add AArch32 guest support to KVM64 Greg Bellows
2015-02-12 9:10 ` Edgar E. Iglesias [this message]
2015-02-12 9:46 ` [Qemu-devel] [PATCH v7 0/4] target-arm: ARM64: Adding EL1 AARCH32 guest support Greg Bellows
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150212091051.GA29368@toto \
--to=edgar.iglesias@gmail.com \
--cc=a.spyridakis@virtualopensystems.com \
--cc=alex.bennee@linaro.org \
--cc=christoffer.dall@linaro.org \
--cc=greg.bellows@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).