All of lore.kernel.org
 help / color / mirror / Atom feed
From: Artturi Alm <artturi.alm@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 11/18] efi_loader: make efi_disk_create_partitions a global symbol
Date: Fri, 9 Feb 2018 11:44:36 +0200	[thread overview]
Message-ID: <20180209094436.GA68710@gmail.com> (raw)
In-Reply-To: <25300726-a753-458d-2931-7e82a7a71afa@gmx.de>

On Fri, Feb 09, 2018 at 05:07:52AM +0100, Heinrich Schuchardt wrote:
> On 02/09/2018 01:15 AM, Jonathan Gray wrote:
> > On Fri, Jan 19, 2018 at 08:24:47PM +0100, Heinrich Schuchardt wrote:
> > > Up to now we have been using efi_disk_create_partitions() to create
> > > partitions for block devices that existed before starting an EFI
> > > application.
> > > 
> > > We need to call it for block devices created by EFI
> > > applications at run time. The EFI application will define the
> > > handle for the block device and install a device path protocol
> > > on it. We have to use this device path as stem for the partition
> > > device paths.
> > > 
> > > Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> > > ---
> > > v3
> > > 	fix typos in comments
> > > v2
> > > 	no change
> > 
> > breakage on mx6cuboxi with OpenBSD bootarm.efi, bisects to
> > 
> > commit 64e4db0f119151a1345e1da19d152eda550394e7
> > Author:     Heinrich Schuchardt <xypron.glpk@gmx.de>
> > AuthorDate: Fri Jan 19 20:24:47 2018 +0100
> > Commit:     Alexander Graf <agraf@suse.de>
> > CommitDate: Mon Jan 22 23:09:14 2018 +0100
> > 
> >      efi_loader: make efi_disk_create_partitions a global symbol
> >      Up to now we have been using efi_disk_create_partitions() to create
> >      partitions for block devices that existed before starting an EFI
> >      application.
> >      We need to call it for block devices created by EFI
> >      applications at run time. The EFI application will define the
> >      handle for the block device and install a device path protocol
> >      on it. We have to use this device path as stem for the partition
> >      device paths.
> >      Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> >      Signed-off-by: Alexander Graf <agraf@suse.de>
> > 
> > with master as of e24bd1e79e223aa89854c0be95a53e2d538144a5
> > 
> > U-Boot SPL 2018.03-rc1-00185-g1e19c70639 (Feb 09 2018 - 11:43:18 +1300)
> > Trying to boot from MMC1
> > 
> > 
> > U-Boot 2018.03-rc1-00185-g1e19c70639 (Feb 09 2018 - 11:43:18 +1300)
> > 
> > CPU:   Freescale i.MX6Q rev1.5 996 MHz (running at 792 MHz)
> > CPU:   Extended Commercial temperature grade (-20C to 105C) at 24C
> > Reset cause: POR
> > Board: MX6 Cubox-i
> > DRAM:  2 GiB
> > MMC:   FSL_SDHC: 0
> > Loading Environment from MMC... OK
> > No panel detected: default to HDMI
> > Display: HDMI (1024x768)
> > In:    serial
> > Out:   serial
> > Err:   serial
> > Net:   FEC
> > Hit any key to stop autoboot:  0
> > switch to partitions #0, OK
> > mmc0 is current device
> > Scanning mmc 0:1...
> > 37503 bytes read in 17 ms (2.1 MiB/s)
> > Found EFI removable media binary efi/boot/bootarm.efi
> > Scanning disks on usb...
> > 76528 bytes read in 31 ms (2.4 MiB/s)
> > ## Starting EFI application at 12000000 ...
> > BS->LocateHandle() returns -2147483634
> 
> -2147483634 == EFI_NOT_FOUND
> 
> Without debug output it is impossible to understand what is going wrong.
> Please, insert
> 
> #define DEBUG 1
> 
> at the top of lib/efi_loader/efi_boottime.c
> 
> I assume you are again trying to boot OpenBSD.
> 
> Does this image reproduce the error:
> https://ftp.eu.openbsd.org/pub/OpenBSD/6.2/armv7/miniroot-cubox-62.fs
> 
> Otherwise provide a disk image that can be used for testing.
> 
> I only have a Wandboard Quad. But that has the same i.MX6Q processor. So
> once I know which image to use I could run a test.
> 
> Best regards
> 
> Heinrich
> 

Hi,

you could try this:
http://ftp.eu.openbsd.org/pub/OpenBSD/snapshots/armv7/miniroot-wandboard-62.fs

-Artturi

> > undefined instruction
> > pc : [<8e560348>]	   lr : [<8e56444c>]
> > reloc pc : [<15de4348>]	   lr : [<15de844c>]
> > sp : 8f57af10  ip : 8ffc2474	 fp : 8f57af1c
> > r10: 0000b000  r9 : 8f57bee0	 r8 : 0000000b
> > r7 : 8ffa1a9d  r6 : 8ffa16ad	 r5 : 8e56f0d0	r4 : 8e56e88a
> > r3 : 8e56dac8  r2 : 00000001	 r1 : 00000000	r0 : 00000000
> > Flags: nZCv  IRQs off  FIQs off	 Mode SVC_32
> > Resetting CPU ...
> > 
> > resetting ...
> > 
> > (undefined instruction is used to reset as efi reset was not
> > present in earlier U-Boot versions).
> > 
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

  reply	other threads:[~2018-02-09  9:44 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-19 19:24 [U-Boot] [PATCH v3 00/18] efi_loader: enable EFI driver provided block device Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 01/18] efi_loader: return NULL from device path functions Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 02/18] efi_loader: address of the simple file system protocol Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 03/18] efi_loader: correct find " Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 04/18] efi_loader: print device path when entering efi_load_image Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 05/18] efi_loader: allocate correct memory type for EFI image Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 06/18] efi_loader: check tables in helloworld.efi Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 07/18] efi_loader: fix StartImage bootservice Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 08/18] efi_loader: efi_disk_register: correctly determine if_type_name Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 09/18] efi_loader: make efi_block_io_guid a global symbol Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 10/18] efi_loader: provide a function to create a partition node Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 11/18] efi_loader: make efi_disk_create_partitions a global symbol Heinrich Schuchardt
2018-02-09  0:15   ` Jonathan Gray
2018-02-09  4:07     ` Heinrich Schuchardt
2018-02-09  9:44       ` Artturi Alm [this message]
2018-02-09 16:42         ` Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 12/18] efi_loader: correct EFI_BLOCK_IO_PROTOCOL definitions Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 13/18] efi_loader: provide function to get last node of a device path Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 14/18] efi_loader: provide link between devices and EFI handles Heinrich Schuchardt
2018-01-19 20:20   ` Alexander Graf
2018-01-19 20:33     ` Heinrich Schuchardt
2018-01-19 20:36       ` Alexander Graf
2018-01-19 19:24 ` [U-Boot] [PATCH v3 15/18] efi_loader: add check_tpl parameter to efi_signal_event Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 16/18] efi_loader: fix ExitBootServices Heinrich Schuchardt
2018-01-19 19:24 ` [U-Boot] [PATCH v3 17/18] efi_driver: EFI block driver Heinrich Schuchardt
2018-01-19 21:03   ` Alexander Graf
2018-01-19 19:24 ` [U-Boot] [PATCH v3 18/18] efi_selftest: provide a test for block io Heinrich Schuchardt

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=20180209094436.GA68710@gmail.com \
    --to=artturi.alm@gmail.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 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.