linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: marc.zyngier@arm.com (Marc Zyngier)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/3] KVM: ARM: Add support for Cortex-A7
Date: Mon, 07 Oct 2013 09:07:56 +0100	[thread overview]
Message-ID: <52526BDC.3050305@arm.com> (raw)
In-Reply-To: <CA+eFSM0k3mA4uOiEvmE_tfVjnpq8DpXV-=nJXhucUDePJRTYXw@mail.gmail.com>

On 07/10/13 07:16, Gavin Guo wrote:
> On Sun, Oct 6, 2013 at 10:56 PM, Peter Maydell <peter.maydell@linaro.org
> <mailto:peter.maydell@linaro.org>> wrote:
> 
>     On 6 October 2013 23:03, Gavin Guo <gavin.guo@canonical.com
>     <mailto:gavin.guo@canonical.com>> wrote:
>     > Does it mean the currently qemu can't support A15 vcpu running on
>     host A7?
> 
>     No, that's a kernel restriction. That is why the patch description
>     reads:
>     # This patch adds support for running Cortex-A7 guests on Cortex-A7
>     hosts.
> 
>     (What QEMU is currently missing is A7 support, so you can't ask it to
>     run an A7 guest CPU.)
> 
>     -- PMM
> 
> 
> Thanks, I saw the checking in the kvm_vcpu_set_target(). I'm confused
> that what is the concern to run the same cpu type? Does it matter to run
> A15 on A7 or inverse?

It does. MIDR registers are different. Cache size is different. It may
work, or it may not. And at the very least, you should present a
consistent view of the underlying system to the unsuspecting OS.

	M.
-- 
Jazz is not dead. It just smells funny...

  parent reply	other threads:[~2013-10-07  8:07 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-26 15:49 [PATCH 0/3] KVM: ARM: Cortex-A7 support and fixes Jonathan Austin
2013-09-26 15:49 ` [PATCH 1/3] KVM: ARM: fix the size of TTBCR_{T0SZ,T1SZ} masks Jonathan Austin
2013-09-26 16:07   ` Christoffer Dall
2013-09-26 15:49 ` [PATCH 2/3] KVM: ARM: Fix calculation of virtual CPU ID Jonathan Austin
2013-09-26 16:12   ` Christoffer Dall
2013-09-26 15:49 ` [PATCH 3/3] KVM: ARM: Add support for Cortex-A7 Jonathan Austin
2013-09-26 16:17   ` Christoffer Dall
2013-09-26 16:50     ` Marc Zyngier
2013-09-26 17:09     ` Jonathan Austin
     [not found]       ` <CA+eFSM1YdhBDMAvyXknMA62Ao3CEntMptdi7RD=NXis1F-BmfQ@mail.gmail.com>
2013-10-06 14:56         ` Peter Maydell
     [not found]           ` <CA+eFSM0k3mA4uOiEvmE_tfVjnpq8DpXV-=nJXhucUDePJRTYXw@mail.gmail.com>
2013-10-07  8:07             ` Marc Zyngier [this message]
2013-10-07  9:17               ` Gavin Guo
2013-10-07  9:29                 ` Marc Zyngier
2013-10-09 13:06           ` Diana Craciun
2013-10-08 14:16 ` [PATCH 0/3] KVM: ARM: Cortex-A7 support and fixes Gavin Guo
2013-10-09  9:21   ` Jonathan Austin
2013-10-16  0:25 ` Christoffer Dall

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=52526BDC.3050305@arm.com \
    --to=marc.zyngier@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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).