u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: drambo <drambo@broadcom.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v15 07/10] arm64: core support
Date: Wed, 22 Jan 2014 17:06:26 -0800 (PST)	[thread overview]
Message-ID: <52E06B04.3020405@broadcom.com> (raw)
In-Reply-To: <1390436886.24905.539.camel@snotra.buserror.net>



On 14-01-22 04:29 PM, Scott Wood-2 [via U-Boot] wrote:
>
>
> On Tue, 2014-01-14 at 09:52 +0800, FengHua wrote:
>> hi bhupesh,
>>
>>> Hi David,
>>>
>>> In reference to my mail above, I see that the transition to EL2 (from EL3) which occurs very early
>>> in start.S needs to be changed on lines of the ARMv7 code, i.e. the EL2 transition should happen just
>>> before Linux is booted up by the u-boot.
>>>
>>> The reason for the same is that a no of ARM IPs like GIC, SMMU and TZPC/TZASC need to be configured to
>>> allow non-secure accesses from Linux world (which runs in EL1 mode). Adding the assembly code for all
>>> such IPs in 'setup_el3' function in start.S, will bloat the start.S and also increase the chances of a
>>> bug in the assembly code.
>>>
>>> Hence, I would like to propose a strategy to shift from EL3 to EL2 to some point in u-boot code after the
>>> C Run Time has been initialized (similar to present ARMv7 u-boot code).
>>>
>>> If you are ok with the same, I can try to send out some RFC patches rebased against your latest v16 code-base.
>>>
>>> Please let me know.
>>> Regards,
>>> Bhupesh
>>>
>> Actually, patch v16 did exception level switch in the way as you said. please review the code.
>> Both master and slaves switch to el2(el1) just before jumping to linux kernel. BTW,if any good conception please feel free to patch it.
>
> How would you handle running U-Boot under a secure firmware, or under a
> hypervisor?  Why not take the Linux approach of running most code in
> EL1, with exception handlers pointing at code to handle special
> situations (such as returning to EL2 before OS entry)?
>
> As for bloating start.S, could leaving EL3 be done in early C code
> rather than in early asm or late C code?  Or, bundle U-Boot with a tiny
> "insecure firmware" that provides the minimum functionality needed with
> similar APIs that would be used with real secure firmware.

Hi Scott,
Why is any EL3 code in u-boot at all? That's not the ARM ATF approach I 
believe but I'm not an expert in this. Please see 
http://lists.denx.de/pipermail/u-boot/2014-January/171581.html and 
(https://github.com/ARM-software/arm-trusted-firmware/blob/master/docs/user-guide.md 

See section "Normal World Software Execution")

Thanks.
Darwin

>
> -Scott
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
>
>
>
> _______________________________________________
> If you reply to this email, your message will be added to the discussion below:
> http://u-boot.10912.n7.nabble.com/PATCH-v15-00-10-arm64-patch-tp167751p172101.html
>
> To unsubscribe from [PATCH v15 00/10] arm64 patch, visit http://u-boot.10912.n7.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=167751&code=ZHJhbWJvQGJyb2FkY29tLmNvbXwxNjc3NTF8LTQ0Nzc3MTIxNQ==
>




