All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andre Przywara <andre.przywara@linaro.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 0/6] ARMv7: Add HYP mode switching support
Date: Fri, 26 Apr 2013 15:24:10 +0200	[thread overview]
Message-ID: <517A7FFA.2010000@linaro.org> (raw)
In-Reply-To: <CAFEAcA93X-QAMn=v7Ty9sOv7qwgSsXNb1C0GDgNNup0qSob+rw@mail.gmail.com>

On 04/26/2013 03:18 PM, Peter Maydell wrote:
> On 26 April 2013 14:14, Andre Przywara <andre.przywara@linaro.org> wrote:
>> ARM CPUs with the virtualization extension have a new mode called
>> HYP mode, which allows hypervisors to safely control and monitor
>> guests. The current hypervisor (KVM and Xen) implementations
>> require the kernel to be entered in that HYP mode.
>>
>> This patch series introduces a new U-Boot command called "hypmode"
>> which can be used at any time at the U-Boot command prompt to
>> switch the CPU into HYP mode - ideally this would be done before
>> starting the kernel.
>
> The obvious question here is "why do we need a new command?".
> The kernel booting specification says "boot the kernel in
> Hyp mode" so we should just always do that for booting Linux,
> surely?

Because it avoids regressions. I kind of feel uneasy to do a lot of 
tinkering with secure state and the GIC unconditionally, especially if 
enabled on many boards with virt-capable CPUs.

As written in question 2) later in that mail, I can also live with a 
command to _dis_able the HYP mode switching in case it causes problems.

Regards,
Andre.

  reply	other threads:[~2013-04-26 13:24 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-26 13:14 [U-Boot] [RFC PATCH 0/6] ARMv7: Add HYP mode switching support Andre Przywara
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 1/6] ARM: add secure monitor handler to switch to non-secure state Andre Przywara
2013-04-26 22:11   ` Christoffer Dall
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 2/6] ARM: add assembly routine " Andre Przywara
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 3/6] ARM: add U-Boot command "hypmode" " Andre Przywara
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 4/6] ARM: add SMP support for non-secure switch Andre Przywara
2013-04-26 22:13   ` Christoffer Dall
2013-05-06 13:19     ` Andre Przywara
2013-05-10 22:41       ` Christoffer Dall
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 5/6] ARM: extend non-secure switch to also go into HYP mode Andre Przywara
2013-04-26 13:14 ` [U-Boot] [RFC PATCH 6/6] ARM: VExpress: enable ARMv7 virt support for VExpress A15 Andre Przywara
2013-04-26 13:18 ` [U-Boot] [RFC PATCH 0/6] ARMv7: Add HYP mode switching support Peter Maydell
2013-04-26 13:24   ` Andre Przywara [this message]
2013-04-26 13:42     ` Peter Maydell
2013-04-26 14:14       ` Andre Przywara
2013-04-26 18:29         ` 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=517A7FFA.2010000@linaro.org \
    --to=andre.przywara@linaro.org \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.