* BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
@ 2021-06-06 14:37 Matwey V. Kornilov
2021-06-06 15:20 ` Heinrich Schuchardt
0 siblings, 1 reply; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-06 14:37 UTC (permalink / raw)
To: U-Boot Mailing List; +Cc: michael, Heinrich Schuchardt
Hi,
I've found that
f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
breaks running EFI application from USB device on BeagleBone Black
(am335x) device.
With this patch I see the following:
Booting /efi\boot\bootarm.efi
Welcome to GRUB!
data abort
pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
Flags: NzCv IRQs off FIQs on Mode SVC_32
Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
Resetting CPU ...
while without the patch, GRUB works as usual.
Could you please help me to figure out what is going wrong here?
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 14:37 BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot") Matwey V. Kornilov
@ 2021-06-06 15:20 ` Heinrich Schuchardt
2021-06-06 15:42 ` Matwey V. Kornilov
0 siblings, 1 reply; 14+ messages in thread
From: Heinrich Schuchardt @ 2021-06-06 15:20 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: michael, U-Boot Mailing List
On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> Hi,
>
> I've found that
>
> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
>
> breaks running EFI application from USB device on BeagleBone Black
> (am335x) device.
>
> With this patch I see the following:
>
> Booting /efi\boot\bootarm.efi
> Welcome to GRUB!
>
> data abort
> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> Flags: NzCv IRQs off FIQs on Mode SVC_32
> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> Resetting CPU ...
Hello Matwey,
thank you for reporting the issue.
$ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
All code
========
0: e3500000 cmp r0, #0
4: 0a000015 beq 0x60
8: e590000c ldr r0, [r0, #12]
c: eb00f96e bl 0x3e5cc
10:* e5d03000 ldrb r3, [r0] <-- trapping
instruction
Code starting with the faulting instruction
===========================================
0: e5d03000 ldrb r3, [r0]
Looking at the disassembly above we see that reading memory location
NULL fails.
We need to find out where the exception occurs. The code position is
neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
position of bootarm.efi, so it is below the relocated U-Boot code).
Please, add the following line at the start of grub.cfg to get more
output from GRUB:
debug=all
When building U-Boot, please, add
#define DEBUG 1
in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
#include <common.h>.
Best regards
Heinrich
>
> while without the patch, GRUB works as usual.
> Could you please help me to figure out what is going wrong here?
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 15:20 ` Heinrich Schuchardt
@ 2021-06-06 15:42 ` Matwey V. Kornilov
2021-06-06 16:21 ` Heinrich Schuchardt
0 siblings, 1 reply; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-06 15:42 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: michael, U-Boot Mailing List
вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>
> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> > Hi,
> >
> > I've found that
> >
> > f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
> >
> > breaks running EFI application from USB device on BeagleBone Black
> > (am335x) device.
> >
> > With this patch I see the following:
> >
> > Booting /efi\boot\bootarm.efi
> > Welcome to GRUB!
> >
> > data abort
> > pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> > reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> > sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> > r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> > r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> > r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> > Flags: NzCv IRQs off FIQs on Mode SVC_32
> > Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> > Resetting CPU ...
>
> Hello Matwey,
>
> thank you for reporting the issue.
>
> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
>
> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> All code
> ========
> 0: e3500000 cmp r0, #0
> 4: 0a000015 beq 0x60
> 8: e590000c ldr r0, [r0, #12]
> c: eb00f96e bl 0x3e5cc
> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> instruction
>
> Code starting with the faulting instruction
> ===========================================
> 0: e5d03000 ldrb r3, [r0]
>
> Looking at the disassembly above we see that reading memory location
> NULL fails.
>
> We need to find out where the exception occurs. The code position is
> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> position of bootarm.efi, so it is below the relocated U-Boot code).
>
> Please, add the following line at the start of grub.cfg to get more
> output from GRUB:
>
> debug=all
This doesn't provide any additional output from GRUB :(
>
> When building U-Boot, please, add
>
> #define DEBUG 1
>
> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> #include <common.h>.
This doesn't provide much output as well:
Scanning disk mmc@48060000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
** Unrecognized filesystem type **
Scanning disk mmc@481d8000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
Found 3 disks
> Best regards
>
> Heinrich
>
> >
> > while without the patch, GRUB works as usual.
> > Could you please help me to figure out what is going wrong here?
> >
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 15:42 ` Matwey V. Kornilov
@ 2021-06-06 16:21 ` Heinrich Schuchardt
2021-06-06 16:47 ` Heinrich Schuchardt
2021-06-07 17:36 ` Matwey V. Kornilov
0 siblings, 2 replies; 14+ messages in thread
From: Heinrich Schuchardt @ 2021-06-06 16:21 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: michael, U-Boot Mailing List
On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>
>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
>>> Hi,
>>>
>>> I've found that
>>>
>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
>>>
>>> breaks running EFI application from USB device on BeagleBone Black
>>> (am335x) device.
>>>
>>> With this patch I see the following:
>>>
>>> Booting /efi\boot\bootarm.efi
>>> Welcome to GRUB!
>>>
>>> data abort
>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
>> > Resetting CPU ...
>>
>> Hello Matwey,
>>
>> thank you for reporting the issue.
>>
>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
>>
>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>> All code
>> ========
>> 0: e3500000 cmp r0, #0
>> 4: 0a000015 beq 0x60
>> 8: e590000c ldr r0, [r0, #12]
>> c: eb00f96e bl 0x3e5cc
>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
>> instruction
>>
>> Code starting with the faulting instruction
>> ===========================================
>> 0: e5d03000 ldrb r3, [r0]
>>
>> Looking at the disassembly above we see that reading memory location
>> NULL fails.
>>
>> We need to find out where the exception occurs. The code position is
>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
>> position of bootarm.efi, so it is below the relocated U-Boot code).
>>
>> Please, add the following line at the start of grub.cfg to get more
>> output from GRUB:
>>
>> debug=all
>
> This doesn't provide any additional output from GRUB :(
>
>>
>> When building U-Boot, please, add
>>
>> #define DEBUG 1
>>
>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
>> #include <common.h>.
>
>
> This doesn't provide much output as well:
>
> Scanning disk mmc@48060000.blk...
> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> ** Unrecognized filesystem type **
> Scanning disk mmc@481d8000.blk...
> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> NULL)
> Found 3 disks
This implies that GRUB is crashing before even accessing the file system
(including grub.cfg).
On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
/EFI/boot/bootarm.efi. It boots without problem.
What version of GRUB are you using?
How were you booting before updating U-Boot?
What version of U-Boot are you using where the error occurs?
Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
used by installers.
If the boot manager is started by distroboot it may not have an
appropriate device path. It tries to load the file given by environment
variable $fdtfile from the boot device.
From the U-Boot console could you, please, try:
1)
load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
bootefi bootmgr
2)
load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
load usb 0:2 $fdt_addr_r dtb
bootefi bootmgr $fdt_addr_r
where you need to replace dtb by the correct device tree file and adjust
the partition numbers.
Best regards
Heinrich
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 16:21 ` Heinrich Schuchardt
@ 2021-06-06 16:47 ` Heinrich Schuchardt
2021-06-07 17:51 ` Matwey V. Kornilov
2021-06-07 17:36 ` Matwey V. Kornilov
1 sibling, 1 reply; 14+ messages in thread
From: Heinrich Schuchardt @ 2021-06-06 16:47 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: michael, U-Boot Mailing List
On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>>
>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
>>>> Hi,
>>>>
>>>> I've found that
>>>>
>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
>>>> /EFI/boot")
>>>>
>>>> breaks running EFI application from USB device on BeagleBone Black
>>>> (am335x) device.
>>>>
>>>> With this patch I see the following:
>>>>
>>>> Booting /efi\boot\bootarm.efi
>>>> Welcome to GRUB!
>>>>
>>>> data abort
>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
>>> > Resetting CPU ...
>>>
>>> Hello Matwey,
>>>
>>> thank you for reporting the issue.
>>>
>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
>>>
>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>> All code
>>> ========
>>> 0: e3500000 cmp r0, #0
>>> 4: 0a000015 beq 0x60
>>> 8: e590000c ldr r0, [r0, #12]
>>> c: eb00f96e bl 0x3e5cc
>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
>>> instruction
>>>
>>> Code starting with the faulting instruction
>>> ===========================================
>>> 0: e5d03000 ldrb r3, [r0]
>>>
>>> Looking at the disassembly above we see that reading memory location
>>> NULL fails.
>>>
>>> We need to find out where the exception occurs. The code position is
>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
>>> position of bootarm.efi, so it is below the relocated U-Boot code).
>>>
>>> Please, add the following line at the start of grub.cfg to get more
>>> output from GRUB:
>>>
>>> debug=all
>>
>> This doesn't provide any additional output from GRUB :(
>>
>>>
>>> When building U-Boot, please, add
>>>
>>> #define DEBUG 1
>>>
>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
>>> #include <common.h>.
>>
>>
>> This doesn't provide much output as well:
>>
>> Scanning disk mmc@48060000.blk...
>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> ** Unrecognized filesystem type **
>> Scanning disk mmc@481d8000.blk...
>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>> NULL)
>> Found 3 disks
>
> This implies that GRUB is crashing before even accessing the file system
> (including grub.cfg).
>
> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> /EFI/boot/bootarm.efi. It boots without problem.
>
> What version of GRUB are you using?
> How were you booting before updating U-Boot?
> What version of U-Boot are you using where the error occurs?
> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> used by installers.
>
> If the boot manager is started by distroboot it may not have an
> appropriate device path. It tries to load the file given by environment
> variable $fdtfile from the boot device.
>
> From the U-Boot console could you, please, try:
>
> 1)
> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> bootefi bootmgr
>
>
> 2)
> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> load usb 0:2 $fdt_addr_r dtb
> bootefi bootmgr $fdt_addr_r
>
> where you need to replace dtb by the correct device tree file and adjust
> the partition numbers.
>
> Best regards
>
> Heinrich
To catch the earlier EFI API calls you can add
#define DEBUG 1
to lib/efi_loader/efi_boottime.c
Best regards
Heinrich
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 16:21 ` Heinrich Schuchardt
2021-06-06 16:47 ` Heinrich Schuchardt
@ 2021-06-07 17:36 ` Matwey V. Kornilov
1 sibling, 0 replies; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-07 17:36 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: michael, U-Boot Mailing List
вс, 6 июн. 2021 г. в 19:21, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>
> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> > вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>
> >> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> >>> Hi,
> >>>
> >>> I've found that
> >>>
> >>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
> >>>
> >>> breaks running EFI application from USB device on BeagleBone Black
> >>> (am335x) device.
> >>>
> >>> With this patch I see the following:
> >>>
> >>> Booting /efi\boot\bootarm.efi
> >>> Welcome to GRUB!
> >>>
> >>> data abort
> >>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> >>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> >>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> >>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> >>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> >>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> >>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> >>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> >> > Resetting CPU ...
> >>
> >> Hello Matwey,
> >>
> >> thank you for reporting the issue.
> >>
> >> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> >> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> >>
> >> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >> All code
> >> ========
> >> 0: e3500000 cmp r0, #0
> >> 4: 0a000015 beq 0x60
> >> 8: e590000c ldr r0, [r0, #12]
> >> c: eb00f96e bl 0x3e5cc
> >> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> >> instruction
> >>
> >> Code starting with the faulting instruction
> >> ===========================================
> >> 0: e5d03000 ldrb r3, [r0]
> >>
> >> Looking at the disassembly above we see that reading memory location
> >> NULL fails.
> >>
> >> We need to find out where the exception occurs. The code position is
> >> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> >> position of bootarm.efi, so it is below the relocated U-Boot code).
> >>
> >> Please, add the following line at the start of grub.cfg to get more
> >> output from GRUB:
> >>
> >> debug=all
> >
> > This doesn't provide any additional output from GRUB :(
> >
> >>
> >> When building U-Boot, please, add
> >>
> >> #define DEBUG 1
> >>
> >> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> >> #include <common.h>.
> >
> >
> > This doesn't provide much output as well:
> >
> > Scanning disk mmc@48060000.blk...
> > EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > ** Unrecognized filesystem type **
> > Scanning disk mmc@481d8000.blk...
> > EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > NULL)
> > Found 3 disks
>
> This implies that GRUB is crashing before even accessing the file system
> (including grub.cfg).
>
> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> /EFI/boot/bootarm.efi. It boots without problem.
>
> What version of GRUB are you using?
Grub 2.04
> How were you booting before updating U-Boot?
The same path: u-boot was running EFI Grub which booted the kernel
> What version of U-Boot are you using where the error occurs?
The error firstly occurs in f3866909e350 and present in master.
> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> used by installers.
This is default boot loader path for openSUSE JeOS images.
>
> If the boot manager is started by distroboot it may not have an
> appropriate device path. It tries to load the file given by environment
> variable $fdtfile from the boot device.
>
> From the U-Boot console could you, please, try:
>
> 1)
> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> bootefi bootmgr
>
=> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
929792 bytes read in 92 ms (9.6 MiB/s)
=> bootefi bootmgr
Scanning disk mmc@48060000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
** Unrecognized filesystem type **
Scanning disk mmc@481d8000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
Scanning disk usb_mass_storage.lun0...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: EFI system partition: usb 0:1
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
** Unrecognized filesystem type **
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
Found 7 disks
** Unable to read file ubootefi.var **
Failed to load EFI variables
BootOrder not defined
EFI boot manager: Cannot load any image
>
> 2)
> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> load usb 0:2 $fdt_addr_r dtb
> bootefi bootmgr $fdt_addr_r
=> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
929792 bytes read in 95 ms (9.3 MiB/s)
=> load usb 0:3 $fdt_addr_r /boot/dtb/am335x-boneblack.dtb
96415 bytes read in 13 ms (7.1 MiB/s)
=> bootefi bootmgr $fdt_addr_r
Scanning disk mmc@48060000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
** Unrecognized filesystem type **
Scanning disk mmc@481d8000.blk...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
Scanning disk usb_mass_storage.lun0...
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: EFI system partition: usb 0:1
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
** Unrecognized filesystem type **
EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
&efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
NULL)
Found 7 disks
** Unable to read file ubootefi.var **
Failed to load EFI variables
BootOrder not defined
EFI boot manager: Cannot load any image
>
> where you need to replace dtb by the correct device tree file and adjust
> the partition numbers.
>
> Best regards
>
> Heinrich
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-06 16:47 ` Heinrich Schuchardt
@ 2021-06-07 17:51 ` Matwey V. Kornilov
2021-06-10 20:05 ` Heinrich Schuchardt
0 siblings, 1 reply; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-07 17:51 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: michael, U-Boot Mailing List
вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>
> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> > On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> >> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>>
> >>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> >>>> Hi,
> >>>>
> >>>> I've found that
> >>>>
> >>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> >>>> /EFI/boot")
> >>>>
> >>>> breaks running EFI application from USB device on BeagleBone Black
> >>>> (am335x) device.
> >>>>
> >>>> With this patch I see the following:
> >>>>
> >>>> Booting /efi\boot\bootarm.efi
> >>>> Welcome to GRUB!
> >>>>
> >>>> data abort
> >>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> >>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> >>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> >>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> >>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> >>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> >>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> >>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> >>> > Resetting CPU ...
> >>>
> >>> Hello Matwey,
> >>>
> >>> thank you for reporting the issue.
> >>>
> >>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> >>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> >>>
> >>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>> All code
> >>> ========
> >>> 0: e3500000 cmp r0, #0
> >>> 4: 0a000015 beq 0x60
> >>> 8: e590000c ldr r0, [r0, #12]
> >>> c: eb00f96e bl 0x3e5cc
> >>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> >>> instruction
> >>>
> >>> Code starting with the faulting instruction
> >>> ===========================================
> >>> 0: e5d03000 ldrb r3, [r0]
> >>>
> >>> Looking at the disassembly above we see that reading memory location
> >>> NULL fails.
> >>>
> >>> We need to find out where the exception occurs. The code position is
> >>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> >>> position of bootarm.efi, so it is below the relocated U-Boot code).
> >>>
> >>> Please, add the following line at the start of grub.cfg to get more
> >>> output from GRUB:
> >>>
> >>> debug=all
> >>
> >> This doesn't provide any additional output from GRUB :(
> >>
> >>>
> >>> When building U-Boot, please, add
> >>>
> >>> #define DEBUG 1
> >>>
> >>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> >>> #include <common.h>.
> >>
> >>
> >> This doesn't provide much output as well:
> >>
> >> Scanning disk mmc@48060000.blk...
> >> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> ** Unrecognized filesystem type **
> >> Scanning disk mmc@481d8000.blk...
> >> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >> NULL)
> >> Found 3 disks
> >
> > This implies that GRUB is crashing before even accessing the file system
> > (including grub.cfg).
> >
> > On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> > /EFI/boot/bootarm.efi. It boots without problem.
> >
> > What version of GRUB are you using?
> > How were you booting before updating U-Boot?
> > What version of U-Boot are you using where the error occurs?
> > Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> > specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> > used by installers.
> >
> > If the boot manager is started by distroboot it may not have an
> > appropriate device path. It tries to load the file given by environment
> > variable $fdtfile from the boot device.
> >
> > From the U-Boot console could you, please, try:
> >
> > 1)
> > load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > bootefi bootmgr
> >
> >
> > 2)
> > load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > load usb 0:2 $fdt_addr_r dtb
> > bootefi bootmgr $fdt_addr_r
> >
> > where you need to replace dtb by the correct device tree file and adjust
> > the partition numbers.
> >
> > Best regards
> >
> > Heinrich
>
> To catch the earlier EFI API calls you can add
>
> #define DEBUG 1
>
> to lib/efi_loader/efi_boottime.c
Welcome to GRUB!
EFI: Entry efi_locate_handle_ext(2,
9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
EFI: Exit: efi_locate_handle_ext: 14
EFI: Entry efi_open_protocol(9df5f298,
5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
0x2)
EFI: Exit: efi_open_protocol: 0
EFI: Entry efi_open_protocol(00000000,
09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
0x2)
EFI: Exit: efi_open_protocol: 2
data abort
pc : [<9ce076d0>] lr : [<9ffa85b3>]
reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
sp : 9df40e28 ip : 00000000 fp : 00000003
r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
Flags: NzCv IRQs off FIQs on Mode SVC_32
Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
Resetting CPU ...
>
> Best regards
>
> Heinrich
>
>
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-07 17:51 ` Matwey V. Kornilov
@ 2021-06-10 20:05 ` Heinrich Schuchardt
2021-06-10 21:02 ` Heinrich Schuchardt
2021-06-11 15:08 ` Matwey V. Kornilov
0 siblings, 2 replies; 14+ messages in thread
From: Heinrich Schuchardt @ 2021-06-10 20:05 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: michael, U-Boot Mailing List
On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>
>> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
>>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
>>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>>>>
>>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
>>>>>> Hi,
>>>>>>
>>>>>> I've found that
>>>>>>
>>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
>>>>>> /EFI/boot")
>>>>>>
>>>>>> breaks running EFI application from USB device on BeagleBone Black
>>>>>> (am335x) device.
>>>>>>
>>>>>> With this patch I see the following:
>>>>>>
>>>>>> Booting /efi\boot\bootarm.efi
>>>>>> Welcome to GRUB!
>>>>>>
>>>>>> data abort
>>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
>>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
>>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
>>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
>>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
>>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
>>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
>>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
>>>>> > Resetting CPU ...
>>>>>
>>>>> Hello Matwey,
>>>>>
>>>>> thank you for reporting the issue.
>>>>>
>>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
>>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
>>>>>
>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>>>> All code
>>>>> ========
>>>>> 0: e3500000 cmp r0, #0
>>>>> 4: 0a000015 beq 0x60
>>>>> 8: e590000c ldr r0, [r0, #12]
>>>>> c: eb00f96e bl 0x3e5cc
>>>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
>>>>> instruction
>>>>>
>>>>> Code starting with the faulting instruction
>>>>> ===========================================
>>>>> 0: e5d03000 ldrb r3, [r0]
>>>>>
>>>>> Looking at the disassembly above we see that reading memory location
>>>>> NULL fails.
>>>>>
>>>>> We need to find out where the exception occurs. The code position is
>>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
>>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
>>>>>
>>>>> Please, add the following line at the start of grub.cfg to get more
>>>>> output from GRUB:
>>>>>
>>>>> debug=all
>>>>
>>>> This doesn't provide any additional output from GRUB :(
>>>>
>>>>>
>>>>> When building U-Boot, please, add
>>>>>
>>>>> #define DEBUG 1
>>>>>
>>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
>>>>> #include <common.h>.
>>>>
>>>>
>>>> This doesn't provide much output as well:
>>>>
>>>> Scanning disk mmc@48060000.blk...
>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> ** Unrecognized filesystem type **
>>>> Scanning disk mmc@481d8000.blk...
>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>> NULL)
>>>> Found 3 disks
>>>
>>> This implies that GRUB is crashing before even accessing the file system
>>> (including grub.cfg).
>>>
>>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
>>> /EFI/boot/bootarm.efi. It boots without problem.
>>>
>>> What version of GRUB are you using?
>>> How were you booting before updating U-Boot?
>>> What version of U-Boot are you using where the error occurs?
>>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
>>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
>>> used by installers.
>>>
>>> If the boot manager is started by distroboot it may not have an
>>> appropriate device path. It tries to load the file given by environment
>>> variable $fdtfile from the boot device.
>>>
>>> From the U-Boot console could you, please, try:
>>>
>>> 1)
>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
>>> bootefi bootmgr
>>>
>>>
>>> 2)
>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
>>> load usb 0:2 $fdt_addr_r dtb
>>> bootefi bootmgr $fdt_addr_r
>>>
>>> where you need to replace dtb by the correct device tree file and adjust
>>> the partition numbers.
>>>
>>> Best regards
>>>
>>> Heinrich
>>
>> To catch the earlier EFI API calls you can add
>>
>> #define DEBUG 1
>>
>> to lib/efi_loader/efi_boottime.c
>
>
> Welcome to GRUB!
>
> EFI: Entry efi_locate_handle_ext(2,
> 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
This call could be from grub-core/video/efi_gop.c, check_protocol().
> EFI: Exit: efi_locate_handle_ext: 14
EFI_NOT_FOUND
> EFI: Entry efi_open_protocol(9df5f298,
> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> 0x2)
EFI_LOADED_IMAGE_PROTOCOL_GUID
This call could be from grub-core/kern/efi/efi.c,
grub_efi_get_loaded_image().
> EFI: Exit: efi_open_protocol: 0
> EFI: Entry efi_open_protocol(00000000,
The parameter @handle must not be NULL.
> 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
EFI_DEVICE_PATH_PROTOCOL_GUID
This could be called from grub-core/kern/efi/efi.c,
grub_efi_get_device_path() which is invoked from
grub_machine_get_bootlocation().
> 0x2)
EFI_INVALID_PARAMETER is returned because the handle is NULL.
I could partially reproduce the problem by setting
info->device_handle = NULL;
at the end of efi_setup_loaded_image():
Welcome to GRUB!
EFI: Entry efi_open_protocol(79fdea40,
5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
EFI: Exit: efi_open_protocol: 0
EFI: Entry efi_open_protocol(00000000,
09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
EFI: Exit: efi_open_protocol: 2
error: disk `,msdos2' not found.
grub rescue> >
This leaves me with two questions:
Why does GRUB not handle
*device = grub_efidisk_get_device_name (NULL);
gracefully? Maybe it is because it tries to print via the graphical
output protocol which does not exist?
Why is image->device_handle NULL?
Next step is to verify that image->device_handle is really NULL.
Please apply the following change to efi/efi_loader/efi_boottime.c
@@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
free(info);
}
error:
+ printf("*** %p\n", info->device_handle);
return EFI_EXIT(ret);
}
Best regards
Heinrich
> EFI: Exit: efi_open_protocol: 2
> data abort
> pc : [<9ce076d0>] lr : [<9ffa85b3>]
> reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> sp : 9df40e28 ip : 00000000 fp : 00000003
> r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
> r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> Flags: NzCv IRQs off FIQs on Mode SVC_32
> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> Resetting CPU ...
>
>
>>
>> Best regards
>>
>> Heinrich
>>
>>
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-10 20:05 ` Heinrich Schuchardt
@ 2021-06-10 21:02 ` Heinrich Schuchardt
2021-06-11 15:18 ` Matwey V. Kornilov
2021-06-11 15:08 ` Matwey V. Kornilov
1 sibling, 1 reply; 14+ messages in thread
From: Heinrich Schuchardt @ 2021-06-10 21:02 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: michael, U-Boot Mailing List
On 6/10/21 10:05 PM, Heinrich Schuchardt wrote:
> On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
>> вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>>
>>> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
>>>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
>>>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>>>>>>
>>>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> I've found that
>>>>>>>
>>>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
>>>>>>> /EFI/boot")
>>>>>>>
>>>>>>> breaks running EFI application from USB device on BeagleBone Black
>>>>>>> (am335x) device.
>>>>>>>
>>>>>>> With this patch I see the following:
>>>>>>>
>>>>>>> Booting /efi\boot\bootarm.efi
>>>>>>> Welcome to GRUB!
>>>>>>>
>>>>>>> data abort
>>>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
>>>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
>>>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
>>>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
>>>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
>>>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
>>>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
>>>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
>>>>>> > Resetting CPU ...
>>>>>>
>>>>>> Hello Matwey,
>>>>>>
>>>>>> thank you for reporting the issue.
>>>>>>
>>>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
>>>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
>>>>>>
>>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>>>>>> All code
>>>>>> ========
>>>>>> 0: e3500000 cmp r0, #0
>>>>>> 4: 0a000015 beq 0x60
>>>>>> 8: e590000c ldr r0, [r0, #12]
>>>>>> c: eb00f96e bl 0x3e5cc
>>>>>> 10:* e5d03000 ldrb r3, [r0] <--
>>>>>> trapping
>>>>>> instruction
>>>>>>
>>>>>> Code starting with the faulting instruction
>>>>>> ===========================================
>>>>>> 0: e5d03000 ldrb r3, [r0]
>>>>>>
>>>>>> Looking at the disassembly above we see that reading memory location
>>>>>> NULL fails.
>>>>>>
>>>>>> We need to find out where the exception occurs. The code position is
>>>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
>>>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
>>>>>>
>>>>>> Please, add the following line at the start of grub.cfg to get more
>>>>>> output from GRUB:
>>>>>>
>>>>>> debug=all
>>>>>
>>>>> This doesn't provide any additional output from GRUB :(
>>>>>
>>>>>>
>>>>>> When building U-Boot, please, add
>>>>>>
>>>>>> #define DEBUG 1
>>>>>>
>>>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
>>>>>> #include <common.h>.
>>>>>
>>>>>
>>>>> This doesn't provide much output as well:
>>>>>
>>>>> Scanning disk mmc@48060000.blk...
>>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> ** Unrecognized filesystem type **
>>>>> Scanning disk mmc@481d8000.blk...
>>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
>>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
>>>>> NULL)
>>>>> Found 3 disks
>>>>
>>>> This implies that GRUB is crashing before even accessing the file
>>>> system
>>>> (including grub.cfg).
>>>>
>>>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
>>>> /EFI/boot/bootarm.efi. It boots without problem.
>>>>
>>>> What version of GRUB are you using?
>>>> How were you booting before updating U-Boot?
>>>> What version of U-Boot are you using where the error occurs?
>>>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
>>>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically
>>>> only
>>>> used by installers.
>>>>
>>>> If the boot manager is started by distroboot it may not have an
>>>> appropriate device path. It tries to load the file given by environment
>>>> variable $fdtfile from the boot device.
>>>>
>>>> From the U-Boot console could you, please, try:
>>>>
>>>> 1)
>>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
>>>> bootefi bootmgr
>>>>
>>>>
>>>> 2)
>>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
>>>> load usb 0:2 $fdt_addr_r dtb
>>>> bootefi bootmgr $fdt_addr_r
>>>>
>>>> where you need to replace dtb by the correct device tree file and
>>>> adjust
>>>> the partition numbers.
>>>>
>>>> Best regards
>>>>
>>>> Heinrich
>>>
>>> To catch the earlier EFI API calls you can add
>>>
>>> #define DEBUG 1
>>>
>>> to lib/efi_loader/efi_boottime.c
>>
>>
>> Welcome to GRUB!
>>
>> EFI: Entry efi_locate_handle_ext(2,
>> 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
>
> EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
>
> This call could be from grub-core/video/efi_gop.c, check_protocol().
>
>> EFI: Exit: efi_locate_handle_ext: 14
>
> EFI_NOT_FOUND
>
>> EFI: Entry efi_open_protocol(9df5f298,
>> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
>> 0x2)
>
> EFI_LOADED_IMAGE_PROTOCOL_GUID
>
> This call could be from grub-core/kern/efi/efi.c,
> grub_efi_get_loaded_image().
>
>> EFI: Exit: efi_open_protocol: 0
>> EFI: Entry efi_open_protocol(00000000,
>
> The parameter @handle must not be NULL.
>
>> 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
>
> EFI_DEVICE_PATH_PROTOCOL_GUID
>
> This could be called from grub-core/kern/efi/efi.c,
> grub_efi_get_device_path() which is invoked from
> grub_machine_get_bootlocation().
>
>> 0x2)
>
> EFI_INVALID_PARAMETER is returned because the handle is NULL.
>
> I could partially reproduce the problem by setting
>
> info->device_handle = NULL;
>
> at the end of efi_setup_loaded_image():
>
> Welcome to GRUB!
>
> EFI: Entry efi_open_protocol(79fdea40,
> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> EFI: Exit: efi_open_protocol: 0
> EFI: Entry efi_open_protocol(00000000,
> 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> EFI: Exit: efi_open_protocol: 2
> error: disk `,msdos2' not found.
> grub rescue> >
>
> This leaves me with two questions:
>
> Why does GRUB not handle
>
> *device = grub_efidisk_get_device_name (NULL);
>
> gracefully? Maybe it is because it tries to print via the graphical
> output protocol which does not exist?
You wrote that you are using Suse JeOS.
In grub.cfg you could replace
with_gfx=1
gfxmode=auto
by
with_gfx=0
set textmode=true
This should give you output on the serial console and avoid the crash.
Best regards
Heinrich
>
> Why is image->device_handle NULL?
>
> Next step is to verify that image->device_handle is really NULL.
>
> Please apply the following change to efi/efi_loader/efi_boottime.c
>
> @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> free(info);
> }
> error:
> + printf("*** %p\n", info->device_handle);
> return EFI_EXIT(ret);
> }
>
> Best regards
>
> Heinrich
>
>> EFI: Exit: efi_open_protocol: 2
>> data abort
>> pc : [<9ce076d0>] lr : [<9ffa85b3>]
>> reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
>> sp : 9df40e28 ip : 00000000 fp : 00000003
>> r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
>> r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
>> r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
>> Flags: NzCv IRQs off FIQs on Mode SVC_32
>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
>> UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
>> Resetting CPU ...
>>
>>
>>>
>>> Best regards
>>>
>>> Heinrich
>>>
>>>
>>
>>
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-10 20:05 ` Heinrich Schuchardt
2021-06-10 21:02 ` Heinrich Schuchardt
@ 2021-06-11 15:08 ` Matwey V. Kornilov
2021-06-12 2:05 ` AKASHI Takahiro
1 sibling, 1 reply; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-11 15:08 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: michael, U-Boot Mailing List
чт, 10 июн. 2021 г. в 23:05, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>
> On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> > вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>
> >> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> >>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> >>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>>>>
> >>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> >>>>>> Hi,
> >>>>>>
> >>>>>> I've found that
> >>>>>>
> >>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> >>>>>> /EFI/boot")
> >>>>>>
> >>>>>> breaks running EFI application from USB device on BeagleBone Black
> >>>>>> (am335x) device.
> >>>>>>
> >>>>>> With this patch I see the following:
> >>>>>>
> >>>>>> Booting /efi\boot\bootarm.efi
> >>>>>> Welcome to GRUB!
> >>>>>>
> >>>>>> data abort
> >>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> >>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> >>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> >>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> >>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> >>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> >>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> >>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> >>>>> > Resetting CPU ...
> >>>>>
> >>>>> Hello Matwey,
> >>>>>
> >>>>> thank you for reporting the issue.
> >>>>>
> >>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> >>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> >>>>>
> >>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>>>> All code
> >>>>> ========
> >>>>> 0: e3500000 cmp r0, #0
> >>>>> 4: 0a000015 beq 0x60
> >>>>> 8: e590000c ldr r0, [r0, #12]
> >>>>> c: eb00f96e bl 0x3e5cc
> >>>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> >>>>> instruction
> >>>>>
> >>>>> Code starting with the faulting instruction
> >>>>> ===========================================
> >>>>> 0: e5d03000 ldrb r3, [r0]
> >>>>>
> >>>>> Looking at the disassembly above we see that reading memory location
> >>>>> NULL fails.
> >>>>>
> >>>>> We need to find out where the exception occurs. The code position is
> >>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> >>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
> >>>>>
> >>>>> Please, add the following line at the start of grub.cfg to get more
> >>>>> output from GRUB:
> >>>>>
> >>>>> debug=all
> >>>>
> >>>> This doesn't provide any additional output from GRUB :(
> >>>>
> >>>>>
> >>>>> When building U-Boot, please, add
> >>>>>
> >>>>> #define DEBUG 1
> >>>>>
> >>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> >>>>> #include <common.h>.
> >>>>
> >>>>
> >>>> This doesn't provide much output as well:
> >>>>
> >>>> Scanning disk mmc@48060000.blk...
> >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> ** Unrecognized filesystem type **
> >>>> Scanning disk mmc@481d8000.blk...
> >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>> NULL)
> >>>> Found 3 disks
> >>>
> >>> This implies that GRUB is crashing before even accessing the file system
> >>> (including grub.cfg).
> >>>
> >>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> >>> /EFI/boot/bootarm.efi. It boots without problem.
> >>>
> >>> What version of GRUB are you using?
> >>> How were you booting before updating U-Boot?
> >>> What version of U-Boot are you using where the error occurs?
> >>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> >>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> >>> used by installers.
> >>>
> >>> If the boot manager is started by distroboot it may not have an
> >>> appropriate device path. It tries to load the file given by environment
> >>> variable $fdtfile from the boot device.
> >>>
> >>> From the U-Boot console could you, please, try:
> >>>
> >>> 1)
> >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> >>> bootefi bootmgr
> >>>
> >>>
> >>> 2)
> >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> >>> load usb 0:2 $fdt_addr_r dtb
> >>> bootefi bootmgr $fdt_addr_r
> >>>
> >>> where you need to replace dtb by the correct device tree file and adjust
> >>> the partition numbers.
> >>>
> >>> Best regards
> >>>
> >>> Heinrich
> >>
> >> To catch the earlier EFI API calls you can add
> >>
> >> #define DEBUG 1
> >>
> >> to lib/efi_loader/efi_boottime.c
> >
> >
> > Welcome to GRUB!
> >
> > EFI: Entry efi_locate_handle_ext(2,
> > 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
>
> EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
>
> This call could be from grub-core/video/efi_gop.c, check_protocol().
>
> > EFI: Exit: efi_locate_handle_ext: 14
>
> EFI_NOT_FOUND
>
> > EFI: Entry efi_open_protocol(9df5f298,
> > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > 0x2)
>
> EFI_LOADED_IMAGE_PROTOCOL_GUID
>
> This call could be from grub-core/kern/efi/efi.c,
> grub_efi_get_loaded_image().
>
> > EFI: Exit: efi_open_protocol: 0
> > EFI: Entry efi_open_protocol(00000000,
>
> The parameter @handle must not be NULL.
>
> > 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
>
> EFI_DEVICE_PATH_PROTOCOL_GUID
>
> This could be called from grub-core/kern/efi/efi.c,
> grub_efi_get_device_path() which is invoked from
> grub_machine_get_bootlocation().
>
> > 0x2)
>
> EFI_INVALID_PARAMETER is returned because the handle is NULL.
>
> I could partially reproduce the problem by setting
>
> info->device_handle = NULL;
>
> at the end of efi_setup_loaded_image():
>
> Welcome to GRUB!
>
> EFI: Entry efi_open_protocol(79fdea40,
> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> EFI: Exit: efi_open_protocol: 0
> EFI: Entry efi_open_protocol(00000000,
> 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> EFI: Exit: efi_open_protocol: 2
> error: disk `,msdos2' not found.
> grub rescue> >
>
> This leaves me with two questions:
>
> Why does GRUB not handle
>
> *device = grub_efidisk_get_device_name (NULL);
>
> gracefully? Maybe it is because it tries to print via the graphical
> output protocol which does not exist?
>
> Why is image->device_handle NULL?
>
> Next step is to verify that image->device_handle is really NULL.
>
> Please apply the following change to efi/efi_loader/efi_boottime.c
>
> @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> free(info);
> }
> error:
> + printf("*** %p\n", info->device_handle);
> return EFI_EXIT(ret);
> }
Booting /efi\boot\bootarm.efi
EFI: Entry efi_load_image(0, 9df56d90,
/VenHw(e61d73b9-a384-4acc-aeab-82e828f3628b)/UsbClass(0x13fe,0x4123,0x0,0x0,0x0)/HD(1,GPT,c3d6ff69-0ec0-4be1-91a3-e5fbeed5c4b9,0x2000,0x8000)/efi\boot\bootarm.efi,
82000000, 929792, 9df40ea0)
*** 00000000
EFI: Exit: efi_load_image: 0
EFI: Entry efi_start_image(9df5f298, 9df40e78, 9df40e7c)
EFI: Call: efi_open_protocol(image_handle, &efi_guid_loaded_image,
&info, NULL, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL)
EFI: Entry efi_open_protocol(9df5f298,
5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e60, 00000000, 00000000,
0x2)
EFI: Exit: efi_open_protocol: 0
>
> Best regards
>
> Heinrich
>
> > EFI: Exit: efi_open_protocol: 2
> > data abort
> > pc : [<9ce076d0>] lr : [<9ffa85b3>]
> > reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> > sp : 9df40e28 ip : 00000000 fp : 00000003
> > r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
> > r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> > r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> > Flags: NzCv IRQs off FIQs on Mode SVC_32
> > Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> > Resetting CPU ...
> >
> >
> >>
> >> Best regards
> >>
> >> Heinrich
> >>
> >>
> >
> >
>
>
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-10 21:02 ` Heinrich Schuchardt
@ 2021-06-11 15:18 ` Matwey V. Kornilov
0 siblings, 0 replies; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-11 15:18 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: michael, U-Boot Mailing List
пт, 11 июн. 2021 г. в 00:02, Heinrich Schuchardt <xypron.glpk@gmx.de>:
>
> On 6/10/21 10:05 PM, Heinrich Schuchardt wrote:
> > On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> >> вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>>
> >>> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> >>>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> >>>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >>>>>>
> >>>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> >>>>>>> Hi,
> >>>>>>>
> >>>>>>> I've found that
> >>>>>>>
> >>>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> >>>>>>> /EFI/boot")
> >>>>>>>
> >>>>>>> breaks running EFI application from USB device on BeagleBone Black
> >>>>>>> (am335x) device.
> >>>>>>>
> >>>>>>> With this patch I see the following:
> >>>>>>>
> >>>>>>> Booting /efi\boot\bootarm.efi
> >>>>>>> Welcome to GRUB!
> >>>>>>>
> >>>>>>> data abort
> >>>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> >>>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> >>>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> >>>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> >>>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> >>>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> >>>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> >>>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> >>>>>> > Resetting CPU ...
> >>>>>>
> >>>>>> Hello Matwey,
> >>>>>>
> >>>>>> thank you for reporting the issue.
> >>>>>>
> >>>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> >>>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> >>>>>>
> >>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >>>>>> All code
> >>>>>> ========
> >>>>>> 0: e3500000 cmp r0, #0
> >>>>>> 4: 0a000015 beq 0x60
> >>>>>> 8: e590000c ldr r0, [r0, #12]
> >>>>>> c: eb00f96e bl 0x3e5cc
> >>>>>> 10:* e5d03000 ldrb r3, [r0] <--
> >>>>>> trapping
> >>>>>> instruction
> >>>>>>
> >>>>>> Code starting with the faulting instruction
> >>>>>> ===========================================
> >>>>>> 0: e5d03000 ldrb r3, [r0]
> >>>>>>
> >>>>>> Looking at the disassembly above we see that reading memory location
> >>>>>> NULL fails.
> >>>>>>
> >>>>>> We need to find out where the exception occurs. The code position is
> >>>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> >>>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
> >>>>>>
> >>>>>> Please, add the following line at the start of grub.cfg to get more
> >>>>>> output from GRUB:
> >>>>>>
> >>>>>> debug=all
> >>>>>
> >>>>> This doesn't provide any additional output from GRUB :(
> >>>>>
> >>>>>>
> >>>>>> When building U-Boot, please, add
> >>>>>>
> >>>>>> #define DEBUG 1
> >>>>>>
> >>>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> >>>>>> #include <common.h>.
> >>>>>
> >>>>>
> >>>>> This doesn't provide much output as well:
> >>>>>
> >>>>> Scanning disk mmc@48060000.blk...
> >>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> ** Unrecognized filesystem type **
> >>>>> Scanning disk mmc@481d8000.blk...
> >>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> >>>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> >>>>> NULL)
> >>>>> Found 3 disks
> >>>>
> >>>> This implies that GRUB is crashing before even accessing the file
> >>>> system
> >>>> (including grub.cfg).
> >>>>
> >>>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> >>>> /EFI/boot/bootarm.efi. It boots without problem.
> >>>>
> >>>> What version of GRUB are you using?
> >>>> How were you booting before updating U-Boot?
> >>>> What version of U-Boot are you using where the error occurs?
> >>>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> >>>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically
> >>>> only
> >>>> used by installers.
> >>>>
> >>>> If the boot manager is started by distroboot it may not have an
> >>>> appropriate device path. It tries to load the file given by environment
> >>>> variable $fdtfile from the boot device.
> >>>>
> >>>> From the U-Boot console could you, please, try:
> >>>>
> >>>> 1)
> >>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> >>>> bootefi bootmgr
> >>>>
> >>>>
> >>>> 2)
> >>>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> >>>> load usb 0:2 $fdt_addr_r dtb
> >>>> bootefi bootmgr $fdt_addr_r
> >>>>
> >>>> where you need to replace dtb by the correct device tree file and
> >>>> adjust
> >>>> the partition numbers.
> >>>>
> >>>> Best regards
> >>>>
> >>>> Heinrich
> >>>
> >>> To catch the earlier EFI API calls you can add
> >>>
> >>> #define DEBUG 1
> >>>
> >>> to lib/efi_loader/efi_boottime.c
> >>
> >>
> >> Welcome to GRUB!
> >>
> >> EFI: Entry efi_locate_handle_ext(2,
> >> 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
> >
> > EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
> >
> > This call could be from grub-core/video/efi_gop.c, check_protocol().
> >
> >> EFI: Exit: efi_locate_handle_ext: 14
> >
> > EFI_NOT_FOUND
> >
> >> EFI: Entry efi_open_protocol(9df5f298,
> >> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> >> 0x2)
> >
> > EFI_LOADED_IMAGE_PROTOCOL_GUID
> >
> > This call could be from grub-core/kern/efi/efi.c,
> > grub_efi_get_loaded_image().
> >
> >> EFI: Exit: efi_open_protocol: 0
> >> EFI: Entry efi_open_protocol(00000000,
> >
> > The parameter @handle must not be NULL.
> >
> >> 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> >
> > EFI_DEVICE_PATH_PROTOCOL_GUID
> >
> > This could be called from grub-core/kern/efi/efi.c,
> > grub_efi_get_device_path() which is invoked from
> > grub_machine_get_bootlocation().
> >
> >> 0x2)
> >
> > EFI_INVALID_PARAMETER is returned because the handle is NULL.
> >
> > I could partially reproduce the problem by setting
> >
> > info->device_handle = NULL;
> >
> > at the end of efi_setup_loaded_image():
> >
> > Welcome to GRUB!
> >
> > EFI: Entry efi_open_protocol(79fdea40,
> > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> > EFI: Exit: efi_open_protocol: 0
> > EFI: Entry efi_open_protocol(00000000,
> > 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> > EFI: Exit: efi_open_protocol: 2
> > error: disk `,msdos2' not found.
> > grub rescue> >
> >
> > This leaves me with two questions:
> >
> > Why does GRUB not handle
> >
> > *device = grub_efidisk_get_device_name (NULL);
> >
> > gracefully? Maybe it is because it tries to print via the graphical
> > output protocol which does not exist?
>
> You wrote that you are using Suse JeOS.
>
> In grub.cfg you could replace
>
> with_gfx=1
> gfxmode=auto
>
> by
>
> with_gfx=0
> set textmode=true
>
> This should give you output on the serial console and avoid the crash.
I've tried but this doesn't seem to work :(
>
> Best regards
>
> Heinrich
>
> >
> > Why is image->device_handle NULL?
> >
> > Next step is to verify that image->device_handle is really NULL.
> >
> > Please apply the following change to efi/efi_loader/efi_boottime.c
> >
> > @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> > free(info);
> > }
> > error:
> > + printf("*** %p\n", info->device_handle);
> > return EFI_EXIT(ret);
> > }
> >
> > Best regards
> >
> > Heinrich
> >
> >> EFI: Exit: efi_open_protocol: 2
> >> data abort
> >> pc : [<9ce076d0>] lr : [<9ffa85b3>]
> >> reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> >> sp : 9df40e28 ip : 00000000 fp : 00000003
> >> r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
> >> r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> >> r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> >> Flags: NzCv IRQs off FIQs on Mode SVC_32
> >> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> >> UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> >> Resetting CPU ...
> >>
> >>
> >>>
> >>> Best regards
> >>>
> >>> Heinrich
> >>>
> >>>
> >>
> >>
> >
> >
>
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-11 15:08 ` Matwey V. Kornilov
@ 2021-06-12 2:05 ` AKASHI Takahiro
2021-06-12 13:50 ` Matwey V. Kornilov
0 siblings, 1 reply; 14+ messages in thread
From: AKASHI Takahiro @ 2021-06-12 2:05 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: Heinrich Schuchardt, michael, U-Boot Mailing List
Hi Matwey,
On Fri, Jun 11, 2021 at 06:08:28PM +0300, Matwey V. Kornilov wrote:
> чт, 10 июн. 2021 г. в 23:05, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> >
> > On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> > > вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > >>
> > >> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> > >>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> > >>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > >>>>>
> > >>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> > >>>>>> Hi,
> > >>>>>>
> > >>>>>> I've found that
> > >>>>>>
> > >>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> > >>>>>> /EFI/boot")
> > >>>>>>
> > >>>>>> breaks running EFI application from USB device on BeagleBone Black
> > >>>>>> (am335x) device.
> > >>>>>>
> > >>>>>> With this patch I see the following:
> > >>>>>>
> > >>>>>> Booting /efi\boot\bootarm.efi
> > >>>>>> Welcome to GRUB!
> > >>>>>>
> > >>>>>> data abort
> > >>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> > >>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> > >>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> > >>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> > >>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> > >>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> > >>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> > >>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > >>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> > >>>>> > Resetting CPU ...
> > >>>>>
> > >>>>> Hello Matwey,
> > >>>>>
> > >>>>> thank you for reporting the issue.
> > >>>>>
> > >>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> > >>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> > >>>>>
> > >>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > >>>>> All code
> > >>>>> ========
> > >>>>> 0: e3500000 cmp r0, #0
> > >>>>> 4: 0a000015 beq 0x60
> > >>>>> 8: e590000c ldr r0, [r0, #12]
> > >>>>> c: eb00f96e bl 0x3e5cc
> > >>>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> > >>>>> instruction
> > >>>>>
> > >>>>> Code starting with the faulting instruction
> > >>>>> ===========================================
> > >>>>> 0: e5d03000 ldrb r3, [r0]
> > >>>>>
> > >>>>> Looking at the disassembly above we see that reading memory location
> > >>>>> NULL fails.
> > >>>>>
> > >>>>> We need to find out where the exception occurs. The code position is
> > >>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> > >>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
> > >>>>>
> > >>>>> Please, add the following line at the start of grub.cfg to get more
> > >>>>> output from GRUB:
> > >>>>>
> > >>>>> debug=all
> > >>>>
> > >>>> This doesn't provide any additional output from GRUB :(
> > >>>>
> > >>>>>
> > >>>>> When building U-Boot, please, add
> > >>>>>
> > >>>>> #define DEBUG 1
> > >>>>>
> > >>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> > >>>>> #include <common.h>.
> > >>>>
> > >>>>
> > >>>> This doesn't provide much output as well:
> > >>>>
> > >>>> Scanning disk mmc@48060000.blk...
> > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> ** Unrecognized filesystem type **
> > >>>> Scanning disk mmc@481d8000.blk...
> > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > >>>> NULL)
> > >>>> Found 3 disks
> > >>>
> > >>> This implies that GRUB is crashing before even accessing the file system
> > >>> (including grub.cfg).
> > >>>
> > >>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> > >>> /EFI/boot/bootarm.efi. It boots without problem.
> > >>>
> > >>> What version of GRUB are you using?
> > >>> How were you booting before updating U-Boot?
> > >>> What version of U-Boot are you using where the error occurs?
> > >>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> > >>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> > >>> used by installers.
> > >>>
> > >>> If the boot manager is started by distroboot it may not have an
> > >>> appropriate device path. It tries to load the file given by environment
> > >>> variable $fdtfile from the boot device.
> > >>>
> > >>> From the U-Boot console could you, please, try:
> > >>>
> > >>> 1)
> > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > >>> bootefi bootmgr
> > >>>
> > >>>
> > >>> 2)
> > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > >>> load usb 0:2 $fdt_addr_r dtb
> > >>> bootefi bootmgr $fdt_addr_r
> > >>>
> > >>> where you need to replace dtb by the correct device tree file and adjust
> > >>> the partition numbers.
> > >>>
> > >>> Best regards
> > >>>
> > >>> Heinrich
> > >>
> > >> To catch the earlier EFI API calls you can add
> > >>
> > >> #define DEBUG 1
> > >>
> > >> to lib/efi_loader/efi_boottime.c
> > >
> > >
> > > Welcome to GRUB!
> > >
> > > EFI: Entry efi_locate_handle_ext(2,
> > > 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
> >
> > EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
> >
> > This call could be from grub-core/video/efi_gop.c, check_protocol().
> >
> > > EFI: Exit: efi_locate_handle_ext: 14
> >
> > EFI_NOT_FOUND
> >
> > > EFI: Entry efi_open_protocol(9df5f298,
> > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > > 0x2)
> >
> > EFI_LOADED_IMAGE_PROTOCOL_GUID
> >
> > This call could be from grub-core/kern/efi/efi.c,
> > grub_efi_get_loaded_image().
> >
> > > EFI: Exit: efi_open_protocol: 0
> > > EFI: Entry efi_open_protocol(00000000,
> >
> > The parameter @handle must not be NULL.
> >
> > > 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> >
> > EFI_DEVICE_PATH_PROTOCOL_GUID
> >
> > This could be called from grub-core/kern/efi/efi.c,
> > grub_efi_get_device_path() which is invoked from
> > grub_machine_get_bootlocation().
> >
> > > 0x2)
> >
> > EFI_INVALID_PARAMETER is returned because the handle is NULL.
> >
> > I could partially reproduce the problem by setting
> >
> > info->device_handle = NULL;
> >
> > at the end of efi_setup_loaded_image():
> >
> > Welcome to GRUB!
> >
> > EFI: Entry efi_open_protocol(79fdea40,
> > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> > EFI: Exit: efi_open_protocol: 0
> > EFI: Entry efi_open_protocol(00000000,
> > 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> > EFI: Exit: efi_open_protocol: 2
> > error: disk `,msdos2' not found.
> > grub rescue> >
> >
> > This leaves me with two questions:
> >
> > Why does GRUB not handle
> >
> > *device = grub_efidisk_get_device_name (NULL);
> >
> > gracefully? Maybe it is because it tries to print via the graphical
> > output protocol which does not exist?
> >
> > Why is image->device_handle NULL?
> >
> > Next step is to verify that image->device_handle is really NULL.
> >
> > Please apply the following change to efi/efi_loader/efi_boottime.c
> >
> > @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> > free(info);
> > }
> > error:
> > + printf("*** %p\n", info->device_handle);
> > return EFI_EXIT(ret);
> > }
>
>
> Booting /efi\boot\bootarm.efi
> EFI: Entry efi_load_image(0, 9df56d90,
> /VenHw(e61d73b9-a384-4acc-aeab-82e828f3628b)/UsbClass(0x13fe,0x4123,0x0,0x0,0x0)/HD(1,GPT,c3d6ff69-0ec0-4be1-91a3-e5fbeed5c4b9,0x2000,0x8000)/efi\boot\bootarm.efi,
> 82000000, 929792, 9df40ea0)
> *** 00000000
The commit message of f3866909e350 ("distro_bootcmd: call EFI bootmgr
even without having /EFI/boot") says:
Thus, call the bootmgr before we try to boot the EFI binary inside
the removable media path.
Given the above, your log messages suggest:
- bootmgr didn't boot grub
- So grub in USB:/efi/boot/bootarm.efi was booted by bootefi
after enumerating/probing USB devices
- Then, grub tried to open a protocol using the handle of
the (boot?) device (i.e. USB partition) to access some file
and got corrupted.
- the handle was NULL because efi_dp_find_obj() in
efi_setup_loaded_image() failed and returned NULL
when booting grub
The last problem happened because:
- When bootmgr was first called, uefi subsystem was initialized
and all the U-Boot disks at this point were enumerated as UEFI disks.
Unfortunately, any USB devices had not yet enumerated due to the commit
above.
- Then, bootefi was called after enumerating USB devices in
distro_bootcmd, but the result was never reflected to UEFI
subsystem due to the limitation of the current UEFI implementation.
- So grub image was successfully loaded from USB, yet UEFI never recognized
the device.
- Please note that efi_set_bootdev() doesn't care whether the device is
already recognized by UEFI or not.
If my guess is right, this is another example of the defect in
UEFI disk implementation.
FYI, see:
https://lists.denx.de/pipermail/u-boot/2021-June/451828.html
-Takahiro Akashi
> EFI: Exit: efi_load_image: 0
> EFI: Entry efi_start_image(9df5f298, 9df40e78, 9df40e7c)
> EFI: Call: efi_open_protocol(image_handle, &efi_guid_loaded_image,
> &info, NULL, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL)
> EFI: Entry efi_open_protocol(9df5f298,
> 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e60, 00000000, 00000000,
> 0x2)
> EFI: Exit: efi_open_protocol: 0
>
>
> >
> > Best regards
> >
> > Heinrich
> >
> > > EFI: Exit: efi_open_protocol: 2
> > > data abort
> > > pc : [<9ce076d0>] lr : [<9ffa85b3>]
> > > reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> > > sp : 9df40e28 ip : 00000000 fp : 00000003
> > > r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
> > > r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> > > r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> > > Flags: NzCv IRQs off FIQs on Mode SVC_32
> > > Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> > > Resetting CPU ...
> > >
> > >
> > >>
> > >> Best regards
> > >>
> > >> Heinrich
> > >>
> > >>
> > >
> > >
> >
> >
>
>
> --
> With best regards,
> Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-12 2:05 ` AKASHI Takahiro
@ 2021-06-12 13:50 ` Matwey V. Kornilov
2021-06-13 5:28 ` AKASHI Takahiro
0 siblings, 1 reply; 14+ messages in thread
From: Matwey V. Kornilov @ 2021-06-12 13:50 UTC (permalink / raw)
To: AKASHI Takahiro, Matwey V. Kornilov, Heinrich Schuchardt,
michael, U-Boot Mailing List
сб, 12 июн. 2021 г. в 05:05, AKASHI Takahiro <takahiro.akashi@linaro.org>:
>
> Hi Matwey,
>
> On Fri, Jun 11, 2021 at 06:08:28PM +0300, Matwey V. Kornilov wrote:
> > чт, 10 июн. 2021 г. в 23:05, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > >
> > > On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> > > > вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > > >>
> > > >> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> > > >>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> > > >>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > > >>>>>
> > > >>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> > > >>>>>> Hi,
> > > >>>>>>
> > > >>>>>> I've found that
> > > >>>>>>
> > > >>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> > > >>>>>> /EFI/boot")
> > > >>>>>>
> > > >>>>>> breaks running EFI application from USB device on BeagleBone Black
> > > >>>>>> (am335x) device.
> > > >>>>>>
> > > >>>>>> With this patch I see the following:
> > > >>>>>>
> > > >>>>>> Booting /efi\boot\bootarm.efi
> > > >>>>>> Welcome to GRUB!
> > > >>>>>>
> > > >>>>>> data abort
> > > >>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> > > >>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> > > >>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> > > >>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> > > >>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> > > >>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> > > >>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> > > >>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > >>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> > > >>>>> > Resetting CPU ...
> > > >>>>>
> > > >>>>> Hello Matwey,
> > > >>>>>
> > > >>>>> thank you for reporting the issue.
> > > >>>>>
> > > >>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> > > >>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> > > >>>>>
> > > >>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > >>>>> All code
> > > >>>>> ========
> > > >>>>> 0: e3500000 cmp r0, #0
> > > >>>>> 4: 0a000015 beq 0x60
> > > >>>>> 8: e590000c ldr r0, [r0, #12]
> > > >>>>> c: eb00f96e bl 0x3e5cc
> > > >>>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> > > >>>>> instruction
> > > >>>>>
> > > >>>>> Code starting with the faulting instruction
> > > >>>>> ===========================================
> > > >>>>> 0: e5d03000 ldrb r3, [r0]
> > > >>>>>
> > > >>>>> Looking at the disassembly above we see that reading memory location
> > > >>>>> NULL fails.
> > > >>>>>
> > > >>>>> We need to find out where the exception occurs. The code position is
> > > >>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> > > >>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
> > > >>>>>
> > > >>>>> Please, add the following line at the start of grub.cfg to get more
> > > >>>>> output from GRUB:
> > > >>>>>
> > > >>>>> debug=all
> > > >>>>
> > > >>>> This doesn't provide any additional output from GRUB :(
> > > >>>>
> > > >>>>>
> > > >>>>> When building U-Boot, please, add
> > > >>>>>
> > > >>>>> #define DEBUG 1
> > > >>>>>
> > > >>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> > > >>>>> #include <common.h>.
> > > >>>>
> > > >>>>
> > > >>>> This doesn't provide much output as well:
> > > >>>>
> > > >>>> Scanning disk mmc@48060000.blk...
> > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> ** Unrecognized filesystem type **
> > > >>>> Scanning disk mmc@481d8000.blk...
> > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > >>>> NULL)
> > > >>>> Found 3 disks
> > > >>>
> > > >>> This implies that GRUB is crashing before even accessing the file system
> > > >>> (including grub.cfg).
> > > >>>
> > > >>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> > > >>> /EFI/boot/bootarm.efi. It boots without problem.
> > > >>>
> > > >>> What version of GRUB are you using?
> > > >>> How were you booting before updating U-Boot?
> > > >>> What version of U-Boot are you using where the error occurs?
> > > >>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> > > >>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> > > >>> used by installers.
> > > >>>
> > > >>> If the boot manager is started by distroboot it may not have an
> > > >>> appropriate device path. It tries to load the file given by environment
> > > >>> variable $fdtfile from the boot device.
> > > >>>
> > > >>> From the U-Boot console could you, please, try:
> > > >>>
> > > >>> 1)
> > > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > > >>> bootefi bootmgr
> > > >>>
> > > >>>
> > > >>> 2)
> > > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > > >>> load usb 0:2 $fdt_addr_r dtb
> > > >>> bootefi bootmgr $fdt_addr_r
> > > >>>
> > > >>> where you need to replace dtb by the correct device tree file and adjust
> > > >>> the partition numbers.
> > > >>>
> > > >>> Best regards
> > > >>>
> > > >>> Heinrich
> > > >>
> > > >> To catch the earlier EFI API calls you can add
> > > >>
> > > >> #define DEBUG 1
> > > >>
> > > >> to lib/efi_loader/efi_boottime.c
> > > >
> > > >
> > > > Welcome to GRUB!
> > > >
> > > > EFI: Entry efi_locate_handle_ext(2,
> > > > 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
> > >
> > > EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
> > >
> > > This call could be from grub-core/video/efi_gop.c, check_protocol().
> > >
> > > > EFI: Exit: efi_locate_handle_ext: 14
> > >
> > > EFI_NOT_FOUND
> > >
> > > > EFI: Entry efi_open_protocol(9df5f298,
> > > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > > > 0x2)
> > >
> > > EFI_LOADED_IMAGE_PROTOCOL_GUID
> > >
> > > This call could be from grub-core/kern/efi/efi.c,
> > > grub_efi_get_loaded_image().
> > >
> > > > EFI: Exit: efi_open_protocol: 0
> > > > EFI: Entry efi_open_protocol(00000000,
> > >
> > > The parameter @handle must not be NULL.
> > >
> > > > 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > >
> > > EFI_DEVICE_PATH_PROTOCOL_GUID
> > >
> > > This could be called from grub-core/kern/efi/efi.c,
> > > grub_efi_get_device_path() which is invoked from
> > > grub_machine_get_bootlocation().
> > >
> > > > 0x2)
> > >
> > > EFI_INVALID_PARAMETER is returned because the handle is NULL.
> > >
> > > I could partially reproduce the problem by setting
> > >
> > > info->device_handle = NULL;
> > >
> > > at the end of efi_setup_loaded_image():
> > >
> > > Welcome to GRUB!
> > >
> > > EFI: Entry efi_open_protocol(79fdea40,
> > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> > > EFI: Exit: efi_open_protocol: 0
> > > EFI: Entry efi_open_protocol(00000000,
> > > 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> > > EFI: Exit: efi_open_protocol: 2
> > > error: disk `,msdos2' not found.
> > > grub rescue> >
> > >
> > > This leaves me with two questions:
> > >
> > > Why does GRUB not handle
> > >
> > > *device = grub_efidisk_get_device_name (NULL);
> > >
> > > gracefully? Maybe it is because it tries to print via the graphical
> > > output protocol which does not exist?
> > >
> > > Why is image->device_handle NULL?
> > >
> > > Next step is to verify that image->device_handle is really NULL.
> > >
> > > Please apply the following change to efi/efi_loader/efi_boottime.c
> > >
> > > @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> > > free(info);
> > > }
> > > error:
> > > + printf("*** %p\n", info->device_handle);
> > > return EFI_EXIT(ret);
> > > }
> >
> >
> > Booting /efi\boot\bootarm.efi
> > EFI: Entry efi_load_image(0, 9df56d90,
> > /VenHw(e61d73b9-a384-4acc-aeab-82e828f3628b)/UsbClass(0x13fe,0x4123,0x0,0x0,0x0)/HD(1,GPT,c3d6ff69-0ec0-4be1-91a3-e5fbeed5c4b9,0x2000,0x8000)/efi\boot\bootarm.efi,
> > 82000000, 929792, 9df40ea0)
> > *** 00000000
>
> The commit message of f3866909e350 ("distro_bootcmd: call EFI bootmgr
> even without having /EFI/boot") says:
> Thus, call the bootmgr before we try to boot the EFI binary inside
> the removable media path.
>
> Given the above, your log messages suggest:
> - bootmgr didn't boot grub
> - So grub in USB:/efi/boot/bootarm.efi was booted by bootefi
> after enumerating/probing USB devices
> - Then, grub tried to open a protocol using the handle of
> the (boot?) device (i.e. USB partition) to access some file
> and got corrupted.
> - the handle was NULL because efi_dp_find_obj() in
> efi_setup_loaded_image() failed and returned NULL
> when booting grub
>
> The last problem happened because:
> - When bootmgr was first called, uefi subsystem was initialized
> and all the U-Boot disks at this point were enumerated as UEFI disks.
> Unfortunately, any USB devices had not yet enumerated due to the commit
> above.
> - Then, bootefi was called after enumerating USB devices in
> distro_bootcmd, but the result was never reflected to UEFI
> subsystem due to the limitation of the current UEFI implementation.
> - So grub image was successfully loaded from USB, yet UEFI never recognized
> the device.
> - Please note that efi_set_bootdev() doesn't care whether the device is
> already recognized by UEFI or not.
>
> If my guess is right, this is another example of the defect in
> UEFI disk implementation.
> FYI, see:
> https://lists.denx.de/pipermail/u-boot/2021-June/451828.html
Seems to make sense. Thank you for the explanation.
But how to fix it?
>
> -Takahiro Akashi
>
>
> > EFI: Exit: efi_load_image: 0
> > EFI: Entry efi_start_image(9df5f298, 9df40e78, 9df40e7c)
> > EFI: Call: efi_open_protocol(image_handle, &efi_guid_loaded_image,
> > &info, NULL, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL)
> > EFI: Entry efi_open_protocol(9df5f298,
> > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e60, 00000000, 00000000,
> > 0x2)
> > EFI: Exit: efi_open_protocol: 0
> >
> >
> > >
> > > Best regards
> > >
> > > Heinrich
> > >
> > > > EFI: Exit: efi_open_protocol: 2
> > > > data abort
> > > > pc : [<9ce076d0>] lr : [<9ffa85b3>]
> > > > reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> > > > sp : 9df40e28 ip : 00000000 fp : 00000003
> > > > r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
>
> > > > r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> > > > r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> > > > Flags: NzCv IRQs off FIQs on Mode SVC_32
> > > > Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > > UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> > > > Resetting CPU ...
> > > >
> > > >
> > > >>
> > > >> Best regards
> > > >>
> > > >> Heinrich
> > > >>
> > > >>
> > > >
> > > >
> > >
> > >
> >
> >
> > --
> > With best regards,
> > Matwey V. Kornilov
--
With best regards,
Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot")
2021-06-12 13:50 ` Matwey V. Kornilov
@ 2021-06-13 5:28 ` AKASHI Takahiro
0 siblings, 0 replies; 14+ messages in thread
From: AKASHI Takahiro @ 2021-06-13 5:28 UTC (permalink / raw)
To: Matwey V. Kornilov; +Cc: Heinrich Schuchardt, michael, U-Boot Mailing List
On Sat, Jun 12, 2021 at 04:50:46PM +0300, Matwey V. Kornilov wrote:
> сб, 12 июн. 2021 г. в 05:05, AKASHI Takahiro <takahiro.akashi@linaro.org>:
> >
> > Hi Matwey,
> >
> > On Fri, Jun 11, 2021 at 06:08:28PM +0300, Matwey V. Kornilov wrote:
> > > чт, 10 июн. 2021 г. в 23:05, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > > >
> > > > On 6/7/21 7:51 PM, Matwey V. Kornilov wrote:
> > > > > вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > > > >>
> > > > >> On 6/6/21 6:21 PM, Heinrich Schuchardt wrote:
> > > > >>> On 6/6/21 5:42 PM, Matwey V. Kornilov wrote:
> > > > >>>> вс, 6 июн. 2021 г. в 18:20, Heinrich Schuchardt <xypron.glpk@gmx.de>:
> > > > >>>>>
> > > > >>>>> On 6/6/21 4:37 PM, Matwey V. Kornilov wrote:
> > > > >>>>>> Hi,
> > > > >>>>>>
> > > > >>>>>> I've found that
> > > > >>>>>>
> > > > >>>>>> f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having
> > > > >>>>>> /EFI/boot")
> > > > >>>>>>
> > > > >>>>>> breaks running EFI application from USB device on BeagleBone Black
> > > > >>>>>> (am335x) device.
> > > > >>>>>>
> > > > >>>>>> With this patch I see the following:
> > > > >>>>>>
> > > > >>>>>> Booting /efi\boot\bootarm.efi
> > > > >>>>>> Welcome to GRUB!
> > > > >>>>>>
> > > > >>>>>> data abort
> > > > >>>>>> pc : [<9ce0b6d0>] lr : [<9ffab7c7>]
> > > > >>>>>> reloc pc : [<7d69d6d0>] lr : [<8083d7c7>]
> > > > >>>>>> sp : 9df44e28 ip : 9ffdfe90 fp : 00000003
> > > > >>>>>> r10: 9ffe3300 r9 : 00000000 r8 : 9df6fe88
> > > > >>>>>> r7 : 00000000 r6 : 9ce5da08 r5 : 9ce571f8 r4 : 9ce2c040
> > > > >>>>>> r3 : 00000000 r2 : 00000001 r1 : 9ce56598 r0 : 00000000
> > > > >>>>>> Flags: NzCv IRQs off FIQs on Mode SVC_32
> > > > >>>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > > >>>>> > UEFI image [0x9ce46000:0x9cf28fff] '/efi\boot\bootarm.efi'
> > > > >>>>> > Resetting CPU ...
> > > > >>>>>
> > > > >>>>> Hello Matwey,
> > > > >>>>>
> > > > >>>>> thank you for reporting the issue.
> > > > >>>>>
> > > > >>>>> $ echo 'Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)' |
> > > > >>>>> CROSS_COMPILE=arm-linux-gnueabihf- ARCH=arm scripts/decodecode
> > > > >>>>>
> > > > >>>>> Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > > >>>>> All code
> > > > >>>>> ========
> > > > >>>>> 0: e3500000 cmp r0, #0
> > > > >>>>> 4: 0a000015 beq 0x60
> > > > >>>>> 8: e590000c ldr r0, [r0, #12]
> > > > >>>>> c: eb00f96e bl 0x3e5cc
> > > > >>>>> 10:* e5d03000 ldrb r3, [r0] <-- trapping
> > > > >>>>> instruction
> > > > >>>>>
> > > > >>>>> Code starting with the faulting instruction
> > > > >>>>> ===========================================
> > > > >>>>> 0: e5d03000 ldrb r3, [r0]
> > > > >>>>>
> > > > >>>>> Looking at the disassembly above we see that reading memory location
> > > > >>>>> NULL fails.
> > > > >>>>>
> > > > >>>>> We need to find out where the exception occurs. The code position is
> > > > >>>>> neither in bootarm.efi nor in U-Boot (9ce0b6d0 is lower than the load
> > > > >>>>> position of bootarm.efi, so it is below the relocated U-Boot code).
> > > > >>>>>
> > > > >>>>> Please, add the following line at the start of grub.cfg to get more
> > > > >>>>> output from GRUB:
> > > > >>>>>
> > > > >>>>> debug=all
> > > > >>>>
> > > > >>>> This doesn't provide any additional output from GRUB :(
> > > > >>>>
> > > > >>>>>
> > > > >>>>> When building U-Boot, please, add
> > > > >>>>>
> > > > >>>>> #define DEBUG 1
> > > > >>>>>
> > > > >>>>> in lib/efi_loader/efi_disk.c and lib/efi_loader_file.c a line before
> > > > >>>>> #include <common.h>.
> > > > >>>>
> > > > >>>>
> > > > >>>> This doesn't provide much output as well:
> > > > >>>>
> > > > >>>> Scanning disk mmc@48060000.blk...
> > > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> ** Unrecognized filesystem type **
> > > > >>>> Scanning disk mmc@481d8000.blk...
> > > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> EFI: Call: efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle,
> > > > >>>> &efi_guid_device_path, diskobj->dp, &efi_block_io_guid, &diskobj->ops,
> > > > >>>> NULL)
> > > > >>>> Found 3 disks
> > > > >>>
> > > > >>> This implies that GRUB is crashing before even accessing the file system
> > > > >>> (including grub.cfg).
> > > > >>>
> > > > >>> On an OrangePi PC I deleted /boot.scr and moved grubarm.efi to
> > > > >>> /EFI/boot/bootarm.efi. It boots without problem.
> > > > >>>
> > > > >>> What version of GRUB are you using?
> > > > >>> How were you booting before updating U-Boot?
> > > > >>> What version of U-Boot are you using where the error occurs?
> > > > >>> Why do you have grub in /EFI/boot/bootarm.efi and not in a distro
> > > > >>> specific path, e.g. /EFI/debian/grubarm.efi? /EFI/boot is typically only
> > > > >>> used by installers.
> > > > >>>
> > > > >>> If the boot manager is started by distroboot it may not have an
> > > > >>> appropriate device path. It tries to load the file given by environment
> > > > >>> variable $fdtfile from the boot device.
> > > > >>>
> > > > >>> From the U-Boot console could you, please, try:
> > > > >>>
> > > > >>> 1)
> > > > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > > > >>> bootefi bootmgr
> > > > >>>
> > > > >>>
> > > > >>> 2)
> > > > >>> load usb 0:1 $kernel_addr_r EFI/boot/bootarm.efi
> > > > >>> load usb 0:2 $fdt_addr_r dtb
> > > > >>> bootefi bootmgr $fdt_addr_r
> > > > >>>
> > > > >>> where you need to replace dtb by the correct device tree file and adjust
> > > > >>> the partition numbers.
> > > > >>>
> > > > >>> Best regards
> > > > >>>
> > > > >>> Heinrich
> > > > >>
> > > > >> To catch the earlier EFI API calls you can add
> > > > >>
> > > > >> #define DEBUG 1
> > > > >>
> > > > >> to lib/efi_loader/efi_boottime.c
> > > > >
> > > > >
> > > > > Welcome to GRUB!
> > > > >
> > > > > EFI: Entry efi_locate_handle_ext(2,
> > > > > 9042a9de-23dc-4a38-96fb-7aded080516a, 00000000, 9df40dfc, 9ce2d660)
> > > >
> > > > EFI_GRAPHICS_OUTPUT_PROTOCOL_GUID
> > > >
> > > > This call could be from grub-core/video/efi_gop.c, check_protocol().
> > > >
> > > > > EFI: Exit: efi_locate_handle_ext: 14
> > > >
> > > > EFI_NOT_FOUND
> > > >
> > > > > EFI: Entry efi_open_protocol(9df5f298,
> > > > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > > > > 0x2)
> > > >
> > > > EFI_LOADED_IMAGE_PROTOCOL_GUID
> > > >
> > > > This call could be from grub-core/kern/efi/efi.c,
> > > > grub_efi_get_loaded_image().
> > > >
> > > > > EFI: Exit: efi_open_protocol: 0
> > > > > EFI: Entry efi_open_protocol(00000000,
> > > >
> > > > The parameter @handle must not be NULL.
> > > >
> > > > > 09576e91-6d3f-11d2-8e39-00a0c969723b, 9df40e14, 9df5f298, 00000000,
> > > >
> > > > EFI_DEVICE_PATH_PROTOCOL_GUID
> > > >
> > > > This could be called from grub-core/kern/efi/efi.c,
> > > > grub_efi_get_device_path() which is invoked from
> > > > grub_machine_get_bootlocation().
> > > >
> > > > > 0x2)
> > > >
> > > > EFI_INVALID_PARAMETER is returned because the handle is NULL.
> > > >
> > > > I could partially reproduce the problem by setting
> > > >
> > > > info->device_handle = NULL;
> > > >
> > > > at the end of efi_setup_loaded_image():
> > > >
> > > > Welcome to GRUB!
> > > >
> > > > EFI: Entry efi_open_protocol(79fdea40,
> > > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 79f570e4, 79fdea40, 00000000, 0x2)
> > > > EFI: Exit: efi_open_protocol: 0
> > > > EFI: Entry efi_open_protocol(00000000,
> > > > 09576e91-6d3f-11d2-8e39-00a0c969723b, 79f570b4, 79fdea40, 00000000, 0x2)
> > > > EFI: Exit: efi_open_protocol: 2
> > > > error: disk `,msdos2' not found.
> > > > grub rescue> >
> > > >
> > > > This leaves me with two questions:
> > > >
> > > > Why does GRUB not handle
> > > >
> > > > *device = grub_efidisk_get_device_name (NULL);
> > > >
> > > > gracefully? Maybe it is because it tries to print via the graphical
> > > > output protocol which does not exist?
> > > >
> > > > Why is image->device_handle NULL?
> > > >
> > > > Next step is to verify that image->device_handle is really NULL.
> > > >
> > > > Please apply the following change to efi/efi_loader/efi_boottime.c
> > > >
> > > > @@ -2060,6 +2069,7 @@ efi_status_t EFIAPI efi_load_image(bool boot_policy,
> > > > free(info);
> > > > }
> > > > error:
> > > > + printf("*** %p\n", info->device_handle);
> > > > return EFI_EXIT(ret);
> > > > }
> > >
> > >
> > > Booting /efi\boot\bootarm.efi
> > > EFI: Entry efi_load_image(0, 9df56d90,
> > > /VenHw(e61d73b9-a384-4acc-aeab-82e828f3628b)/UsbClass(0x13fe,0x4123,0x0,0x0,0x0)/HD(1,GPT,c3d6ff69-0ec0-4be1-91a3-e5fbeed5c4b9,0x2000,0x8000)/efi\boot\bootarm.efi,
> > > 82000000, 929792, 9df40ea0)
> > > *** 00000000
> >
> > The commit message of f3866909e350 ("distro_bootcmd: call EFI bootmgr
> > even without having /EFI/boot") says:
> > Thus, call the bootmgr before we try to boot the EFI binary inside
> > the removable media path.
> >
> > Given the above, your log messages suggest:
> > - bootmgr didn't boot grub
> > - So grub in USB:/efi/boot/bootarm.efi was booted by bootefi
> > after enumerating/probing USB devices
> > - Then, grub tried to open a protocol using the handle of
> > the (boot?) device (i.e. USB partition) to access some file
> > and got corrupted.
> > - the handle was NULL because efi_dp_find_obj() in
> > efi_setup_loaded_image() failed and returned NULL
> > when booting grub
> >
> > The last problem happened because:
> > - When bootmgr was first called, uefi subsystem was initialized
> > and all the U-Boot disks at this point were enumerated as UEFI disks.
> > Unfortunately, any USB devices had not yet enumerated due to the commit
> > above.
> > - Then, bootefi was called after enumerating USB devices in
> > distro_bootcmd, but the result was never reflected to UEFI
> > subsystem due to the limitation of the current UEFI implementation.
> > - So grub image was successfully loaded from USB, yet UEFI never recognized
> > the device.
> > - Please note that efi_set_bootdev() doesn't care whether the device is
> > already recognized by UEFI or not.
> >
> > If my guess is right, this is another example of the defect in
> > UEFI disk implementation.
> > FYI, see:
> > https://lists.denx.de/pipermail/u-boot/2021-June/451828.html
>
> Seems to make sense. Thank you for the explanation.
> But how to fix it?
If I'm correct, you need to change the boot device order and
put USB in the first place in BOOT_TARGET_DEVICES() in am335x_*.h.
But please note this is a quick hack, and the same issue can happen
to everybody and on any platform. The ultimate solution is to fix
the defect in UEFI disk implementation as I proposed in the message above.
-Takahiro Akashi
>
>
> >
> > -Takahiro Akashi
> >
> >
> > > EFI: Exit: efi_load_image: 0
> > > EFI: Entry efi_start_image(9df5f298, 9df40e78, 9df40e7c)
> > > EFI: Call: efi_open_protocol(image_handle, &efi_guid_loaded_image,
> > > &info, NULL, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL)
> > > EFI: Entry efi_open_protocol(9df5f298,
> > > 5b1b31a1-9562-11d2-8e3f-00a0c969723b, 9df40e60, 00000000, 00000000,
> > > 0x2)
> > > EFI: Exit: efi_open_protocol: 0
> > >
> > >
> > > >
> > > > Best regards
> > > >
> > > > Heinrich
> > > >
> > > > > EFI: Exit: efi_open_protocol: 2
> > > > > data abort
> > > > > pc : [<9ce076d0>] lr : [<9ffa85b3>]
> > > > > reloc pc : [<7d69d6d0>] lr : [<8083e5b3>]
> > > > > sp : 9df40e28 ip : 00000000 fp : 00000003
> > > > > r10: 9ffe2df8 r9 : 00000000 r8 : 9df5f298
> >
> > > > > r7 : 00000000 r6 : 9ce59a08 r5 : 9ce531f8 r4 : 9ce28040
> > > > > r3 : 00000000 r2 : 9ffeb328 r1 : 00000000 r0 : 00000000
> > > > > Flags: NzCv IRQs off FIQs on Mode SVC_32
> > > > > Code: e3500000 0a000015 e590000c eb00f96e (e5d03000)
> > > > > UEFI image [0x9ce42000:0x9cf24fff] '/efi\boot\bootarm.efi'
> > > > > Resetting CPU ...
> > > > >
> > > > >
> > > > >>
> > > > >> Best regards
> > > > >>
> > > > >> Heinrich
> > > > >>
> > > > >>
> > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > With best regards,
> > > Matwey V. Kornilov
>
>
>
> --
> With best regards,
> Matwey V. Kornilov
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2021-06-13 5:29 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-06 14:37 BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot") Matwey V. Kornilov
2021-06-06 15:20 ` Heinrich Schuchardt
2021-06-06 15:42 ` Matwey V. Kornilov
2021-06-06 16:21 ` Heinrich Schuchardt
2021-06-06 16:47 ` Heinrich Schuchardt
2021-06-07 17:51 ` Matwey V. Kornilov
2021-06-10 20:05 ` Heinrich Schuchardt
2021-06-10 21:02 ` Heinrich Schuchardt
2021-06-11 15:18 ` Matwey V. Kornilov
2021-06-11 15:08 ` Matwey V. Kornilov
2021-06-12 2:05 ` AKASHI Takahiro
2021-06-12 13:50 ` Matwey V. Kornilov
2021-06-13 5:28 ` AKASHI Takahiro
2021-06-07 17:36 ` Matwey V. Kornilov
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.