From: Andre Przywara <andre.przywara@linaro.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 3/8] ARM: add assembly routine to switch to non-secure state
Date: Thu, 19 Sep 2013 23:09:15 +0200 [thread overview]
Message-ID: <523B67FB.3090705@linaro.org> (raw)
In-Reply-To: <CAPUj1OPF0fT=_V7o8rc+Rqe-mgOvLZ-Nq6NxJt2PwbaRojh0kA@mail.gmail.com>
On 09/19/2013 09:57 PM, Mj Embd wrote:
> two quick points
> (a) xen already has a mode_switch code, so AFAIK xen might not use it
> (as suggested by comment in another patch in this patch set)
Just a few days ago Ian sent out patches to remove this code from Xen.
That code was never meant to survive, just to have something working to
let people play with Xen easily. Xen now requires to be started in HYP
mode, just like KVM.
http://lists.xen.org/archives/html/xen-devel/2013-09/msg01690.html
Patch 5/7 has more details.
Regards,
Andre.
> (b) There are 2 methods of switching from Secure to Hyp mode
> one you have proposed another implemented in xen. I was suggesting
> take the best approach
>
>
>
> On 9/20/13, Christoffer Dall <christoffer.dall@linaro.org> wrote:
>> On Thu, Sep 19, 2013 at 10:00:03PM +0530, Mj Embd wrote:
>>> Hi Andre,
>>>
>>> There is another approach taken in xen. (xen/arch/arm/mode_switch.S)
>>> Which do you think is the better approach
>>>
>> Hi there,
>>
>> I'm not sure I completely understand your question. Do you think this
>> patch series should be changed to take something from Xen? If so, can
>> you please clarify why?
>>
>> For the record, this patch series has been reviewed and tested quite a
>> lot for U-Boot and we would very much like to get this merged and avoid
>> further churn, unless there's a compelling technical reason to change
>> it.
>>
>> Thanks,
>> -Christoffer
>>
>
>
next prev parent reply other threads:[~2013-09-19 21:09 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-19 16:06 [U-Boot] [PATCH v5 0/8] ARMv7: Add HYP mode switching support Andre Przywara
2013-09-19 16:06 ` [U-Boot] [PATCH v5 1/8] ARM: prepare armv7.h to be included from assembly source Andre Przywara
2013-09-19 16:06 ` [U-Boot] [PATCH v5 2/8] ARM: add secure monitor handler to switch to non-secure state Andre Przywara
2013-09-19 21:50 ` Mj Embd
2013-09-20 0:42 ` Christoffer Dall
2013-09-20 2:38 ` Mj Embd
2013-09-20 3:47 ` Christoffer Dall
2013-10-03 6:30 ` Albert ARIBAUD
2013-10-12 9:27 ` [U-Boot] [PATCH v5 2/8] ARM: add secure monitor handler to switchto " TigerLiu at viatech.com.cn
2013-10-12 19:50 ` Albert ARIBAUD
2013-10-14 2:14 ` TigerLiu at viatech.com.cn
2013-10-14 5:16 ` Albert ARIBAUD
2013-10-14 5:23 ` TigerLiu at viatech.com.cn
2013-09-19 16:06 ` [U-Boot] [PATCH v5 3/8] ARM: add assembly routine to switch to " Andre Przywara
2013-09-19 16:30 ` Mj Embd
2013-09-19 19:26 ` Christoffer Dall
2013-09-19 19:57 ` Mj Embd
2013-09-19 20:11 ` Christoffer Dall
2013-09-19 20:38 ` Mj Embd
2013-09-19 21:31 ` Andre Przywara
2013-09-19 21:39 ` Mj Embd
2013-09-19 21:55 ` Mj Embd
2013-09-19 22:35 ` Peter Maydell
2013-09-19 22:50 ` Mj Embd
2013-09-19 23:21 ` Peter Maydell
2013-09-19 21:28 ` Ian Campbell
2013-09-19 21:09 ` Andre Przywara [this message]
2013-09-19 21:27 ` Ian Campbell
2013-09-19 16:06 ` [U-Boot] [PATCH v5 4/8] ARM: add C function " Andre Przywara
2013-09-19 16:06 ` [U-Boot] [PATCH v5 5/8] ARM: trigger non-secure state switch during bootm execution Andre Przywara
2013-09-19 16:06 ` [U-Boot] [PATCH v5 6/8] ARM: add SMP support for non-secure switch Andre Przywara
2013-09-19 16:06 ` [U-Boot] [PATCH v5 7/8] ARM: extend non-secure switch to also go into HYP mode Andre Przywara
2013-10-03 6:24 ` Albert ARIBAUD
2013-10-03 18:55 ` Christoffer Dall
2013-10-03 19:14 ` Albert ARIBAUD
2013-09-19 16:06 ` [U-Boot] [PATCH v5 8/8] ARM: VExpress: enable ARMv7 virt support for VExpress A15 Andre Przywara
2013-10-03 19:31 ` [U-Boot] [PATCH v5 0/8] ARMv7: Add HYP mode switching support Albert ARIBAUD
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=523B67FB.3090705@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox