[{"id":1771699,"web_url":"http://patchwork.ozlabs.org/comment/1771699/","msgid":"<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>","list_archive_url":null,"date":"2017-09-20T09:08:44","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"On 14.09.17 00:05, Rob Clark wrote:\n> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n> BootOrder and BootXXXX variables to try to find an EFI payload to load\n> and boot.  This is added as a sub-command of bootefi.\n> \n> The idea is that the distro bootcmd would first try loading a payload\n> via the bootmgr, and then if that fails (ie. first boot or corrupted\n> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n> populate BootOrder and BootXXXX based on what it found.)\n> \n> Signed-off-by: Rob Clark <robdclark@gmail.com>\n\nWould it make sense to convert the bootmgr into a genuine EFI \napplication now that we have Heinrich's test framework available?\n\n\nAlex","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xxv7L3kC5z9sBW\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 20 Sep 2017 19:09:00 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid BEE63C21CB3; Wed, 20 Sep 2017 09:08:53 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5C48BC21CB3;\n\tWed, 20 Sep 2017 09:08:50 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 619B8C21CB3; Wed, 20 Sep 2017 09:08:48 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 10760C21C35\n\tfor <u-boot@lists.denx.de>; Wed, 20 Sep 2017 09:08:48 +0000 (UTC)","from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 879195CB5E;\n\tWed, 20 Sep 2017 09:08:47 +0000 (UTC)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-Virus-Scanned":"by amavisd-new at test-mx.suse.de","To":"Rob Clark <robdclark@gmail.com>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>","From":"Alexander Graf <agraf@suse.de>","Message-ID":"<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>","Date":"Wed, 20 Sep 2017 11:08:44 +0200","User-Agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)\n\tGecko/20100101 Thunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170913220546.19560-16-robdclark@gmail.com>","Content-Language":"en-US","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>, Peter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1771899,"web_url":"http://patchwork.ozlabs.org/comment/1771899/","msgid":"<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>","list_archive_url":null,"date":"2017-09-20T14:09:15","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":18760,"url":"http://patchwork.ozlabs.org/api/people/18760/","name":"Rob Clark","email":"robdclark@gmail.com"},"content":"On Wed, Sep 20, 2017 at 5:08 AM, Alexander Graf <agraf@suse.de> wrote:\n>\n>\n> On 14.09.17 00:05, Rob Clark wrote:\n>>\n>> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n>> BootOrder and BootXXXX variables to try to find an EFI payload to load\n>> and boot.  This is added as a sub-command of bootefi.\n>>\n>> The idea is that the distro bootcmd would first try loading a payload\n>> via the bootmgr, and then if that fails (ie. first boot or corrupted\n>> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n>> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n>> populate BootOrder and BootXXXX based on what it found.)\n>>\n>> Signed-off-by: Rob Clark <robdclark@gmail.com>\n>\n>\n> Would it make sense to convert the bootmgr into a genuine EFI application\n> now that we have Heinrich's test framework available?\n>\n\nI had considered that, but then decided it was nice to be able to use\nprintf()/malloc()/etc.. plus easier to gdb/debug..\n\nMaybe at some point it would be worth trying to fixup edk2 build so\nsome things like this and HII/unicode protocols and maybe a few other\nthings could be built as standalone .efi drivers and loaded by u-boot.\n(Might make sense by the time someone wants a full blown HII \"bios\nsetup menu\" ;-))\n\nBR,\n-R","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"j1qSIUjR\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xy1nz5wzlz9s8J\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 00:09:26 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid ECC4FC21F1A; Wed, 20 Sep 2017 14:09:21 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 473D8C21F03;\n\tWed, 20 Sep 2017 14:09:19 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 5A8ACC21EAE; Wed, 20 Sep 2017 14:09:17 +0000 (UTC)","from mail-lf0-f50.google.com (mail-lf0-f50.google.com\n\t[209.85.215.50])\n\tby lists.denx.de (Postfix) with ESMTPS id E1264C21DA2\n\tfor <u-boot@lists.denx.de>; Wed, 20 Sep 2017 14:09:16 +0000 (UTC)","by mail-lf0-f50.google.com with SMTP id c80so2860725lfh.0\n\tfor <u-boot@lists.denx.de>; Wed, 20 Sep 2017 07:09:16 -0700 (PDT)","by 10.46.41.75 with HTTP; Wed, 20 Sep 2017 07:09:15 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=CDNqdVq93TrGXDADb73pLLIa8G3PlJ9AoZeLtsXEbAA=;\n\tb=j1qSIUjRyvhgcYo6bAz60rMBs7tVEmJkqWhd+eRh6eGBxayWorSy3OaK0zjcX6Sj/8\n\t5lVb42EpvA5yj3HSRfOn/GawBOkP6BBicTdPaeHIERUTkKnyo88De8Z0T8Z+V6mPu4cx\n\tiVOADt90SzSHey34utDiJn68IlfhmvnMPlFSXunTBnYjSrrrOspikc1LjD7zJSJWYZj9\n\tpLHnLA9BYnrrJ15uuZT4/ZQs7rXV6MAnVmbZyfsBsqZEfnrNRbJrAlrwxjJT6hh8Wzpi\n\tzLbQKfVjBhlTM7ywJawFNugjBRjJs+iE4iRynhA69bz7cDf4q77ZOp6R0zAORUEHSpEt\n\turOA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=CDNqdVq93TrGXDADb73pLLIa8G3PlJ9AoZeLtsXEbAA=;\n\tb=FrQq0n8bHR01nGwFRHhZkT8OX1hNl4bvv6s3EUwwsj406KhYVR7kIpIzBP1rhes/k1\n\ti6TXilj9z/4/L+TCZ726vpA/04knueXz1aJNQww9jJKWdyWA0mSnZh1Ql7p9as/CNSB6\n\t47Qi7qhJa1xtnSju5MyoRd+XjuewlCpKMU3vHiMdIZu+RTgB+QltQdIGmZgDTLwV3BOf\n\tmsBtpXCjUFo26EeUuDw10eFUiNJZ5h/l5OGZ29bHdm46fZlNgQ2dXO8+4pgu92H/FHFj\n\tkxYoTANVT+lN7BOkGRy94xEBNXDAmX/T+9bKM8DCHxFzZrGJG5ptzQkn+uOqy1uLu6jF\n\tfp5w==","X-Gm-Message-State":"AHPjjUjBnLY0kJKKRE4PBdStol18+JMN1IU6Lhe6FaBSafNeqXiIxPkg\n\tyQJjGUwrnlV9uVk6WQlH2rpZkN5oQCYCwfk1Yn4=","X-Google-Smtp-Source":"AOwi7QDaacASPCr16UcX+ymOR13d32hVpu8apGxHpJOaCKbAStrSYDh4pczM65WHJnup+RC5/PeDUd0hUvZOcztiZ5w=","X-Received":"by 10.25.166.69 with SMTP id p66mr896209lfe.190.1505916556329;\n\tWed, 20 Sep 2017 07:09:16 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>\n\t<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>","From":"Rob Clark <robdclark@gmail.com>","Date":"Wed, 20 Sep 2017 10:09:15 -0400","Message-ID":"<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>","To":"Alexander Graf <agraf@suse.de>","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tPeter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1772408,"web_url":"http://patchwork.ozlabs.org/comment/1772408/","msgid":"<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","list_archive_url":null,"date":"2017-09-21T04:58:36","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi,\n\nOn 20 September 2017 at 08:09, Rob Clark <robdclark@gmail.com> wrote:\n> On Wed, Sep 20, 2017 at 5:08 AM, Alexander Graf <agraf@suse.de> wrote:\n>>\n>>\n>> On 14.09.17 00:05, Rob Clark wrote:\n>>>\n>>> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n>>> BootOrder and BootXXXX variables to try to find an EFI payload to load\n>>> and boot.  This is added as a sub-command of bootefi.\n>>>\n>>> The idea is that the distro bootcmd would first try loading a payload\n>>> via the bootmgr, and then if that fails (ie. first boot or corrupted\n>>> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n>>> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n>>> populate BootOrder and BootXXXX based on what it found.)\n>>>\n>>> Signed-off-by: Rob Clark <robdclark@gmail.com>\n>>\n>>\n>> Would it make sense to convert the bootmgr into a genuine EFI application\n>> now that we have Heinrich's test framework available?\n>>\n>\n> I had considered that, but then decided it was nice to be able to use\n> printf()/malloc()/etc.. plus easier to gdb/debug..\n>\n> Maybe at some point it would be worth trying to fixup edk2 build so\n> some things like this and HII/unicode protocols and maybe a few other\n> things could be built as standalone .efi drivers and loaded by u-boot.\n> (Might make sense by the time someone wants a full blown HII \"bios\n> setup menu\" ;-))\n\nAnother advantage of the current approach used by this series is that\nwe can test it with sandbox. With a separate EFI application we would\nlose that ability.\n\nRegards,\nSimon","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"BCNIBD+x\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"cxsbYkdt\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyPYG6ks9z9ryv\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 14:59:46 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 8F138C21E26; Thu, 21 Sep 2017 04:59:12 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5C1C8C21EE7;\n\tThu, 21 Sep 2017 04:59:11 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 9B915C21F05; Thu, 21 Sep 2017 04:59:02 +0000 (UTC)","from mail-qt0-f171.google.com (mail-qt0-f171.google.com\n\t[209.85.216.171])\n\tby lists.denx.de (Postfix) with ESMTPS id 10D5CC21ECE\n\tfor <u-boot@lists.denx.de>; Thu, 21 Sep 2017 04:58:59 +0000 (UTC)","by mail-qt0-f171.google.com with SMTP id 47so4854086qts.10\n\tfor <u-boot@lists.denx.de>; Wed, 20 Sep 2017 21:58:59 -0700 (PDT)","by 10.200.37.200 with HTTP; Wed, 20 Sep 2017 21:58:36 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=E0CALKXfYMEC00o8UTUQ42PW2IJ4KHGPoe+HVCjCzbo=;\n\tb=BCNIBD+xZXEIi3r03TYw0PRAnLyx7gmiexAQoWOtOci74/VRzR4bEx/0gm0nQeLz4T\n\t5Gbs9oTLf3I/Txl1+hb5HzPlXCxtUpEIHTBqD64TMWXGlM8883/OmIq26WK6jTy7EFM/\n\t7duN+YVtz7Ur9HuBIf9F1ey7mvjp0iomevEHnvvtYH5QtBCzPcyl4ebe+v7+AKb/luib\n\tVmMXTM5FDLyHeYvFA5vCFKjxjhbBOBHQ2EaYv2ZYN4dV6b1kkdxiu96zRKNs3nFzmPC3\n\tgvDLuyjbvKPubnmmYHGspXKPd1Jtdgnrrgtv87SEb4bCObMn+v28h+stELJWcMQrO0aO\n\t9a2w==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=E0CALKXfYMEC00o8UTUQ42PW2IJ4KHGPoe+HVCjCzbo=;\n\tb=cxsbYkdtezBkVIt8+Ym+rHdgjDM8Mt8le+3hzBoe569XFngfoLdZ//p8JB/aleI3HF\n\t7fcWD9bHFJFh3Hwet2p5pwp1TJovYFAOiJJG77eLe3U3DLroH4kmpxSDLhF6k4JehMBv\n\tR66bGNqf2oOj/lBXJxxZb3DRUYhGeKaWDMMaM="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=E0CALKXfYMEC00o8UTUQ42PW2IJ4KHGPoe+HVCjCzbo=;\n\tb=dvl6OQFZClF1IFLpNzZB7UDDud6969ezzMyo4ziYH5FSo4DRRgyKrkt5RjmXcpW5WW\n\tg4rGwYa6nXSuWKIRE5CbwtXh7VzMnjfdRYGev3zd4qXGhubTbqfIcR7VR7rxw3vKDQw4\n\twxMbdCn+Gi+o78iAVIxNV542U7I7h6OOLDN6BDdp7sdugHAo/i9V3ELRp9r4p9E10T8h\n\tvzaMdEqGTMrTdXG2giv4q1Efv9GrL7qKYaR8OHm26X8b5PXrzd2PTVKXDpMzGz5raE82\n\thYSG4tIOulVIvhHn/RhucNqIHPTclOxhUem1ha/oPwxUOFeDVewMkO+2R9fkz6tZeNsQ\n\tE3Ig==","X-Gm-Message-State":"AHPjjUhZDGUEP27hJDspQYXoZam1LZNf/herInWUG1i18kDNhDIby443\n\tsndwCSn1iQukwswyhuO+PxBImR7yLIYgq3/r7mLasQ==","X-Google-Smtp-Source":"AOwi7QC2sQyorjyhYYtJUNzYsN6NGyWVScoqLgr/fi/XOWINHJ52Y6BqfLCYLwHyTn2VxV8krcHWr/pXs8oAl26Ok/w=","X-Received":"by 10.237.37.182 with SMTP id x51mr1387655qtc.5.1505969937502;\n\tWed, 20 Sep 2017 21:58:57 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>\n\t<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>\n\t<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Wed, 20 Sep 2017 22:58:36 -0600","X-Google-Sender-Auth":"SW6nl1fJRZOzwYgScuT1Xrk3xzE","Message-ID":"<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","To":"Rob Clark <robdclark@gmail.com>","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tPeter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1772414,"web_url":"http://patchwork.ozlabs.org/comment/1772414/","msgid":"<B11F8684-A5C5-4A6B-AED0-E00D90C8A68F@suse.de>","list_archive_url":null,"date":"2017-09-21T05:07:28","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"> Am 21.09.2017 um 06:58 schrieb Simon Glass <sjg@chromium.org>:\n> \n> Hi,\n> \n>> On 20 September 2017 at 08:09, Rob Clark <robdclark@gmail.com> wrote:\n>>> On Wed, Sep 20, 2017 at 5:08 AM, Alexander Graf <agraf@suse.de> wrote:\n>>> \n>>> \n>>>> On 14.09.17 00:05, Rob Clark wrote:\n>>>> \n>>>> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n>>>> BootOrder and BootXXXX variables to try to find an EFI payload to load\n>>>> and boot.  This is added as a sub-command of bootefi.\n>>>> \n>>>> The idea is that the distro bootcmd would first try loading a payload\n>>>> via the bootmgr, and then if that fails (ie. first boot or corrupted\n>>>> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n>>>> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n>>>> populate BootOrder and BootXXXX based on what it found.)\n>>>> \n>>>> Signed-off-by: Rob Clark <robdclark@gmail.com>\n>>> \n>>> \n>>> Would it make sense to convert the bootmgr into a genuine EFI application\n>>> now that we have Heinrich's test framework available?\n>>> \n>> \n>> I had considered that, but then decided it was nice to be able to use\n>> printf()/malloc()/etc.. plus easier to gdb/debug..\n>> \n>> Maybe at some point it would be worth trying to fixup edk2 build so\n>> some things like this and HII/unicode protocols and maybe a few other\n>> things could be built as standalone .efi drivers and loaded by u-boot.\n>> (Might make sense by the time someone wants a full blown HII \"bios\n>> setup menu\" ;-))\n> \n> Another advantage of the current approach used by this series is that\n> we can test it with sandbox. With a separate EFI application we would\n> lose that ability.\n\nBy separate I meant something similar to Heinrich's self-test that is compiled in, but runs in uefi context rather than u-boot context.\n\nI agree that it's not terribly important though. I was just wondering if there was any benefit to it, as it mostly seems to consist of EFI_CALL() calls ;).\n\n\nAlex","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyPkS2tmYz9s78\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 15:07:44 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 03927C21F07; Thu, 21 Sep 2017 05:07:35 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 8C800C21DA9;\n\tThu, 21 Sep 2017 05:07:32 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 87909C21DA9; Thu, 21 Sep 2017 05:07:31 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 1EB2CC21D90\n\tfor <u-boot@lists.denx.de>; Thu, 21 Sep 2017 05:07:31 +0000 (UTC)","from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 5531EAEA4;\n\tThu, 21 Sep 2017 05:07:30 +0000 (UTC)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-Virus-Scanned":"by amavisd-new at test-mx.suse.de","Mime-Version":"1.0 (1.0)","From":"Alexander Graf <agraf@suse.de>","X-Mailer":"iPhone Mail (14G60)","In-Reply-To":"<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","Date":"Thu, 21 Sep 2017 07:07:28 +0200","Message-Id":"<B11F8684-A5C5-4A6B-AED0-E00D90C8A68F@suse.de>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>\n\t<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>\n\t<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>\n\t<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","To":"Simon Glass <sjg@chromium.org>","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tPeter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1772481,"web_url":"http://patchwork.ozlabs.org/comment/1772481/","msgid":"<150597753874.58454.11650781899317851077@achrid.arch.suse.de>","list_archive_url":null,"date":"2017-09-21T07:05:39","subject":"Re: [U-Boot] [U-Boot,v3,15/21] efi_loader: add bootmgr","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n> BootOrder and BootXXXX variables to try to find an EFI payload to load\n> and boot.  This is added as a sub-command of bootefi.\n> \n> The idea is that the distro bootcmd would first try loading a payload\n> via the bootmgr, and then if that fails (ie. first boot or corrupted\n> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n> populate BootOrder and BootXXXX based on what it found.)\n> \n> Signed-off-by: Rob Clark <robdclark@gmail.com>\n\nThanks, applied to efi-next\n\nAlex","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xySbV0PL8z9sNc\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 17:16:54 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CFB57C21F79; Thu, 21 Sep 2017 07:14:19 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5F2A8C21F5F;\n\tThu, 21 Sep 2017 07:14:17 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 58CA8C21F5F; Thu, 21 Sep 2017 07:05:44 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 98924C21F0D\n\tfor <u-boot@lists.denx.de>; Thu, 21 Sep 2017 07:05:40 +0000 (UTC)","from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 3137EABDB;\n\tThu, 21 Sep 2017 07:05:40 +0000 (UTC)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED\n\tautolearn=unavailable autolearn_force=no version=3.4.0","X-Virus-Scanned":"by amavisd-new at test-mx.suse.de","From":"Alexander Graf <agraf@suse.de>","To":"Rob Clark <robdclark@gmail.com>","Date":"Thu, 21 Sep 2017 09:05:39 +0200","Message-Id":"<150597753874.58454.11650781899317851077@achrid.arch.suse.de>","X-Mailer":"git-send-email 2.12.3","In-Reply-To":"<20170913220546.19560-16-robdclark@gmail.com>","References":"<20170913220546.19560-16-robdclark@gmail.com>","MIME-Version":": 1.0","Cc":"U-Boot Mailing List <u-boot@lists.denx.de>","Subject":"Re: [U-Boot] [U-Boot,v3,15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1772837,"web_url":"http://patchwork.ozlabs.org/comment/1772837/","msgid":"<CAF6AEGs+QTakuOh6WrRSaKvCf8rJJDxBYw0au4JjriHv4=DCwQ@mail.gmail.com>","list_archive_url":null,"date":"2017-09-21T14:22:39","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":18760,"url":"http://patchwork.ozlabs.org/api/people/18760/","name":"Rob Clark","email":"robdclark@gmail.com"},"content":"On Thu, Sep 21, 2017 at 12:58 AM, Simon Glass <sjg@chromium.org> wrote:\n> Hi,\n>\n> On 20 September 2017 at 08:09, Rob Clark <robdclark@gmail.com> wrote:\n>> On Wed, Sep 20, 2017 at 5:08 AM, Alexander Graf <agraf@suse.de> wrote:\n>>>\n>>>\n>>> On 14.09.17 00:05, Rob Clark wrote:\n>>>>\n>>>> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n>>>> BootOrder and BootXXXX variables to try to find an EFI payload to load\n>>>> and boot.  This is added as a sub-command of bootefi.\n>>>>\n>>>> The idea is that the distro bootcmd would first try loading a payload\n>>>> via the bootmgr, and then if that fails (ie. first boot or corrupted\n>>>> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n>>>> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n>>>> populate BootOrder and BootXXXX based on what it found.)\n>>>>\n>>>> Signed-off-by: Rob Clark <robdclark@gmail.com>\n>>>\n>>>\n>>> Would it make sense to convert the bootmgr into a genuine EFI application\n>>> now that we have Heinrich's test framework available?\n>>>\n>>\n>> I had considered that, but then decided it was nice to be able to use\n>> printf()/malloc()/etc.. plus easier to gdb/debug..\n>>\n>> Maybe at some point it would be worth trying to fixup edk2 build so\n>> some things like this and HII/unicode protocols and maybe a few other\n>> things could be built as standalone .efi drivers and loaded by u-boot.\n>> (Might make sense by the time someone wants a full blown HII \"bios\n>> setup menu\" ;-))\n>\n> Another advantage of the current approach used by this series is that\n> we can test it with sandbox. With a separate EFI application we would\n> lose that ability.\n>\n\njfwiw, I do actually have Shell.efi very nearly loading in sandbox..\nit crashes part-way through startup (shortly after reading \"ShellOpt\"\nvariable) and I haven't had a chance to debug further.. but I suspect\nin some form or another having VA != \"PA\" in sandbox is biting us.\n\nAnyways, getting a bit off topic, but separate .efi in sandbox doesn't\nseem like it should be completely out of the question for sandbox..\nand it would make sandbox *way* more useful for EFI testing, since\nthis is really half of the point of efi..\n\nBR,\n-R","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"pMs+q2eK\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyf300l3nz9s06\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 22 Sep 2017 00:22:51 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid AC638C21D95; Thu, 21 Sep 2017 14:22:44 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 955AFC21E0A;\n\tThu, 21 Sep 2017 14:22:42 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid C375AC21E0A; Thu, 21 Sep 2017 14:22:41 +0000 (UTC)","from mail-lf0-f46.google.com (mail-lf0-f46.google.com\n\t[209.85.215.46])\n\tby lists.denx.de (Postfix) with ESMTPS id 23FC1C21D95\n\tfor <u-boot@lists.denx.de>; Thu, 21 Sep 2017 14:22:41 +0000 (UTC)","by mail-lf0-f46.google.com with SMTP id k23so5917146lfi.11\n\tfor <u-boot@lists.denx.de>; Thu, 21 Sep 2017 07:22:41 -0700 (PDT)","by 10.46.41.75 with HTTP; Thu, 21 Sep 2017 07:22:39 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=oA3hkiJTnaG+EIn2nRcsuQRxLo0fIsl4ga7kS65dp3Y=;\n\tb=pMs+q2eKs3vUv6tlsbJfFxWrh8OzKluh32fppLbbaDKLe8TBzXc3b2tFcPwMExXK/2\n\tJRljb/5ORYVpgDzHpjQ3UtzziIyX0AL1c0m8gm1e+uktyee9OylYrVSsFCd7ILee+xko\n\tB1EsxKCP8YHeUAa5hw9jOteZjXkx/RYGNvF53xIipqMSXfPB6UiohSltC1cllLd9zegW\n\t2teY24+g/yK3ep+jXygnNsx552QVPNGj745e/OARCt56akRpzdJTLI0/vaAEUGPkvVaK\n\tCP4BmnYTwY2BHK3i7EejPxv2IeP4wKEWzzOUmjV/ePY5Zg4BFNZDSyl5OsBwPb6eCSsM\n\tHvHw==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=oA3hkiJTnaG+EIn2nRcsuQRxLo0fIsl4ga7kS65dp3Y=;\n\tb=PVZEHbZ5XZqRbCFhCME4ORSgWOsTwFe0gWuk4Lfe/PfpB/xXUfXgMQNQzYjKutj2dl\n\t9FoO+ZEMo8OXvkQYI8Sznvw0fFapgabHSD3LIhhgTor4UK7KBUgZA0b4SfA3zg17lSw+\n\tuxquOj8FtnpZaOUDoGqiHl/uhi8mDu92y7Fg8shQnp4GHzuigFWGAkQzlbc1Z0IoEvXh\n\t9xH5hPyd/HwVXUbF+uKu37QeyN6daf/N/dO8wJVbt/h7g3Yn7Dw3mPxod+wX/wgT8Xwk\n\ttosbjsvzsLBwH3MFn5jAt+Y+//23BTzOTLdWMrlgWCJGdFzMq15XSqaUTQGreKY0EEvu\n\tBnoA==","X-Gm-Message-State":"AHPjjUgVu2a1tWgt6v3B4/lQ9LLAMkV26iSEPiAy69zEiBGvX7KMjuO8\n\tfqZILjcI8T3M6LbdQW0A2EfdkI3vz6oBosCVVxE=","X-Google-Smtp-Source":"AOwi7QBBLsm8gDxEMgVGPz1hMysiZba88EmzHJxf1x8BmTlFEPJq2DS7Ymxo73Vrl+2Ki6WjgS9B6Mp0LDOM8Lly5Ys=","X-Received":"by 10.25.166.69 with SMTP id p66mr905151lfe.190.1506003760533;\n\tThu, 21 Sep 2017 07:22:40 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>\n\t<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>\n\t<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>\n\t<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>","From":"Rob Clark <robdclark@gmail.com>","Date":"Thu, 21 Sep 2017 10:22:39 -0400","Message-ID":"<CAF6AEGs+QTakuOh6WrRSaKvCf8rJJDxBYw0au4JjriHv4=DCwQ@mail.gmail.com>","To":"Simon Glass <sjg@chromium.org>","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tPeter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1774366,"web_url":"http://patchwork.ozlabs.org/comment/1774366/","msgid":"<CAPnjgZ3OMGL+x93aLSvLVPwZTnbuBhPp2FPX=9jcSHWjmfLkbw@mail.gmail.com>","list_archive_url":null,"date":"2017-09-25T02:14:05","subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Rob,\n\nOn 21 September 2017 at 08:22, Rob Clark <robdclark@gmail.com> wrote:\n> On Thu, Sep 21, 2017 at 12:58 AM, Simon Glass <sjg@chromium.org> wrote:\n>> Hi,\n>>\n>> On 20 September 2017 at 08:09, Rob Clark <robdclark@gmail.com> wrote:\n>>> On Wed, Sep 20, 2017 at 5:08 AM, Alexander Graf <agraf@suse.de> wrote:\n>>>>\n>>>>\n>>>> On 14.09.17 00:05, Rob Clark wrote:\n>>>>>\n>>>>> Similar to a \"real\" UEFI implementation, the bootmgr looks at the\n>>>>> BootOrder and BootXXXX variables to try to find an EFI payload to load\n>>>>> and boot.  This is added as a sub-command of bootefi.\n>>>>>\n>>>>> The idea is that the distro bootcmd would first try loading a payload\n>>>>> via the bootmgr, and then if that fails (ie. first boot or corrupted\n>>>>> EFI variables) it would fallback to loading bootaa64.efi.  (Which\n>>>>> would then load fallback.efi which would look for \\EFI\\*\\boot.csv and\n>>>>> populate BootOrder and BootXXXX based on what it found.)\n>>>>>\n>>>>> Signed-off-by: Rob Clark <robdclark@gmail.com>\n>>>>\n>>>>\n>>>> Would it make sense to convert the bootmgr into a genuine EFI application\n>>>> now that we have Heinrich's test framework available?\n>>>>\n>>>\n>>> I had considered that, but then decided it was nice to be able to use\n>>> printf()/malloc()/etc.. plus easier to gdb/debug..\n>>>\n>>> Maybe at some point it would be worth trying to fixup edk2 build so\n>>> some things like this and HII/unicode protocols and maybe a few other\n>>> things could be built as standalone .efi drivers and loaded by u-boot.\n>>> (Might make sense by the time someone wants a full blown HII \"bios\n>>> setup menu\" ;-))\n>>\n>> Another advantage of the current approach used by this series is that\n>> we can test it with sandbox. With a separate EFI application we would\n>> lose that ability.\n>>\n>\n> jfwiw, I do actually have Shell.efi very nearly loading in sandbox..\n> it crashes part-way through startup (shortly after reading \"ShellOpt\"\n> variable) and I haven't had a chance to debug further.. but I suspect\n> in some form or another having VA != \"PA\" in sandbox is biting us.\n>\n> Anyways, getting a bit off topic, but separate .efi in sandbox doesn't\n> seem like it should be completely out of the question for sandbox..\n> and it would make sandbox *way* more useful for EFI testing, since\n> this is really half of the point of efi..\n\nSounds very promising! The goal for sandbox is to test all U-Boot code\nwhere it is practical to do so.\n\nRegards,\nSimon","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"Xy4St1Rm\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"JrHL790p\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0pGQ3DPKz9s7F\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 25 Sep 2017 12:40:14 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 0E546C221EF; Mon, 25 Sep 2017 02:28:00 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 92E25C221B9;\n\tMon, 25 Sep 2017 02:27:28 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 0FE8EC22197; Mon, 25 Sep 2017 02:14:33 +0000 (UTC)","from mail-qt0-f175.google.com (mail-qt0-f175.google.com\n\t[209.85.216.175])\n\tby lists.denx.de (Postfix) with ESMTPS id E5F80C21C8F\n\tfor <u-boot@lists.denx.de>; Mon, 25 Sep 2017 02:14:27 +0000 (UTC)","by mail-qt0-f175.google.com with SMTP id q8so5550863qtb.5\n\tfor <u-boot@lists.denx.de>; Sun, 24 Sep 2017 19:14:27 -0700 (PDT)","by 10.200.37.200 with HTTP; Sun, 24 Sep 2017 19:14:05 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=16vtVrw7FvyNybOMuYtbBM1feK4ecl1XqsmVw8wJnSw=;\n\tb=Xy4St1RmwZ3R8NLILFW/78N3+nIv+C3aHicfeh5kBkWMu+20GoJm/+vcSh5vRgvCd7\n\t1JTs2ADmy7f/tv1uBppX5zI1kPoVBihGm4aeNCRQ57keUqUcJ60vkszpNuUuSsPX/k6D\n\t/5bgkDXk11ECGVOfPF2gP98Li/V/cxrcLtpYvohbBhY/2kID6WGzd6XNw0KEJUzUjGM4\n\tBuKpKYN0+3NugAoW7N2eZVvl2g+H0d4Mb+tzkOWQHOWcOen2lKNnJn2ZDPqTf7S4wod6\n\tAch4tzeo+PD3HPBhW1Ng9h70KteDkZExf81UFEEQMzWpgxG4c2FjxzTj+ep5LaMITjVp\n\tSh9A==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=16vtVrw7FvyNybOMuYtbBM1feK4ecl1XqsmVw8wJnSw=;\n\tb=JrHL790pGIRJZCMUec9plNj8KDDuA7Y5t2IiY+rqRa4AmnC4DAll6f5j05ETbdxYG5\n\t4dYGsKjRwYmQdkunPHF/eDYpa/6rgQJlWyXIFpmUUvFj6ic/syM5n4rhzEcP9AayjCW8\n\tc7GOGz+Nql3dCAR7JOCQJtwIsDzi7B50dUt7I="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=16vtVrw7FvyNybOMuYtbBM1feK4ecl1XqsmVw8wJnSw=;\n\tb=ZAFk/akd8/KtK/2b368XjEr+7LUMp8I9HjC13t7TvO7UVBLxHH3jrqDjaNybsA3w6A\n\t1xghyhjT3vgmYuMSjXxHRgs4Z8XBOeEwuPp45Z8avGoUb9v/+NQsYasiHPvE+BU8pQyT\n\tdKqan69jug6uGm+DXOyrClFHFSCTMoE4xWcGJwNO+vH+v13NrBSjr5/MXVuxvRr9dPyW\n\tTCmOyJkfGNknqsszF8ziU7d9VOhmOI9paI+xHND1+3lyUKg3HoAVTriCkGS46y7rPiu9\n\tFunQ1xTnkMxcEbighQOuzKVh4lQ2PN2LgWFS32JjQ4djK4EaHnSiZEz8YzPOuvDYcoDL\n\tgMug==","X-Gm-Message-State":"AHPjjUiqTDjyjLByqeXyeDmUwddneuahWvD55/lkq7/3vLokkCwvH/kI\n\t2a/yB9brQTRc96wM97Yp+wL08DDto1TRdTVXD12wqD5j","X-Google-Smtp-Source":"AOwi7QBvPuoOIcSwuxs8AClWQemf54rVA581z7pf7BbJ0lUGpfqjdS8EnYHMkXgHDiMoND3vWSh7fPbmZkB4cTK9KgY=","X-Received":"by 10.237.35.238 with SMTP id k43mr9220081qtc.214.1506305666623; \n\tSun, 24 Sep 2017 19:14:26 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<CAF6AEGs+QTakuOh6WrRSaKvCf8rJJDxBYw0au4JjriHv4=DCwQ@mail.gmail.com>","References":"<20170913220546.19560-1-robdclark@gmail.com>\n\t<20170913220546.19560-16-robdclark@gmail.com>\n\t<3b11aa60-c191-cad4-d566-fc273bd42883@suse.de>\n\t<CAF6AEGunHpwfBwh8Rji6oK+PWFk9-vLQ=x8Xg3QggOdniJ63Hg@mail.gmail.com>\n\t<CAPnjgZ2R0wkPm0TOi6GoO6dAwACPnfLS5DU3TkTH3nFjHOtBcQ@mail.gmail.com>\n\t<CAF6AEGs+QTakuOh6WrRSaKvCf8rJJDxBYw0au4JjriHv4=DCwQ@mail.gmail.com>","From":"Simon Glass <sjg@chromium.org>","Date":"Sun, 24 Sep 2017 22:14:05 -0400","X-Google-Sender-Auth":"Ckz5PLeMctxfP__0aGN97emKEUI","Message-ID":"<CAPnjgZ3OMGL+x93aLSvLVPwZTnbuBhPp2FPX=9jcSHWjmfLkbw@mail.gmail.com>","To":"Rob Clark <robdclark@gmail.com>","Cc":"Heinrich Schuchardt <xypron.glpk@gmx.de>,\n\tU-Boot Mailing List <u-boot@lists.denx.de>,\n\tPeter Jones <pjones@redhat.com>","Subject":"Re: [U-Boot] [PATCH v3 15/21] efi_loader: add bootmgr","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]