[U-Boot] [PATCH v5 3/8] ARM: add assembly routine to switch to non-secure state

Andre Przywara
2013-09-19T21:09:15Z

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