qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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
> 

  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).