[U-Boot] [PATCH v4 2/2] efi_selftest: provide a test for block io
Simon Glass
sjg at chromium.org
Sun Feb 4 12:39:27 UTC 2018
On 21 January 2018 at 11:29, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> This test checks the driver for block IO devices.
> A disk image is created in memory.
> A handle is created for the new block IO device.
> The block I/O protocol is installed on the handle.
> ConnectController is used to setup partitions and to install the simple
> file protocol.
> A known file is read from the file system and verified.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> v4
> Provide a commit message.
> v3
> no change
> v2
> no change
> ---
> lib/efi_selftest/Makefile | 3 +
> lib/efi_selftest/efi_selftest_block_device.c | 395 +++++++++++++++++++++++++++
> lib/efi_selftest/efi_selftest_disk_image.h | 69 +++++
> 3 files changed, 467 insertions(+)
> create mode 100644 lib/efi_selftest/efi_selftest_block_device.c
> create mode 100644 lib/efi_selftest/efi_selftest_disk_image.h
Reviewed-by: Simon Glass <sjg at chromium.org>
A few general comments:
- I'm not very keen on using 'this' in the code. This is not C++. Can
you use blkio or something, instead?
- How about declaring all the static vars at the top of the file?
Regards,
Simon
More information about the U-Boot
mailing list