[{"id":1745652,"web_url":"http://patchwork.ozlabs.org/comment/1745652/","msgid":"","list_archive_url":null,"date":"2017-08-12T13:38:14","subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"On 05.08.17 22:32, Heinrich Schuchardt wrote:\n> efi_open_protocol_information provides the agent and controller\n> handles as well as the attributes and open count of an protocol\n> on a handle.\n> \n> Cc: Rob Clark \n> Signed-off-by: Heinrich Schuchardt \n\nDo you have an application that leverages these interfaces? Would it be \npossible to add that application to our travis tests?\n\n\nAlex","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 3xV30l3GCQz9t32\n\tfor ;\n\tSat, 12 Aug 2017 23:40:39 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 407E2C21EE8; Sat, 12 Aug 2017 13:39:20 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id EF612C21E60;\n\tSat, 12 Aug 2017 13:39:18 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid E12EFC21E60; Sat, 12 Aug 2017 13:38:16 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 3F0E5C21ED8\n\tfor ; Sat, 12 Aug 2017 13:38:16 +0000 (UTC)","from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 09D19ABB0;\n\tSat, 12 Aug 2017 13:38:16 +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":"Heinrich Schuchardt ","References":"<20170805203230.19796-1-xypron.glpk@gmx.de>\n\t<20170805203230.19796-4-xypron.glpk@gmx.de>","From":"Alexander Graf ","Message-ID":"","Date":"Sat, 12 Aug 2017 15:38:14 +0200","User-Agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)\n\tGecko/20100101 Thunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170805203230.19796-4-xypron.glpk@gmx.de>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1745821,"web_url":"http://patchwork.ozlabs.org/comment/1745821/","msgid":"<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>","list_archive_url":null,"date":"2017-08-13T11:17:26","subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","submitter":{"id":61270,"url":"http://patchwork.ozlabs.org/api/people/61270/","name":"Heinrich Schuchardt","email":"xypron.glpk@gmx.de"},"content":"On 08/12/2017 03:38 PM, Alexander Graf wrote:\n> \n> \n> On 05.08.17 22:32, Heinrich Schuchardt wrote:\n>> efi_open_protocol_information provides the agent and controller\n>> handles as well as the attributes and open count of an protocol\n>> on a handle.\n>>\n>> Cc: Rob Clark \n>> Signed-off-by: Heinrich Schuchardt \n> \n> Do you have an application that leverages these interfaces? Would it be\n> possible to add that application to our travis tests?\n> \n> \n> Alex\n> \n\niPXE (snp.efi) uses the functions but there are other things missing to\nmake it really working.\n\nPutting new tests into the executable created by\nCMD_BOOTEFI_HELLO_COMPILE would be possible but that seems messy to me.\n\nShould we replace CMD_BOOTEFI_HELLO_COMPILE by an option that creates\nmultiple executables for the different EFI API functions we have to test?\n\nEach test then should consist of:\n- start QEMU system\n- download dtb and test executable from tftp server\n- start the test executable\n- evaluate console output\n- shutdown QEMU system\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 3xVbnZ5wN3z9sNV\n\tfor ;\n\tSun, 13 Aug 2017 21:17:53 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 3366FC21E38; Sun, 13 Aug 2017 11:17:43 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 482F9C21D7A;\n\tSun, 13 Aug 2017 11:17:41 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 39386C21D7A; Sun, 13 Aug 2017 11:17:40 +0000 (UTC)","from mout.gmx.net (mout.gmx.net [212.227.15.19])\n\tby lists.denx.de (Postfix) with ESMTPS id DA010C21C6A\n\tfor ; Sun, 13 Aug 2017 11:17:39 +0000 (UTC)","from [192.168.123.55] ([84.118.154.110]) by mail.gmx.com (mrgmx003\n\t[212.227.17.190]) with ESMTPSA (Nemesis) id\n\t0Luxaj-1dYUzr2Q2D-0108PN; Sun, 13 Aug 2017 13:17:38 +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,\n\tRCVD_IN_MSPIKE_H2 autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","To":"Alexander Graf ","References":"<20170805203230.19796-1-xypron.glpk@gmx.de>\n\t<20170805203230.19796-4-xypron.glpk@gmx.de>\n\t","From":"Heinrich Schuchardt ","Message-ID":"<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>","Date":"Sun, 13 Aug 2017 13:17:26 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-Provags-ID":"V03:K0:fZdqvOLYCLA9KLszL+IRZ0AI4m6wORl1tiMPUa0jssP3nQ64tMl\n\tcsG4lSQumFU4N3qGmIOY5OMhK3V1w26+5Wl1SkAv3NnBGrxBTOrzvWiLbJLk5pPltLFD+IZ\n\tuyXyUocl5OXjFROJJSNFzVQsiboA7hjY6zXh3LLTQsCqwqH/FlRXKOqzl2y5msfEDseZZJS\n\t230rfOChC+X7zwrBwEVBQ==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:P8ZHkyQfP7U=:+5cL8U/2UaWk5KTWAGF1Dg\n\tad6Auelk0UDsyNlnaovwZilHObCIFdte5VqtH8ZeIvT2cs5CC987SZ7ZMAlRllUgcTKDMH+1W\n\t4swz4R5+W/1WgAzHmsryi2lb5WLZ/0pEaEvzEuglaruCy14QuOFSRFWLB/0Q4xOc2DEX8txso\n\tkzVIXqt0ckt3C9BLEqS3rSz1gXlBSE5dXkPYiZfmlNXDZN74RY5rtfAvMSrxy21ngz27en48s\n\tTHxgabALQSEyQ+IvUffaQD5odWNOShpPWy5/JclWthEjZSROoSJ8L/Xu1E5ei51tecmLpD13y\n\toZQMDwRoDv4qGeEdTegNPQQ7CHd1oVJiDD6P8TS6sWov+BzNt7brNMv8Fe3aLPKMJhhH2qUM9\n\tVtTNd7EeDrucNLURFA61VRi9bT+8Ymrl3kYmupdlQIT6jwCjThWMzD318NMp36xdDrvRQIYex\n\taJwtjQ/+BCIXty5ntmLzr5w41Ako/2fO6bOyVFzhesjXB/VAIARbdGgnR26/K3Gmtc8gwjCXz\n\tzEUyNeCX3Nra0L7f4BJonRpo1qnRseA0smAKVcq0M3y3gegRuBNadr4LSbs5D7K5TJ6FpR/fN\n\t8kR4l2v1VO2AntM7oGDCVHtASmbh4S9TvmJ527+oZCS/MVLX083lUK5vX4Pzh2lozEdkiYB5L\n\ttq4Vx03gwulCnkZFQoqOm3eCLUy0iTJhhFOyUV5/IerTl1AOp5D6HdI4i9eEk4dcpFKQzdXmU\n\tfQQcLcxkdP1tfFWkbrFQe7ZncEW3fcQFxcYK8BFcj1RAWRab9lPQk+w9kfn4LEIRqchO+hlqk\n\tahHGRz51AI9WMmJcNETvRoJqEwCvAyUORTf7DmHyTKXZEqdPjw=","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","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":1745927,"web_url":"http://patchwork.ozlabs.org/comment/1745927/","msgid":"","list_archive_url":null,"date":"2017-08-13T19:24:20","subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"On 13.08.17 13:17, Heinrich Schuchardt wrote:\n> On 08/12/2017 03:38 PM, Alexander Graf wrote:\n>>\n>>\n>> On 05.08.17 22:32, Heinrich Schuchardt wrote:\n>>> efi_open_protocol_information provides the agent and controller\n>>> handles as well as the attributes and open count of an protocol\n>>> on a handle.\n>>>\n>>> Cc: Rob Clark \n>>> Signed-off-by: Heinrich Schuchardt \n>>\n>> Do you have an application that leverages these interfaces? Would it be\n>> possible to add that application to our travis tests?\n>>\n>>\n>> Alex\n>>\n> \n> iPXE (snp.efi) uses the functions but there are other things missing to\n> make it really working.\n\nAh, I see. How much is missing to make that one work for real?\n\n> Putting new tests into the executable created by\n> CMD_BOOTEFI_HELLO_COMPILE would be possible but that seems messy to me.\n> \n> Should we replace CMD_BOOTEFI_HELLO_COMPILE by an option that creates\n> multiple executables for the different EFI API functions we have to test?\n> \n> Each test then should consist of:\n> - start QEMU system\n> - download dtb and test executable from tftp server\n> - start the test executable\n> - evaluate console output\n> - shutdown QEMU system\n\nWe have most of the above already in travis today. All we would need is \nto extend it to download a built iPXE binaries and add test snippets to \ntests/py for iPXE functionality.\n\n\nAlex","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 3xVpb42l0Vz9s83\n\tfor ;\n\tMon, 14 Aug 2017 05:24:32 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid DD6F6C21D64; Sun, 13 Aug 2017 19:24:25 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 0B352C21C97;\n\tSun, 13 Aug 2017 19:24:24 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8F120C21C26; Sun, 13 Aug 2017 19:24:22 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 496A2C21C26\n\tfor ; Sun, 13 Aug 2017 19:24:22 +0000 (UTC)","from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 016CFABB3;\n\tSun, 13 Aug 2017 19:24:22 +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":"Heinrich Schuchardt ","References":"<20170805203230.19796-1-xypron.glpk@gmx.de>\n\t<20170805203230.19796-4-xypron.glpk@gmx.de>\n\t\n\t<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>","From":"Alexander Graf ","Message-ID":"","Date":"Sun, 13 Aug 2017 21:24:20 +0200","User-Agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)\n\tGecko/20100101 Thunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1745928,"web_url":"http://patchwork.ozlabs.org/comment/1745928/","msgid":"<0fc16cfb-250f-253d-0564-e34f608324f8@gmx.de>","list_archive_url":null,"date":"2017-08-13T19:32:36","subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","submitter":{"id":70394,"url":"http://patchwork.ozlabs.org/api/people/70394/","name":"Heinrich Schuchardt","email":"xypron.debian@gmx.de"},"content":"On 08/13/2017 09:24 PM, Alexander Graf wrote:\n> \n> \n> On 13.08.17 13:17, Heinrich Schuchardt wrote:\n>> On 08/12/2017 03:38 PM, Alexander Graf wrote:\n>>>\n>>>\n>>> On 05.08.17 22:32, Heinrich Schuchardt wrote:\n>>>> efi_open_protocol_information provides the agent and controller\n>>>> handles as well as the attributes and open count of an protocol\n>>>> on a handle.\n>>>>\n>>>> Cc: Rob Clark \n>>>> Signed-off-by: Heinrich Schuchardt \n>>>\n>>> Do you have an application that leverages these interfaces? Would it be\n>>> possible to add that application to our travis tests?\n>>>\n>>>\n>>> Alex\n>>>\n>>\n>> iPXE (snp.efi) uses the functions but there are other things missing to\n>> make it really working.\n> \n> Ah, I see. How much is missing to make that one work for real?\n\nBefore reaching the input prompt ConnectController and\nDisconnectController fail for the Simple Network Protocol.\n\nSo I am not able to say if the network connection will work.\n\nFor testing we will need an iSCSI target.\n\n> \n>> Putting new tests into the executable created by\n>> CMD_BOOTEFI_HELLO_COMPILE would be possible but that seems messy to me.\n>>\n>> Should we replace CMD_BOOTEFI_HELLO_COMPILE by an option that creates\n>> multiple executables for the different EFI API functions we have to test?\n>>\n>> Each test then should consist of:\n>> - start QEMU system\n>> - download dtb and test executable from tftp server\n>> - start the test executable\n>> - evaluate console output\n>> - shutdown QEMU system\n> \n> We have most of the above already in travis today. All we would need is\n> to extend it to download a built iPXE binaries and add test snippets to\n> tests/py for iPXE functionality.\n\nProving that some binary runs is not enough to test the different corner\ncases of this complex API.\n\nI would prefer to have a bunch of test binaries compiled from the U-Boot\ngit tree.\n\nBest regards\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 3xVppD02RPz9s83\n\tfor ;\n\tMon, 14 Aug 2017 05:34:11 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid C0D0BC21D79; Sun, 13 Aug 2017 19:34:06 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id E3B6BC21C4F;\n\tSun, 13 Aug 2017 19:34:03 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8716AC21C4F; Sun, 13 Aug 2017 19:34:02 +0000 (UTC)","from mout.gmx.net (mout.gmx.net [212.227.15.19])\n\tby lists.denx.de (Postfix) with ESMTPS id 3BE24C21C26\n\tfor ; Sun, 13 Aug 2017 19:34:02 +0000 (UTC)","from [192.168.123.55] ([84.118.154.110]) by mail.gmx.com (mrgmx003\n\t[212.227.17.190]) with ESMTPSA (Nemesis) id\n\t0MLfLH-1dhnrP1VEh-000qfO; Sun, 13 Aug 2017 21:34:00 +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,\n\tRCVD_IN_MSPIKE_H2 autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","To":"Alexander Graf , Heinrich Schuchardt ","References":"<20170805203230.19796-1-xypron.glpk@gmx.de>\n\t<20170805203230.19796-4-xypron.glpk@gmx.de>\n\t\n\t<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>\n\t","From":"Heinrich Schuchardt ","Message-ID":"<0fc16cfb-250f-253d-0564-e34f608324f8@gmx.de>","Date":"Sun, 13 Aug 2017 21:32:36 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-Provags-ID":"V03:K0:VfKoJSiGRlfldHkJiDB0jRd5KdI9UTLGmbW35rBzKVLixckN1GM\n\tHaqPRyHHV5LHac75/5/tBNsSKb0Krl3Hdw2tYH0JOFqC7OSyzRbmobpZsesCQwtoW2as/d0\n\tU8jeDbTHHddbiCVhJx+kA6u7Zgx8UODiQAis5H8f7WESDkMuwLZDdVOaJ07d8rYsD809YGT\n\tjhoSqIcBEBaYnm2jtMU3Q==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:By5vDbNBDbw=:co2uVmBkCUqQLCxbNoP/1x\n\tVECyHOEZE9NKiKoLxPeA9Y/6b0Ho5/oPzMtUMyYg+ktlFssQ6OsNdLUk4/G9fL8NSmd4PGdRq\n\tpArkuEK19u8O6DZLAU9dCj6s5K9fb2KJgGfzkDTeCWNp+Vpfv4YPA8Ubi5KzdiUhTxgeS53Xk\n\tOtrUzypQLw1ELLIdpeV1eDcx41hrIQDY/80qCpCWI3S1lKTwptoYzJ8nWUyfpXjTwEvP6koYU\n\tXZi/vnLgHEhgoNv5cAtLSr3Ieyir1wxHs8r2h6faRtl17lcpKf90iE7mgsJSHpc0eGEzjqV6R\n\t8agJQNXVWn5CVgWLdr+e8ndxZCj/5DlAfDqdpvSwVkLZEn3fyqNAQAXJROL0aGCqHbQYeop9o\n\t6JDV0/uLOl/rVrNcWS0OH4pN+6qiAXVe552WtWvTDTU64CmgIURa0keTEpqrw4P9237C7UH6J\n\tE3yp+OKo90vHfEGyiNRHBjenWnINRjgGK9e+fDXNwRefHX3HlG4dium3nq/tpeuIEE44oEUG7\n\tlmrlalPRjaVlMOnNYkpeg8TVNwP5SkGUQ3KJDwaFj0DbrpgT2IXznZ0B45EQViJ0GDs/rrxmb\n\t9cZowAeCteVxBI58qDchGpQZ8BOSrcEsanFxnsVCM1RvafJOgnjcEPM5PW2JGv/A4KEllLxzq\n\tKbf8KTidM6AmvZFiruCw0B1VmM4VKIB4CGn+S75sPHrtdV1MFeLaBrGsdgjdELh79wt3joNmz\n\tiExjMTxxLTZktCz/GppUPn2PPHf+bcoB19gXAbQZHJj8GIJxM2qWVbYmqvgXeLhpRAqrkiVKj\n\tEpk3DIECAMkhmldcJs0sWeW1c7/0XorlXjE4v5Nz07J54Dofwg=","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","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":1745934,"web_url":"http://patchwork.ozlabs.org/comment/1745934/","msgid":"","list_archive_url":null,"date":"2017-08-13T20:37:25","subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","submitter":{"id":1212,"url":"http://patchwork.ozlabs.org/api/people/1212/","name":"Alexander Graf","email":"agraf@suse.de"},"content":"On 13.08.17 21:32, Heinrich Schuchardt wrote:\n> On 08/13/2017 09:24 PM, Alexander Graf wrote:\n>>\n>>\n>> On 13.08.17 13:17, Heinrich Schuchardt wrote:\n>>> On 08/12/2017 03:38 PM, Alexander Graf wrote:\n>>>>\n>>>>\n>>>> On 05.08.17 22:32, Heinrich Schuchardt wrote:\n>>>>> efi_open_protocol_information provides the agent and controller\n>>>>> handles as well as the attributes and open count of an protocol\n>>>>> on a handle.\n>>>>>\n>>>>> Cc: Rob Clark \n>>>>> Signed-off-by: Heinrich Schuchardt \n>>>>\n>>>> Do you have an application that leverages these interfaces? Would it be\n>>>> possible to add that application to our travis tests?\n>>>>\n>>>>\n>>>> Alex\n>>>>\n>>>\n>>> iPXE (snp.efi) uses the functions but there are other things missing to\n>>> make it really working.\n>>\n>> Ah, I see. How much is missing to make that one work for real?\n> \n> Before reaching the input prompt ConnectController and\n> DisconnectController fail for the Simple Network Protocol.\n> \n> So I am not able to say if the network connection will work.\n> \n> For testing we will need an iSCSI target.\n> \n>>\n>>> Putting new tests into the executable created by\n>>> CMD_BOOTEFI_HELLO_COMPILE would be possible but that seems messy to me.\n>>>\n>>> Should we replace CMD_BOOTEFI_HELLO_COMPILE by an option that creates\n>>> multiple executables for the different EFI API functions we have to test?\n>>>\n>>> Each test then should consist of:\n>>> - start QEMU system\n>>> - download dtb and test executable from tftp server\n>>> - start the test executable\n>>> - evaluate console output\n>>> - shutdown QEMU system\n>>\n>> We have most of the above already in travis today. All we would need is\n>> to extend it to download a built iPXE binaries and add test snippets to\n>> tests/py for iPXE functionality.\n> \n> Proving that some binary runs is not enough to test the different corner\n> cases of this complex API.\n> \n> I would prefer to have a bunch of test binaries compiled from the U-Boot\n> git tree.\n\nSure, even better in my book ;). Ideally we would have both. Unit tests \nto check individual interfaces and integration tests to make sure we \ndon't regress real world use cases.\n\n\nAlex","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 3xVrCV4lbGz9t2Z\n\tfor ;\n\tMon, 14 Aug 2017 06:37:41 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 1DC28C21D7B; Sun, 13 Aug 2017 20:37:31 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 57162C21C59;\n\tSun, 13 Aug 2017 20:37:29 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid ECA40C21C59; Sun, 13 Aug 2017 20:37:27 +0000 (UTC)","from mx1.suse.de (mx2.suse.de [195.135.220.15])\n\tby lists.denx.de (Postfix) with ESMTPS id 8E2E2C21C26\n\tfor ; Sun, 13 Aug 2017 20:37:27 +0000 (UTC)","from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254])\n\tby mx1.suse.de (Postfix) with ESMTP id 44D18ABDB;\n\tSun, 13 Aug 2017 20:37:27 +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":"Heinrich Schuchardt ,\n\tHeinrich Schuchardt ","References":"<20170805203230.19796-1-xypron.glpk@gmx.de>\n\t<20170805203230.19796-4-xypron.glpk@gmx.de>\n\t\n\t<1cc82193-6f27-6438-650c-80b098a322cb@gmx.de>\n\t\n\t<0fc16cfb-250f-253d-0564-e34f608324f8@gmx.de>","From":"Alexander Graf ","Message-ID":"","Date":"Sun, 13 Aug 2017 22:37:25 +0200","User-Agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0)\n\tGecko/20100101 Thunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<0fc16cfb-250f-253d-0564-e34f608324f8@gmx.de>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH v3 3/3] efi_loader: implement\n\tOpenProtocolInformation","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-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]