From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C6E02C48BE5 for ; Sat, 12 Jun 2021 02:06:07 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 965E861364 for ; Sat, 12 Jun 2021 02:06:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 965E861364 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 5F9C980808; Sat, 12 Jun 2021 04:06:03 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="rXbVF/WH"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id D12298096D; Sat, 12 Jun 2021 04:06:01 +0200 (CEST) Received: from mail-pf1-x436.google.com (mail-pf1-x436.google.com [IPv6:2607:f8b0:4864:20::436]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 9006D803B9 for ; Sat, 12 Jun 2021 04:05:57 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=takahiro.akashi@linaro.org Received: by mail-pf1-x436.google.com with SMTP id s14so5869896pfd.9 for ; Fri, 11 Jun 2021 19:05:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:mail-followup-to:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=Mmbpc7HAoR80zbK3zDVg9+B2gW7a7wkOyL3jgxvLl7g=; b=rXbVF/WHv/54OH91wW2RnRMI0Wt0WyKz3+SGzA+fRmxXuLFe+vxEDuF3eRRjm5q+be Eaourz3OK9ZK7WewYOqBEG16OZ5bFKD0S27owl6AGG89MM/0gwAuWW8o1ne2O713QNFG fRMzIXHu9eR/Sv/HhvtHEm36jkvvoBO0T7jF5Syd55M3WACE+uhPC4LEkoKNwS7ImZnr XeT0Cs9wby0sHp93TDuJyHnXZJ1B73kFE7pLQOgtjL7DvpokJmBQAgkZkQOkPdIvA47G ZZiUGlwz4O3Zz5BQ6c2YTYro/wzOdcBQ4rXgfuL+fP0uE4h8+BcYk6PIhZ1oAZJH7GTP KS6w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id :mail-followup-to:references:mime-version:content-disposition :content-transfer-encoding:in-reply-to; bh=Mmbpc7HAoR80zbK3zDVg9+B2gW7a7wkOyL3jgxvLl7g=; b=kkMWJDTuH3OMJGWtql9UDnZWZdGPbfFe2sPIdtQ1GGIc5FhE37dDSL9Emjgc6e5GdW 8XX9xfmGmSXdEhBEHbLtOs0Jp5IiUflt0TNlPqPMh1Vd+xWCbt7yegsAiq/jc+gsRWc1 +dSkVMIWxYQZZXVb9dKhwHETi+qCPzzBIOA/SMLbmzav2zo17asoeZrhfvBwfOopcUrk fdEbAAlzP+i4PWukdQaUhUUBaxDqJv5WzhWthvZH/iFLoDVxEWKpQOHXPcH8ZrfuLw93 CNm9SrNS56sQHlavzVVgJPEvUvMtS75vTxJ2Oc28YtXbFQG28462QHaNQb45x9i5k24T jAOA== X-Gm-Message-State: AOAM5302wD1OdVra4j7ZbD6A/bCV3EHgwQBb6UigUpOZkHRSfSQTvYJq 5Ca8V9hT4XKAXPXzTD/l+/Afuw== X-Google-Smtp-Source: ABdhPJwU8Ul48E4uxYFKR+pJpej1ucpkaUxLV3GA7v17dhrvsXVQ5b+u1nfz69Gswx2C2VZxEcYcIw== X-Received: by 2002:a62:f947:0:b029:2e9:c502:7939 with SMTP id g7-20020a62f9470000b02902e9c5027939mr10970795pfm.34.1623463555567; Fri, 11 Jun 2021 19:05:55 -0700 (PDT) Received: from laputa (p3dd30534.tkyea130.ap.so-net.ne.jp. [61.211.5.52]) by smtp.gmail.com with ESMTPSA id d3sm11355521pjk.16.2021.06.11.19.05.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 11 Jun 2021 19:05:55 -0700 (PDT) Date: Sat, 12 Jun 2021 11:05:51 +0900 From: AKASHI Takahiro To: "Matwey V. Kornilov" Cc: Heinrich Schuchardt , michael@walle.cc, U-Boot Mailing List Subject: Re: BISECTED f3866909e350 ("distro_bootcmd: call EFI bootmgr even without having /EFI/boot") Message-ID: <20210612020551.GA50464@laputa> Mail-Followup-To: AKASHI Takahiro , "Matwey V. Kornilov" , Heinrich Schuchardt , michael@walle.cc, U-Boot Mailing List References: <5e1f6595-3371-8d56-ebfa-916d10b5aad4@gmx.de> <8d847043-ab5d-a627-ed71-5f8829bcad8c@gmx.de> <6683bdcc-d83b-de80-b0b2-4d0b8df268f0@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean Hi Matwey, On Fri, Jun 11, 2021 at 06:08:28PM +0300, Matwey V. Kornilov wrote: > чт, 10 июн. 2021 г. в 23:05, Heinrich Schuchardt : > > > > On 6/7/21 7:51 PM, Matwey V. Kornilov wrote: > > > вс, 6 июн. 2021 г. в 19:47, Heinrich Schuchardt : > > >> > > >> 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 : > > >>>>> > > >>>>> 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 . > > >>>> > > >>>> > > >>>> 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