[{"id":1769810,"web_url":"http://patchwork.ozlabs.org/comment/1769810/","msgid":"","list_archive_url":null,"date":"2017-09-17T17:58:54","subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Heinrich,\n\nOn 15 September 2017 at 02:06, Heinrich Schuchardt wrote:\n> This patch series provides:\n> * corrections for the EFI event services\n> * a test framework to check the EFI API implementation\n> * unit tests covering the event services\n>\n> The EFI selftest is written such that it could be easily turned\n> into a standalone EFI application. But this would require\n> modifying the build procedures for EFI. Objcopy cannot generate\n> the necessary relocations.\n>\n> The unit tests are identified by entries in a linker generated\n> array to make them as self sufficient as possible.\n>\n> A Python test case is supplied to call run the EFI tests.\n>\n> Tested with Travis CI\n> https://travis-ci.org/xypron2/u-boot/jobs/275733784\n>\n> Of all my efi_loader patches these are the first I would like\n> to see merged.\n>\n> Simon has commented on some other patches that he misses\n> comments for all EFI API functions. I will add these with\n> a separate patch.\n>\n> Heinrich Schuchardt (10):\n> efi_loader: allow return value in EFI_CALL\n> efi_selftest: provide an EFI selftest application\n> test/py: add a test calling the EFI selftest\n> efi_loader: implement queueing of the notification function\n> efi_loader: efi_set_timer: reset signaled state\n> efi_selftest: provide unit test for event services\n> efi_loader: implement task priority level (TPL)\n> efi_selftest: test task priority levels\n> efi_loader: notify when ExitBootServices is invoked\n> efi_selftest: check notification of ExitBootServices\n>\n\nThis progress makes significant progress on EFI testing at last. I'm\nvery pleased to see it. Thank you for all the work you have put into\nthis.\n\nIn addition to this (not instead of) I would like to see EFI code\nrunning under sandbox. I don't at present see a good reason why this\ncannot be done. I am going to try to enable EFI loader support in\nsandbox to a basic level and then we can see how hard it is to get\nsome of your tests running directly in sandbox. If that works out then\nwe can add that into the mix.\n\nI think this would make for an easier development environment for new\nEFI features. For some years I have developed all new features in\nsandbox and find it painful and unproductive when I need to test every\nchange manually on a board. It should also allow us to run your tests\n(perhaps with some adaptation) with 'make tests' on a local machine\nusing sandbox. Ultimately it should be possible to expand test\ncoverage to cover all significant EFI logic.\n\n[..]\n\nRegards,\nSimon","headers":{"Return-Path":"","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=)","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=\"GYhTRkLo\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"QZFRlCdH\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwNlX5sKNz9s7G\n\tfor ;\n\tMon, 18 Sep 2017 08:16:39 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 89BC8C21DEB; Sun, 17 Sep 2017 22:16:36 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BABA7C21F2E;\n\tSun, 17 Sep 2017 18:00:52 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid CCE27C21CB1; Sun, 17 Sep 2017 18:00:51 +0000 (UTC)","from mail-qt0-f178.google.com (mail-qt0-f178.google.com\n\t[209.85.216.178])\n\tby lists.denx.de (Postfix) with ESMTPS id D4637C21F2E\n\tfor ; Sun, 17 Sep 2017 17:59:16 +0000 (UTC)","by mail-qt0-f178.google.com with SMTP id s18so5786071qta.3\n\tfor ; Sun, 17 Sep 2017 10:59:16 -0700 (PDT)","by 10.200.37.200 with HTTP; Sun, 17 Sep 2017 10:58:54 -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.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\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=4KRoNjT7hrGecXXk3Z70EtuXbMPcyrfM1lUaoA0lskg=;\n\tb=GYhTRkLoIsi/u9tuYeCzj4z0n+ROarC3NIW4pdaHGiqht97gURORN+/TOCWP5a2LSK\n\tl3vJFjw3xUb39HnBvjU/82bmGdPz0oPNJy5sJFml8fSqOW079uqubCT5V8ie0kQBBfN6\n\tty0e2kJDbXW/VGyXvUKgZn+8f6AAsGzDfI+gBXgpLP7UUnvur5dA/sTbpONkrJ81DJiW\n\t363eAgAFDxp2uovhtZc/RHmOhamaEao/GaZ6Y8PfMkQIpEqZoyCobPJVwXCOAy/Y0LSK\n\tg5BG8g0jJge1xYvZmu961iQF9pPBwuKKo/NNpVE3m29BHS/zCdZYR7oHP2imaKpmzTtN\n\ttOLg==","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=4KRoNjT7hrGecXXk3Z70EtuXbMPcyrfM1lUaoA0lskg=;\n\tb=QZFRlCdHHdl/znQ9SjbaJk6N6Gqad6YVKOBCBnh6TLMiAqoxytLp0UHtGAexjMQN7J\n\teCMbz6PlFhlM3qVAU3k5wDAawVsCUuB/hU4Mz4kYlR6Omr/wsCT8jZeFX3yz+6JN5tC7\n\tKdyaBKv16TQX5+uVA3eDzY9lAZXtw3HXG7PNk="],"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=4KRoNjT7hrGecXXk3Z70EtuXbMPcyrfM1lUaoA0lskg=;\n\tb=GNpbv49KLFY4G5oauRASwxwX0meiiAdlY2PI2+jlbTb0AqqvjKNDyNtN7RpC6JRXR+\n\tqYkthK9FYVzrC2qd7UPmtfaU5qAzrGo4+ed1X5OQLFqU1GtVevYe4Q7zVzpCnlJZ1YvJ\n\trFGBeKdZPyzkwGAsfRBZy2ArtrrZ79+XzEF8kfqweYiZILd6pNcfe+E5KglIOrG3UQS5\n\tyLO8raujE8N5unDAvgaHblhVn/Omy8Z9X3eVeF1bC1BPhcKFLrB2Qumq7jl6KMF9ltae\n\tKfuL/52TCOO0zq/XJYzwiGNJyvkm34Lif11GNabOi04jXoqB1qxgyzTW9isqZfMIGenm\n\trqFA==","X-Gm-Message-State":"AHPjjUg/6mGGVJ8h+mYmHhDF+U7+AAKH/d1dulrLdFdsxQzjjMDDAFyi\n\taau/56aNVyWh1Hjikps6xhX/UuYWbH7Z6DquTyZiXA==","X-Google-Smtp-Source":"AOwi7QAxk3Wix8BCWpAiXwdH9Ryn7SF8eeP7CmWLBbniwqjjV1fZVFejcY6nSwqBhF2M8nCfCMxllnoZdAqfJbV9sfI=","X-Received":"by 10.237.37.182 with SMTP id x51mr45168451qtc.5.1505671155547; \n\tSun, 17 Sep 2017 10:59:15 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170915080619.25250-1-xypron.glpk@gmx.de>","References":"<20170915080619.25250-1-xypron.glpk@gmx.de>","From":"Simon Glass ","Date":"Sun, 17 Sep 2017 11:58:54 -0600","X-Google-Sender-Auth":"pUs5qjedVGcIrgP45fLm-Pl5Klw","Message-ID":"","To":"Heinrich Schuchardt ","Cc":"=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , U-Boot Mailing List\n\t, Fabio Estevam ,\n\tMaxime Ripard ","Subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1769815,"web_url":"http://patchwork.ozlabs.org/comment/1769815/","msgid":"","list_archive_url":null,"date":"2017-09-17T19:34:12","subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","submitter":{"id":18760,"url":"http://patchwork.ozlabs.org/api/people/18760/","name":"Rob Clark","email":"robdclark@gmail.com"},"content":"On Sun, Sep 17, 2017 at 1:58 PM, Simon Glass wrote:\n> Hi Heinrich,\n>\n> On 15 September 2017 at 02:06, Heinrich Schuchardt wrote:\n>> This patch series provides:\n>> * corrections for the EFI event services\n>> * a test framework to check the EFI API implementation\n>> * unit tests covering the event services\n>>\n>> The EFI selftest is written such that it could be easily turned\n>> into a standalone EFI application. But this would require\n>> modifying the build procedures for EFI. Objcopy cannot generate\n>> the necessary relocations.\n>>\n>> The unit tests are identified by entries in a linker generated\n>> array to make them as self sufficient as possible.\n>>\n>> A Python test case is supplied to call run the EFI tests.\n>>\n>> Tested with Travis CI\n>> https://travis-ci.org/xypron2/u-boot/jobs/275733784\n>>\n>> Of all my efi_loader patches these are the first I would like\n>> to see merged.\n>>\n>> Simon has commented on some other patches that he misses\n>> comments for all EFI API functions. I will add these with\n>> a separate patch.\n>>\n>> Heinrich Schuchardt (10):\n>> efi_loader: allow return value in EFI_CALL\n>> efi_selftest: provide an EFI selftest application\n>> test/py: add a test calling the EFI selftest\n>> efi_loader: implement queueing of the notification function\n>> efi_loader: efi_set_timer: reset signaled state\n>> efi_selftest: provide unit test for event services\n>> efi_loader: implement task priority level (TPL)\n>> efi_selftest: test task priority levels\n>> efi_loader: notify when ExitBootServices is invoked\n>> efi_selftest: check notification of ExitBootServices\n>>\n>\n> This progress makes significant progress on EFI testing at last. I'm\n> very pleased to see it. Thank you for all the work you have put into\n> this.\n>\n> In addition to this (not instead of) I would like to see EFI code\n> running under sandbox. I don't at present see a good reason why this\n> cannot be done. I am going to try to enable EFI loader support in\n> sandbox to a basic level and then we can see how hard it is to get\n> some of your tests running directly in sandbox. If that works out then\n> we can add that into the mix.\n\nfwiw, I started on trying to get EFI_LOADER working in sandbox earlier\ntoday.. but ran into issues w/ setjmp. I probably should have kept my\nWIP but it was nothing too hard to reproduce (kconfig adds an \"||\nSANDBOX\" to depends, and one or two \"#elif defined(CONFIG_SANDBOX)\"..\nnothing that would take too long to get back to the point I was at\nstuck on setjmp/longjmp (and lack of standard system hdrs).. I just\nswitched to qemu\n\nOther than booting a real OS, seems theoretically possible to get\nEFI_LOADER working in sandbox. It should be enough to (w/ suitable\n'host bind') load/run Shell.efi and eventually SCT.efi.\n\nBR,\n-R\n\n> I think this would make for an easier development environment for new\n> EFI features. For some years I have developed all new features in\n> sandbox and find it painful and unproductive when I need to test every\n> change manually on a board. It should also allow us to run your tests\n> (perhaps with some adaptation) with 'make tests' on a local machine\n> using sandbox. Ultimately it should be possible to expand test\n> coverage to cover all significant EFI logic.\n>\n> [..]\n>\n> Regards,\n> Simon","headers":{"Return-Path":"","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=)","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=\"dCWdr6gU\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwPL30qH6z9s76\n\tfor ;\n\tMon, 18 Sep 2017 08:43:07 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CF274C21DC6; Sun, 17 Sep 2017 22:43:05 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 9B2FCC21DA4;\n\tSun, 17 Sep 2017 19:35:30 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E8D95C21DA4; Sun, 17 Sep 2017 19:35:29 +0000 (UTC)","from mail-lf0-f44.google.com (mail-lf0-f44.google.com\n\t[209.85.215.44])\n\tby lists.denx.de (Postfix) with ESMTPS id 52EBBC21DAA\n\tfor ; Sun, 17 Sep 2017 19:34:13 +0000 (UTC)","by mail-lf0-f44.google.com with SMTP id d4so6217651lfj.7\n\tfor ; Sun, 17 Sep 2017 12:34:13 -0700 (PDT)","by 10.46.41.75 with HTTP; Sun, 17 Sep 2017 12:34:12 -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=BR5DV66m97kU7VeUt+XdIXASbswmo/534pfEtxxLTD4=;\n\tb=dCWdr6gUeJLOoLlhEXqjO/MGLmbzPcWJoDK+DSyR2LWihSwyX5Yle/n6Ik+ZPC2r3J\n\tBcbWOFq+C9bzNxwxDCUk+6ItikhF+GxqTcLkFi4RnL7iJnzYHo13VSc7nY2bqmvhjsBT\n\tl3bAYNXwwkgMHhZAQJRQOgy9rma1UqWyCwLDHcmoY/IMl1GzSt7aaTA0cf3rF1S7N34E\n\tpIiHD+lwaiDSnr8GNgqYjRi/YzDrRSnEnNv/0V6gawnvKs0IzfkggZC4HXFZ1lLMyra1\n\tflKr5e/tNnsIpVrLuZKmJfyb/G+vdgNiaU7po63lU6iw1b7ZjVCTGm69aXORE2OtWGEf\n\tb3ug==","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=BR5DV66m97kU7VeUt+XdIXASbswmo/534pfEtxxLTD4=;\n\tb=OyFo50klqK7ZoXFXeRKOfNB0dydKf9ovEcZsmiPsBs7KlYac3Q3bZCpIT/r1ybxu1j\n\tixM2wZoJiuEPgSRs9u2imd3CPrP0p7I5rOFkKZDS2g676yo2ieSkt6HaqG/3hhB4vqXG\n\tXNvY8aQ9MC6DjkcR21NGnTp7Yt3wiZaR/m+Kw1YvcqvR+OA+eKP4bwDuQ+EPNhjCGUxe\n\tAQPIiaAIJa0fyb/bAI7sBhbCxj7YCH1PzQoUXUmQEw1/YMdYx4hM4uXo4OfnWe3aYO03\n\tIrrZNCy4WATm5/E6AEdoKlg/p1XHiq8BMq0LWUAjPhfYES4DI9+aRz89a9iF15L2yGC/\n\tBMGg==","X-Gm-Message-State":"AHPjjUhtV/p+sjEFQW/dI806VfoCqLWg4iAHSTLTqeCkmnl4RCov1iDa\n\tUsgQZsL/AgJoFKqooQRIybG8C/HcXGaJVNQ1Do8=","X-Google-Smtp-Source":"AOwi7QAqxzlQYYvmVZtt3ylLXt0JPvVRUb7TNR6XbnXTK3I7aagueFu3LHOsfTA2+uq/ve3gP4pVgrszBebxLjDgZdY=","X-Received":"by 10.46.43.157 with SMTP id r29mr151079ljr.56.1505676852807;\n\tSun, 17 Sep 2017 12:34:12 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<20170915080619.25250-1-xypron.glpk@gmx.de>\n\t","From":"Rob Clark ","Date":"Sun, 17 Sep 2017 15:34:12 -0400","Message-ID":"","To":"Simon Glass ","Cc":"=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tHeinrich Schuchardt , Andy Shevchenko\n\t, U-Boot Mailing List\n\t, Fabio Estevam ,\n\tMaxime Ripard ","Subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1769816,"web_url":"http://patchwork.ozlabs.org/comment/1769816/","msgid":"<1275d1af-92f7-7b04-c64a-de8bdab98772@gmx.de>","list_archive_url":null,"date":"2017-09-17T19:36:35","subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","submitter":{"id":61270,"url":"http://patchwork.ozlabs.org/api/people/61270/","name":"Heinrich Schuchardt","email":"xypron.glpk@gmx.de"},"content":"On 09/17/2017 07:58 PM, Simon Glass wrote:\n> Hi Heinrich,\n> \n> On 15 September 2017 at 02:06, Heinrich Schuchardt wrote:\n>> This patch series provides:\n>> * corrections for the EFI event services\n>> * a test framework to check the EFI API implementation\n>> * unit tests covering the event services\n>>\n>> The EFI selftest is written such that it could be easily turned\n>> into a standalone EFI application. But this would require\n>> modifying the build procedures for EFI. Objcopy cannot generate\n>> the necessary relocations.\n>>\n>> The unit tests are identified by entries in a linker generated\n>> array to make them as self sufficient as possible.\n>>\n>> A Python test case is supplied to call run the EFI tests.\n>>\n>> Tested with Travis CI\n>> https://travis-ci.org/xypron2/u-boot/jobs/275733784\n>>\n>> Of all my efi_loader patches these are the first I would like\n>> to see merged.\n>>\n>> Simon has commented on some other patches that he misses\n>> comments for all EFI API functions. I will add these with\n>> a separate patch.\n>>\n>> Heinrich Schuchardt (10):\n>> efi_loader: allow return value in EFI_CALL\n>> efi_selftest: provide an EFI selftest application\n>> test/py: add a test calling the EFI selftest\n>> efi_loader: implement queueing of the notification function\n>> efi_loader: efi_set_timer: reset signaled state\n>> efi_selftest: provide unit test for event services\n>> efi_loader: implement task priority level (TPL)\n>> efi_selftest: test task priority levels\n>> efi_loader: notify when ExitBootServices is invoked\n>> efi_selftest: check notification of ExitBootServices\n>>\n> \n> This progress makes significant progress on EFI testing at last. I'm\n> very pleased to see it. Thank you for all the work you have put into\n> this.\n> \n> In addition to this (not instead of) I would like to see EFI code\n> running under sandbox. I don't at present see a good reason why this\n> cannot be done. I am going to try to enable EFI loader support in\n> sandbox to a basic level and then we can see how hard it is to get\n> some of your tests running directly in sandbox. If that works out then\n> we can add that into the mix.\n> \n> I think this would make for an easier development environment for new\n> EFI features. For some years I have developed all new features in\n> sandbox and find it painful and unproductive when I need to test every\n> change manually on a board. It should also allow us to run your tests\n> (perhaps with some adaptation) with 'make tests' on a local machine\n> using sandbox. Ultimately it should be possible to expand test\n> coverage to cover all significant EFI logic.\n> \n> [..]\n> \n> Regards,\n> Simon\n> \nFor local testing I have been using qemu-x86_defconfig with\nCONFIG_CMD_BOOTEFI_SELFTEST=y.\n\nCf. https://lists.denx.de/pipermail/u-boot/2017-September/306510.html\n\nAs Rob pointed out enabling EFI_LOADER in the sandbox we require an\nimplementation of arch/sandbox/include/asm/setjmp.h Probably this has to\nbe based on the host architecture.\n\narch/x86/cpu/x86_64/setjmp.c teaches that setjmp.c is not yet\nimplemented in U-Boot for this architecture.\n\nLinux ./arch/x86/um/setjmp_64.S is probably a good starting point.\n\nRegards\n\nHeinrich","headers":{"Return-Path":"","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwPRh5dCpz9s7B\n\tfor ;\n\tMon, 18 Sep 2017 08:47:59 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 3C49EC21DD1; Sun, 17 Sep 2017 22:47:56 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id C32C0C21DEF;\n\tSun, 17 Sep 2017 19:39:52 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 96F5FC21D84; Sun, 17 Sep 2017 19:39:51 +0000 (UTC)","from mout.gmx.net (mout.gmx.net [212.227.17.21])\n\tby lists.denx.de (Postfix) with ESMTPS id AEC9AC21CA5\n\tfor ; Sun, 17 Sep 2017 19:38:42 +0000 (UTC)","from [192.168.123.58] ([84.118.154.110]) by mail.gmx.com (mrgmx103\n\t[212.227.17.168]) with ESMTPSA (Nemesis) id\n\t0Lkfii-1dJEB64Atg-00aYtK; Sun, 17 Sep 2017 21:38:13 +0200"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable\n\tautolearn_force=no version=3.4.0","To":"Simon Glass ","References":"<20170915080619.25250-1-xypron.glpk@gmx.de>\n\t","From":"Heinrich Schuchardt ","Message-ID":"<1275d1af-92f7-7b04-c64a-de8bdab98772@gmx.de>","Date":"Sun, 17 Sep 2017 21:36:35 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-Provags-ID":"V03:K0:NkwQfkjs6cyD/lekSdv31/CgiAhbJVf1eBfP3AxlXI/ZbExcIVg\n\tZ9VldnZqWv7BywY+sHBoJGL2kxRabjdHWJPJDq0BwWNoTpchmRdJ4ji3SPqpMkQMzeh1AB7\n\tCNFYUwZGtBLudG7+0uqopzQ3x6AO6gwvQECxkS7PaDa6Bz6oXcdKulm4pS2yW+RKxVHh82g\n\tLqfJsBqhKC3cm+MpKjkEg==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:6eSjsQ6Yg+w=:X+TmfFwaUZpWxANRoytcCZ\n\tqM+F4vJ/LG9gJp726OP7aJcHoNbDHQm7uq1dBUcrFn8oawAhMcutJlO16NLL0F7E8ZbE871O8\n\t+SE+E2t6Itx+9DUIXCVlm/lxeY4JLRnc8IAeJAR1p/l08H71ZjjVTEviznEnMJeQN2K1cS/CK\n\ty25jtFtombqCVOilCfwimvxeDBucNgA3HQXkyHw5Te9cy6J4MqMjoud1JrnryPFEWn/k7fNpL\n\tKcGvySPlZl/Zv/BKQnleqoCvN2H8fpzccUFQ3gCV/hMOIMDXZ/GYvdZFXMQXhkZDATA/XlryB\n\tV6I0b3cR2avseNvYubxBilZfoXOkFpGwTzBaK21VqOUM8N6jgdAi4nObquPpKd/JLX0SEOF6A\n\txWIiyZFkz8diKmuMqyrGVE/jcOFGE+Py92iwjFIEmkUhr890GG0bEeOTn+AMIbAQLmQK6y8k5\n\tkyx52Slkej1DxB9yA0opZJjGDVEhKIer808uHasrNaC1AhuGkYyV/0fp8B9rqWCllbWucMVsH\n\tAtaXj1VOUjvVS+tCdm9kK3s/Rprol+yMiG5dpnCckovGWDRQFF2AJy6pxtA5VhZ5q0G1Odomr\n\tdVMsZMp3y5wv0y4mu3hW4g1wSWFUE81kKPuz5vvfAoquLaWb2ZFHXBudTQ9Lz4Byu5koAilRo\n\tRMZxA2QfEi18JFTXLltjxxmEK8ze/OgsA0BIzFTgP7/Kh2WcJsS1zeC2UKEfxg6aeyYN0jnnO\n\tByLn7U/IS4phlFO3Ed8QhrWE/XsbzNO2xLuZ2J7IKAXtfaXl5b8FToprqP1A1Y1kHlB5xrjrm\n\triU8O2CEpgWhsZ5+HnKV7zcbd4p+zuk3EfJSAfMCs1fHFD8SHA=","Cc":"=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , U-Boot Mailing List\n\t, Fabio Estevam ,\n\tMaxime Ripard ","Subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1769818,"web_url":"http://patchwork.ozlabs.org/comment/1769818/","msgid":"","list_archive_url":null,"date":"2017-09-17T21:57:29","subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi,\n\nOn 17 September 2017 at 13:36, Heinrich Schuchardt wrote:\n> On 09/17/2017 07:58 PM, Simon Glass wrote:\n>> Hi Heinrich,\n>>\n>> On 15 September 2017 at 02:06, Heinrich Schuchardt wrote:\n>>> This patch series provides:\n>>> * corrections for the EFI event services\n>>> * a test framework to check the EFI API implementation\n>>> * unit tests covering the event services\n>>>\n>>> The EFI selftest is written such that it could be easily turned\n>>> into a standalone EFI application. But this would require\n>>> modifying the build procedures for EFI. Objcopy cannot generate\n>>> the necessary relocations.\n>>>\n>>> The unit tests are identified by entries in a linker generated\n>>> array to make them as self sufficient as possible.\n>>>\n>>> A Python test case is supplied to call run the EFI tests.\n>>>\n>>> Tested with Travis CI\n>>> https://travis-ci.org/xypron2/u-boot/jobs/275733784\n>>>\n>>> Of all my efi_loader patches these are the first I would like\n>>> to see merged.\n>>>\n>>> Simon has commented on some other patches that he misses\n>>> comments for all EFI API functions. I will add these with\n>>> a separate patch.\n>>>\n>>> Heinrich Schuchardt (10):\n>>> efi_loader: allow return value in EFI_CALL\n>>> efi_selftest: provide an EFI selftest application\n>>> test/py: add a test calling the EFI selftest\n>>> efi_loader: implement queueing of the notification function\n>>> efi_loader: efi_set_timer: reset signaled state\n>>> efi_selftest: provide unit test for event services\n>>> efi_loader: implement task priority level (TPL)\n>>> efi_selftest: test task priority levels\n>>> efi_loader: notify when ExitBootServices is invoked\n>>> efi_selftest: check notification of ExitBootServices\n>>>\n>>\n>> This progress makes significant progress on EFI testing at last. I'm\n>> very pleased to see it. Thank you for all the work you have put into\n>> this.\n>>\n>> In addition to this (not instead of) I would like to see EFI code\n>> running under sandbox. I don't at present see a good reason why this\n>> cannot be done. I am going to try to enable EFI loader support in\n>> sandbox to a basic level and then we can see how hard it is to get\n>> some of your tests running directly in sandbox. If that works out then\n>> we can add that into the mix.\n>>\n>> I think this would make for an easier development environment for new\n>> EFI features. For some years I have developed all new features in\n>> sandbox and find it painful and unproductive when I need to test every\n>> change manually on a board. It should also allow us to run your tests\n>> (perhaps with some adaptation) with 'make tests' on a local machine\n>> using sandbox. Ultimately it should be possible to expand test\n>> coverage to cover all significant EFI logic.\n>>\n>> [..]\n>>\n>> Regards,\n>> Simon\n>>\n> For local testing I have been using qemu-x86_defconfig with\n> CONFIG_CMD_BOOTEFI_SELFTEST=y.\n>\n> Cf. https://lists.denx.de/pipermail/u-boot/2017-September/306510.html\n>\n> As Rob pointed out enabling EFI_LOADER in the sandbox we require an\n> implementation of arch/sandbox/include/asm/setjmp.h Probably this has to\n> be based on the host architecture.\n>\n> arch/x86/cpu/x86_64/setjmp.c teaches that setjmp.c is not yet\n> implemented in U-Boot for this architecture.\n>\n> Linux ./arch/x86/um/setjmp_64.S is probably a good starting point.\n\nI don't think we should implement this in U-Boot, but instead we\nshould use the host C library version.\n\nI will send some WIP patches later today for discussion.\n\nRegards,\nSimon","headers":{"Return-Path":"","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=)","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=\"Gn7qaQkr\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"f28jXMoS\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwPfw6vN1z9s7B\n\tfor ;\n\tMon, 18 Sep 2017 08:57:44 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid AF79BC21EAB; Sun, 17 Sep 2017 22:57:43 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id AA899C21D84;\n\tSun, 17 Sep 2017 21:59:11 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 05FF0C21D7B; Sun, 17 Sep 2017 21:59:10 +0000 (UTC)","from mail-qt0-f172.google.com (mail-qt0-f172.google.com\n\t[209.85.216.172])\n\tby lists.denx.de (Postfix) with ESMTPS id 0C435C21D64\n\tfor ; Sun, 17 Sep 2017 21:57:52 +0000 (UTC)","by mail-qt0-f172.google.com with SMTP id b1so6022274qtc.4\n\tfor ; Sun, 17 Sep 2017 14:57:51 -0700 (PDT)","by 10.200.37.200 with HTTP; Sun, 17 Sep 2017 14:57:29 -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=+9jDoofFN2fLIVT8U4DRUb/Yx3zYSsC9mL5etbeWUis=;\n\tb=Gn7qaQkrmhgh5+PqI7x1L7h5FP5LyXLd5P8/NXMiw7p2Vnu3YLceOb/uW21n73tpud\n\trlwTl1KcKVj3VqqRW3ZA8qzWIU1eRqvN0NneN/M8QVjhRAwsIMI8Xfctei3cTY7gYy8q\n\tfh5rITf70n2GrbvciGuzYWm7CSrqFt5eko54bMV3LQ354CflqZ4tmNsPRCiqg7FnyFkz\n\tBYzzdJK6IUh98hWy8MObBYsj8sQYFeDzlo90cvmUgvRybLhet+2K/PkGQEGB6e2pfHCX\n\tqZfme1fofcrBGikjH3OPGKAZWRVM+eYxchsEbs3EBMiS8SGCdXcYpJA4vnaP/wEzR/r7\n\t6bRQ==","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=+9jDoofFN2fLIVT8U4DRUb/Yx3zYSsC9mL5etbeWUis=;\n\tb=f28jXMoSljKZV/csj17D9CuKpIuYIjcoYgidtdt2sgCSNCipNsZXdG/J2a+aEXcbX9\n\tQ2hBdRt2TIUwwRDdObMHEF3J9t9PjkxYqsGxL8XINtOCQPeJKhnW/DeVUQ+NvpKYFzOP\n\t+VNC1rKasAsjMZcqtOaA4v4zP5AlbrbEoodiA="],"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=+9jDoofFN2fLIVT8U4DRUb/Yx3zYSsC9mL5etbeWUis=;\n\tb=DjhHLWHYFKUJFkS8NOFzjwBxN5SXMlDhfkZup2ca8ftM/73oGlS+pX7QL6zs7GLN6n\n\tFfOwhf8WtiaIV5DIpSd984Zx5riEj5CyibQwBlXEb678BIGCz2jJdmXCk+pDa/cgcwBn\n\tfdLjR/967cj2X/DW5UMXW0WRSqbEoxlrvw64Hi6ClkFL34vbjLyjXHRXXhkXS8egtnVm\n\tYe5JuntVPTRCMyzJGGFTdeuNboUVy+9jtZxls+TmomC5YzLKD2SaxwpoEMRgJqyRnpd+\n\t5yLuK3ZgzwhgzUNqd86o3piTO1QC9srdz2oEWzgNZyhD9E2K8N+/yC/jIvGajCb3o8ez\n\tHONw==","X-Gm-Message-State":"AHPjjUh3vdhODmUIfXAeTLOySSiHFMfGnrNtmNXpfhBP4b64MMOr5s0i\n\t6HQtmTxpMAVHFWU/MdeemYQGqAOhN1bIJPchFfHlEg==","X-Google-Smtp-Source":"AOwi7QDWH9Yov7kdfzm+MdSj6faO1wH3E86VjQy/oMQdBI9Bc3W1UfbYhjkEcE026IzKRxhgC/8epjD2lKeygqfspTY=","X-Received":"by 10.200.63.154 with SMTP id d26mr47763310qtk.212.1505685470423;\n\tSun, 17 Sep 2017 14:57:50 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1275d1af-92f7-7b04-c64a-de8bdab98772@gmx.de>","References":"<20170915080619.25250-1-xypron.glpk@gmx.de>\n\t\n\t<1275d1af-92f7-7b04-c64a-de8bdab98772@gmx.de>","From":"Simon Glass ","Date":"Sun, 17 Sep 2017 15:57:29 -0600","X-Google-Sender-Auth":"VcIqFxCnjZV2tbXT9ffch2QaXy8","Message-ID":"","To":"Heinrich Schuchardt ","Cc":"=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , U-Boot Mailing List\n\t, Fabio Estevam ,\n\tMaxime Ripard ","Subject":"Re: [U-Boot] [PATCH 00/10] efi_loader: event services & API test","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]