--
View this message in context: http://u-boot.10912.n7.nabble.com/PATCH-v15-00-10-arm64-patch-tp167751p172102.html
Sent from the U-Boot mailing list archive at Nabble.com.

  reply	other threads:[~2014-01-23  1:06 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-15  3:45 [U-Boot] [PATCH v15 00/10] arm64 patch fenghua at phytium.com.cn
2013-11-15  3:45 ` [U-Boot] [PATCH v15 01/10] fdt_support: 64bit initrd start address support fenghua at phytium.com.cn
2013-11-15  3:45   ` [U-Boot] [PATCH v15 02/10] cmd_pxe: remove compiling warnings fenghua at phytium.com.cn
2013-11-15  3:45     ` [U-Boot] [PATCH v15 03/10] add weak entry definition fenghua at phytium.com.cn
2013-11-15  3:45       ` [U-Boot] [PATCH v15 04/10] arm64: Add tool to statically apply RELA relocations fenghua at phytium.com.cn
2013-11-15  3:45         ` [U-Boot] [PATCH v15 05/10] arm64: Turn u-boot.bin back into an ELF file after relocate-rela fenghua at phytium.com.cn
2013-11-15  3:45           ` [U-Boot] [PATCH v15 06/10] arm64: Make checkarmreloc accept arm64 relocations fenghua at phytium.com.cn
2013-11-15  3:45             ` [U-Boot] [PATCH v15 07/10] arm64: core support fenghua at phytium.com.cn
2013-11-15  3:45               ` [U-Boot] [PATCH v15 08/10] arm64: generic board support fenghua at phytium.com.cn
2013-11-15  3:45                 ` [U-Boot] [PATCH v15 09/10] arm64: board support of vexpress_aemv8a fenghua at phytium.com.cn
2013-11-15  3:45                   ` [U-Boot] [PATCH v15 10/10] arm64: MAKEALL, filter armv8 boards from LIST_arm fenghua at phytium.com.cn
2013-11-27 20:38               ` [U-Boot] [PATCH v15 07/10] arm64: core support Bhupesh SHARMA
2013-11-29 13:35                 ` FengHua
2013-11-30 18:44                   ` Bhupesh Sharma
     [not found]                     ` <bcf7ed.a55.142ae85d276.Coremail.fenghua@phytium.com.cn>
2013-12-03 10:02                       ` Bhupesh Sharma
2014-01-13 11:24                         ` bhupesh.sharma at freescale.com
2014-01-14  1:52                           ` FengHua
2014-01-23  0:28                             ` Scott Wood
2014-01-23  1:06                               ` drambo [this message]
2014-01-24  1:20                               ` FengHua
2014-01-09  9:49               ` [U-Boot] how to get u-boot code with " TigerLiu at viatech.com.cn
2014-01-11  6:44                 ` FengHua
2014-01-11  6:50                   ` Jagan Teki
2014-01-13  0:54                     ` TigerLiu at viatech.com.cn
2014-01-14  9:12                   ` TigerLiu at viatech.com.cn
2014-01-15  6:37                     ` Wolfgang Denk
2014-01-15 11:27                       ` Abraham Varricatt
2014-01-15 12:25                         ` Wolfgang Denk
2014-01-20 10:54                 ` TigerLiu at viatech.com.cn
2014-01-20 11:57                   ` bhupesh.sharma at freescale.com
2014-01-21  0:49                     ` TigerLiu at viatech.com.cn
2014-01-22 19:02                     ` drambo
2014-01-23  7:15                       ` bhupesh.sharma at freescale.com
2014-01-23  7:54                         ` TigerLiu at viatech.com.cn
2014-02-11 13:33                           ` bhupesh.sharma at freescale.com
2014-02-12  2:08                             ` TigerLiu at viatech.com.cn
2014-02-12  7:15                               ` bhupesh.sharma at freescale.com
2014-02-12  7:26                                 ` TigerLiu at viatech.com.cn
2014-02-12  7:52                               ` Inderpal Singh
2014-02-12  8:02                                 ` TigerLiu at viatech.com.cn
2014-02-12  8:06                                   ` Inderpal Singh
2014-02-12  8:14                                     ` TigerLiu at viatech.com.cn
2014-02-12  8:25                                       ` bhupesh.sharma at freescale.com
2014-02-12  9:37                                         ` Inderpal Singh
2014-01-23 15:58                         ` Detlev Zundel
2014-01-23 17:04                           ` Darwin Rambo
2014-01-25 19:46                             ` bhupesh.sharma at freescale.com
2014-01-26  1:42                               ` drambo
2013-12-11 21:14 ` [U-Boot] [PATCH v15 00/10] arm64 patch 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=52E06B04.3020405@broadcom.com \
    --to=drambo@broadcom.com \
    --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;
as well as URLs for NNTP newsgroup(s).