From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Sun, 27 Aug 2017 14:10:36 -0600 Subject: [U-Boot] [PATCH 00/23] efi_loader implement missing functions In-Reply-To: <20170826225110.7381-1-xypron.glpk@gmx.de> References: <20170826225110.7381-1-xypron.glpk@gmx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Heinrich, On 26 August 2017 at 16:51, Heinrich Schuchardt wrote: > This patch sequence contains all patches needed to load > iPXE and use it for downloading and executing images > via https or http or to mount iSCSI volumes. > > Network speed on an Odroid C2 reached 30 MB/s which should be > enough for most use cases. > > I have tested the following iPXE commands successfully > * dhcp > * route > * ntp > * sanhook iSCSI-target > * chain http-target > * kernel http-target > * boot (after calling kernel) > * exit > * reboot > > The only adjustment in iPXE was adding file src/config/local/nap.h with > #undef NAP_EFIX86 > #undef NAP_EFIARM > #define NAP_NULL > and src/config/local/myscript.ipxe with > #!ipxe > shell > before building iPXE with > make bin-arm64-efi/snp.efi EMBED=config/local/myscript.ipxe > > The next task will be to put iXPE binaries on a server > and to create Travis CI test cases. Some general comments on the series as a whole: 1. It really needs to have tests. This is a lot of new code in U-Boot, and relying on travis CI test cases (which takes forever to run) is not a good option. We have a 'make tests' target which you should hook into, via the pytests framework. This runs in a minute or so. There is quite a bit of documentation in test/py for this. It should be easy enough to build up the data structures in sandbox and then test that each function does what is expected. 2. Exported functions should be commented to describe their purpose, arguments and return value. Non-trivial static functions should be commented too. Regards, Simon