kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Christoffer Dall <cdall@cs.columbia.edu>
Cc: kvm@vger.kernel.org, catalin.marinas@arm.com,
	tech@virtualopensystems.com, android-virt@lists.cs.columbia.edu
Subject: Re: [PATCH v4 00/10] KVM/ARM Implementation
Date: Tue, 09 Aug 2011 14:43:30 +0300	[thread overview]
Message-ID: <4E411D62.4030305@redhat.com> (raw)
In-Reply-To: <20110806103821.27198.41231.stgit@localhost6.localdomain6>

On 08/06/2011 01:38 PM, Christoffer Dall wrote:
> The following series implements KVM support for ARM processors,
> specifically on the Cortex A-15 platform.
>
> The patch series applies to the arm-lpae branch of ARM Ltd's kernel
> tree. This is Version 4 of the patch series, but the first two versions
> were reviewed outside of the KVM mailing list. Changes can also be
> pulled from:
>    git://git.ncl.cs.columbia.edu/pub/git/linux-kvm-arm kvm-a15-v4
>
> The implementation is broken up into a logical set of patches, the first
> one containing a skeleton of files, makefile changes, the basic user
> space interface and KVM architecture specific stubs.  Subsequent patches
> implement parts of the system as listed:
>   1.  Skeleton
>   2.  Identity Mapping for Hyp mode
>   3.  Hypervisor intitalization
>   4.  Hyp mode memory mappings and 2nd stage preparation
>   5.  World-switch implementation and Hyp exception vectors
>   6.  Emulation framework and CP15 emulation
>   7.  Handle guest user memory aborts
>   8.  Handle guest MMIO aborts
>   9.  Handle userspace IRQ/FIQ injection
>   10. Support guest wait-for-interrupt instructions.
>
> Testing:
> Limited testing, but have run GCC inside guest, which compiled a small
> hellow-world program, which was successfully run. Hardware still
> unavailable, so all testing has been done on ARM Fast Models.
>
> For a guide on how to set up a testing environment and try out these
> patches, see:
>    http://wiki.ncl.cs.columbia.edu/wiki/KVMARM:Guides:Development_Environment

Pretty nice, and once again I congratulate you for not having to 
virtualize x86.

I don't know how close you feel you are to merging, but from my point of 
view things are looking good.  We'll need to coordinate trees and acks 
since this is touching more than just arch/arm/kvm.


>   arch/arm/kvm/arm.c                          |  701 +++++++++++++++++++++++++++
>   arch/arm/kvm/arm_emulate.c                  |  604 +++++++++++++++++++++++
>   arch/arm/kvm/arm_exports.c                  |   26 +
>   arch/arm/kvm/arm_guest.c                    |  150 ++++++
>   arch/arm/kvm/arm_init.S                     |  115 ++++
>   arch/arm/kvm/arm_interrupts.S               |  488 +++++++++++++++++++
>   arch/arm/kvm/arm_mmu.c                      |  549 +++++++++++++++++++++

Suggest eliminating the arm_ prefixes.


-- 
error compiling committee.c: too many arguments to function


      parent reply	other threads:[~2011-08-09 11:43 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-06 10:38 [PATCH v4 00/10] KVM/ARM Implementation Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 01/10] ARM: KVM: Initial skeleton to compile KVM support Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 02/10] ARM: KVM: Hypervisor identity mapping Christoffer Dall
2011-08-09  9:20   ` Avi Kivity
2011-08-09  9:29     ` Catalin Marinas
2011-08-09  9:29     ` Christoffer Dall
2011-08-09 10:23       ` [Android-virt] " Alexey Smirnov
2011-08-09 11:23         ` Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 03/10] ARM: KVM: Add hypervisor inititalization Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 04/10] ARM: KVM: Memory virtualization setup Christoffer Dall
2011-08-09  9:57   ` Avi Kivity
2011-08-09 11:24     ` [Android-virt] " Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 05/10] ARM: KVM: Inject IRQs and FIQs from userspace Christoffer Dall
2011-08-09 10:07   ` Avi Kivity
2011-08-09 11:27     ` [Android-virt] " Christoffer Dall
2011-08-09 11:37       ` Avi Kivity
2011-08-09 11:40         ` Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 06/10] ARM: KVM: World-switch implementation Christoffer Dall
2011-08-09 11:09   ` Avi Kivity
2011-08-09 11:29     ` Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 07/10] ARM: KVM: Emulation framework and CP15 emulation Christoffer Dall
2011-08-09 11:17   ` Avi Kivity
2011-08-09 11:34     ` Christoffer Dall
2011-08-09 11:39       ` Avi Kivity
2011-08-09 11:40         ` Christoffer Dall
2011-08-06 10:39 ` [PATCH v4 08/10] ARM: KVM: Handle guest faults in KVM Christoffer Dall
2011-08-09 11:24   ` Avi Kivity
2011-08-09 11:35     ` Christoffer Dall
2011-08-06 10:40 ` [PATCH v4 09/10] ARM: KVM: Handle I/O aborts Christoffer Dall
2011-08-09 11:34   ` Avi Kivity
2011-08-09 11:39     ` Christoffer Dall
2011-08-09 11:46       ` Avi Kivity
2011-08-06 10:40 ` [PATCH v4 10/10] ARM: KVM: Guest wait-for-interrupts (WFI) support Christoffer Dall
2011-08-09 11:43 ` Avi Kivity [this message]

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=4E411D62.4030305@redhat.com \
    --to=avi@redhat.com \
    --cc=android-virt@lists.cs.columbia.edu \
    --cc=catalin.marinas@arm.com \
    --cc=cdall@cs.columbia.edu \
    --cc=kvm@vger.kernel.org \
    --cc=tech@virtualopensystems.com \
    /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).