All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] efi_loader: ARM: add support for	ARMV7_NONSEC=y
Date: Thu, 14 Jun 2018 00:20:49 +0200 (CEST)	[thread overview]
Message-ID: <2459d0366aa10aba@bloch.sibelius.xs4all.nl> (raw)
In-Reply-To: <17796e39-3ce2-3734-abb0-b4dc7c2e8e6e@gmx.de> (message from Heinrich Schuchardt on Tue, 12 Jun 2018 20:00:28 +0200)

> From: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Date: Tue, 12 Jun 2018 20:00:28 +0200
> 
> On 06/12/2018 07:27 PM, Mark Kettenis wrote:
> > This series makes it possible to run EFI applications in non-secure
> > mode.  It allows me to run OpenBSD on the imx7d-pico-pi board while
> > using the PSCI implementation provided by U-Boot.
> > 
> > Mark Kettenis (3):
> >   ARM: HYP/non-sec: save and restore stack
> >   efi_loader: ARM: run EFI payloads non-secure
> >   Revert "efi_loader: no support for ARMV7_NONSEC=y"
> > 
> >  arch/arm/cpu/armv7/nonsec_virt.S |  6 ++++--
> >  cmd/bootefi.c                    | 32 ++++++++++++++++++++++++++++++++
> >  doc/README.uefi                  |  2 --
> >  lib/efi_loader/Kconfig           |  2 --
> >  4 files changed, 36 insertions(+), 6 deletions(-)
> > 
> 
> This is the output I got with your patches when trying to boot my BananaPi:
> 
> => bootefi hello
> Scanning disk mmc at 01c0f000.blk...
> Found 3 disks
> WARNING: booting without device tree
> ## Starting EFI application at 42000000 ...
> WARNING: using memory device/image path, this may confuse some payloads!
> 
> U-Boot SPL 2018.07-rc1-D001-00104-g5b859da7ca8 (Jun 12 2018 - 19:52:34
> +0200)
> DRAM:

I could reproduce that.  For some reason r3 gets clobbered and we
still end up with a broken stack.  Posted v2 that simply migrates
SP_svc to SP_hyp.  That seems to work on both imx7 and sun7i.

      parent reply	other threads:[~2018-06-13 22:20 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-12 17:27 [U-Boot] [PATCH 0/3] efi_loader: ARM: add support for ARMV7_NONSEC=y Mark Kettenis
2018-06-12 17:27 ` [U-Boot] [PATCH 1/3] ARM: HYP/non-sec: save and restore stack Mark Kettenis
2018-06-12 18:46   ` Alexander Graf
2018-06-12 20:17     ` Mark Kettenis
2018-06-12 20:32       ` Alexander Graf
2018-06-13 14:55         ` Mark Kettenis
2018-06-12 17:27 ` [U-Boot] [PATCH 2/3] efi_loader: ARM: run EFI payloads non-secure Mark Kettenis
2018-06-12 18:49   ` Alexander Graf
2018-06-12 17:27 ` [U-Boot] [PATCH 3/3] Revert "efi_loader: no support for ARMV7_NONSEC=y" Mark Kettenis
2018-06-12 18:00 ` [U-Boot] [PATCH 0/3] efi_loader: ARM: add support for ARMV7_NONSEC=y Heinrich Schuchardt
2018-06-12 20:36   ` Mark Kettenis
2018-06-13 22:20   ` Mark Kettenis [this message]

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=2459d0366aa10aba@bloch.sibelius.xs4all.nl \
    --to=mark.kettenis@xs4all.nl \
    --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.