Re: Fit images and EFI_LOAD_FILE2_PROTOCOL

2020-10-04 Thread Cristian Ciocaltea
Hello Heinrich, On Sat, Oct 03, 2020 at 10:51:24AM +0200, Heinrich Schuchardt wrote: > Hello Ilias, hello Christian, > > with commit ec80b4735a59 ("efi_loader: Implement FileLoad2 for initramfs > loading") Ilias provided the possibility to specify a device path > (CONFIG_EFI_INITRD_FILESPEC) from

Re: [PATCH v2 1/1] travis-ci: Fix "ResourceWarning: unclosed file"

2020-06-24 Thread Cristian Ciocaltea
On Wed, Jun 24, 2020 at 02:30:24PM -0600, Stephen Warren wrote: > On 12/30/19 9:41 AM, Stephen Warren wrote: > > On 12/30/19 4:49 AM, Cristian Ciocaltea wrote: > >> This patch gets rid of the warning messages like: > >> > >> uboot-test-hooks/py/travis-c

Re: [PATCH v2 1/1] travis-ci: provide env__efi_fit_tftp_file

2019-12-31 Thread Cristian Ciocaltea
On Tue, Dec 31, 2019 at 12:42:36PM +0200, Cristian Ciocaltea wrote: > On Mon, Dec 30, 2019 at 09:03:38PM +0100, Heinrich Schuchardt wrote: > > On 12/30/19 8:32 PM, Stephen Warren wrote: > > > On 12/30/19 12:05 PM, Heinrich Schuchardt wrote: > > > > On 12/30/19

Re: [PATCH v2 1/1] travis-ci: provide env__efi_fit_tftp_file

2019-12-31 Thread Cristian Ciocaltea
On Mon, Dec 30, 2019 at 09:03:38PM +0100, Heinrich Schuchardt wrote: > On 12/30/19 8:32 PM, Stephen Warren wrote: > > On 12/30/19 12:05 PM, Heinrich Schuchardt wrote: > > > On 12/30/19 5:38 PM, Stephen Warren wrote: > > > > On 12/30/19 3:52 AM, Heinrich Schuchardt wrote: > > > > > Provide dictionar

[PATCH v2 1/1] travis-ci: Fix "ResourceWarning: unclosed file"

2019-12-30 Thread Cristian Ciocaltea
This patch gets rid of the warning messages like: uboot-test-hooks/py/travis-ci/travis_tftp.py:43: ResourceWarning: unclosed file <_io.BufferedReader name='.bm-work/qemu_arm/u-boot.bin'> Signed-off-by: Cristian Ciocaltea --- Changes in v2: * The warning line number in the c

Re: [PATCH 1/1] travis-ci: provide env__efi_fit_tftp_file

2019-12-30 Thread Cristian Ciocaltea
On Sun, Dec 29, 2019 at 06:46:40PM +0100, Heinrich Schuchardt wrote: > Provide dictionary env__efi_fit_tftp_file describing the file used for the > UEFI FIT image test. > > Cf. "test/py: Create a test for launching UEFI binaries from FIT images" > https://lists.denx.de/pipermail/u-boot/2019-Decemb

[PATCH v5 1/5] image: Add IH_OS_EFI for EFI chain-load boot

2019-12-29 Thread Cristian Ciocaltea
config-grub { kernel = "efi-grub"; signature-1 { algo = "sha256,rsa2048"; sign-images = "kernel"; }; }; }; }; Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt ---

[PATCH v5 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-29 Thread Cristian Ciocaltea
This test verifies the implementation of the 'bootm' extension that handles UEFI binaries inside FIT images (enabled via CONFIG_BOOTM_EFI). Signed-off-by: Cristian Ciocaltea --- test/py/tests/test_efi_fit.py | 458 ++ 1 file changed, 458 insertions(

[PATCH v5 4/5] doc: uefi.rst: Document launching UEFI binaries from FIT images

2019-12-29 Thread Cristian Ciocaltea
This patch adds a new section "Launching a UEFI binary from a FIT image" documenting the usage of the CONFIG_BOOTM_EFI extension to bootm command that offers a verified boot alternative for UEFI binaries such as GRUB2. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt

[PATCH v5 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-29 Thread Cristian Ciocaltea
Add support for booting EFI binaries contained in FIT images. A typical usage scenario is chain-loading GRUB2 in a verified boot environment. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt --- cmd/Kconfig | 7 ++ common/bootm_os.c | 56

[PATCH v5 0/5] Add support for booting EFI FIT images

2019-12-29 Thread Cristian Ciocaltea
the implementation in sandbox environment Changes in v2: * Rebase patches on Heinrich Schuchardt's patch series: efi_loader: prepare for FIT images https://lists.denx.de/pipermail/u-boot/2019-December/393192.html * Add sample configuration: doc/uImage.FIT/uefi.its * Update uefi docu

[PATCH v5 3/5] doc: Add sample uefi.its image description file

2019-12-29 Thread Cristian Ciocaltea
This patch adds an example FIT image description file demonstrating the usage of bootm command to securely launch UEFI binaries. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt --- doc/uImage.FIT/uefi.its | 67 + 1 file changed, 67

[PATCH v1 1/1] travis-ci: Fix "ResourceWarning: unclosed file"

2019-12-29 Thread Cristian Ciocaltea
This patch gets rid of the warning messages like: uboot-test-hooks/py/travis-ci/travis_tftp.py:43: ResourceWarning: unclosed file <_io.BufferedReader name='.bm-work/qemu_arm/u-boot.bin'> Signed-off-by: Cristian Ciocaltea --- py/travis-ci/travis_tftp.py | 5 - 1 file chang

Re: [PATCH v4 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-29 Thread Cristian Ciocaltea
On Sun, Dec 29, 2019 at 08:11:10PM +0100, Heinrich Schuchardt wrote: > On 12/29/19 7:39 PM, Cristian Ciocaltea wrote: > > On Sun, Dec 29, 2019 at 11:22:08AM +0100, Heinrich Schuchardt wrote: > > > On 12/24/19 5:05 PM, Cristian Ciocaltea wrote: > > > > This test veri

Re: [PATCH v4 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-29 Thread Cristian Ciocaltea
On Sun, Dec 29, 2019 at 11:22:08AM +0100, Heinrich Schuchardt wrote: > On 12/24/19 5:05 PM, Cristian Ciocaltea wrote: > > This test verifies the implementation of the 'bootm' extension that > > handles UEFI binaries inside FIT images (enabled via CONFIG_BOOTM_EFI). > &g

Re: [PATCH v4 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-29 Thread Cristian Ciocaltea
On Sun, Dec 29, 2019 at 11:56:01AM +0100, Heinrich Schuchardt wrote: > On 12/29/19 11:34 AM, Heinrich Schuchardt wrote: > > On 12/24/19 5:05 PM, Cristian Ciocaltea wrote: > > > Add support for booting EFI binaries contained in FIT images. > > > A typical usage scenario

Re: [PATCH v4 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-29 Thread Cristian Ciocaltea
On Sun, Dec 29, 2019 at 11:34:09AM +0100, Heinrich Schuchardt wrote: > On 12/24/19 5:05 PM, Cristian Ciocaltea wrote: > > Add support for booting EFI binaries contained in FIT images. > > A typical usage scenario is chain-loading GRUB2 in a verified > > boot environment. &

[PATCH v4 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-24 Thread Cristian Ciocaltea
This test verifies the implementation of the 'bootm' extension that handles UEFI binaries inside FIT images (enabled via CONFIG_BOOTM_EFI). Signed-off-by: Cristian Ciocaltea --- test/py/tests/test_efi_fit.py | 459 ++ 1 file changed, 459 insertions(

[PATCH v4 4/5] doc: uefi.rst: Document launching UEFI binaries from FIT images

2019-12-24 Thread Cristian Ciocaltea
This patch adds a new section "Launching a UEFI binary from a FIT image" documenting the usage of the CONFIG_BOOTM_EFI extension to bootm command that offers a verified boot alternative for UEFI binaries such as GRUB2. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt

[PATCH v4 3/5] doc: Add sample uefi.its image description file

2019-12-24 Thread Cristian Ciocaltea
This patch adds an example FIT image description file demonstrating the usage of bootm command to securely launch UEFI binaries. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt --- doc/uImage.FIT/uefi.its | 67 + 1 file changed, 67

[PATCH v4 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-24 Thread Cristian Ciocaltea
Add support for booting EFI binaries contained in FIT images. A typical usage scenario is chain-loading GRUB2 in a verified boot environment. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt --- cmd/Kconfig | 7 ++ common/bootm_os.c | 56

[PATCH v4 1/5] image: Add IH_OS_EFI for EFI chain-load boot

2019-12-24 Thread Cristian Ciocaltea
config-grub { kernel = "efi-grub"; signature-1 { algo = "sha256,rsa2048"; sign-images = "kernel"; }; }; }; }; Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt ---

[PATCH v4 0/5] Add support for booting EFI FIT images

2019-12-24 Thread Cristian Ciocaltea
n sandbox environment Changes in v2: * Rebase patches on Heinrich Schuchardt's patch series: efi_loader: prepare for FIT images https://lists.denx.de/pipermail/u-boot/2019-December/393192.html * Add sample configuration: doc/uImage.FIT/uefi.its * Update uefi documentation: doc/uefi/

[PATCH 1/1] test/py: Fix broken 'notbuildconfigspec' marker

2019-12-24 Thread Cristian Ciocaltea
('/uboot/test/py/conftest.py', 463, 'Skipped: .config feature "fit" enabled') Signed-off-by: Cristian Ciocaltea --- test/py/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/py/conftest.py b/test/py/conftest.py index bffee6b8a

Re: [PATCH v3 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-18 Thread Cristian Ciocaltea
On Wed, Dec 18, 2019 at 11:56:00AM +0100, Heinrich Schuchardt wrote: > On 12/18/19 9:22 AM, Cristian Ciocaltea wrote: > > > > +}; > > > > +}; > > > > +''' > > > > + > > > > +@pytest.mark.boardspec('sa

Re: [PATCH v3 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-18 Thread Cristian Ciocaltea
On Wed, Dec 18, 2019 at 11:06:48AM +0100, Heinrich Schuchardt wrote: > On 12/18/19 9:22 AM, Cristian Ciocaltea wrote: > > On Tue, Dec 17, 2019 at 10:08:31PM +0100, Heinrich Schuchardt wrote: > > > On 12/17/19 8:47 AM, Cristian Ciocaltea wrote: > > > > This test veri

Re: [PATCH v3 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-18 Thread Cristian Ciocaltea
On Tue, Dec 17, 2019 at 09:34:44PM +0100, Heinrich Schuchardt wrote: > On 12/17/19 8:46 AM, Cristian Ciocaltea wrote: > > Add support for booting EFI binaries contained in FIT images. > > A typical usage scenario is chain-loading GRUB2 in a verified > > boot environment. &

Re: [PATCH v3 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-18 Thread Cristian Ciocaltea
On Tue, Dec 17, 2019 at 10:08:31PM +0100, Heinrich Schuchardt wrote: > On 12/17/19 8:47 AM, Cristian Ciocaltea wrote: > > This test verifies the implementation of the 'bootm' extension that > > handles UEFI binaries inside FIT images (enabled via CONFIG_BOOTM_EFI). > &g

[PATCH v3 5/5] test/py: Create a test for launching UEFI binaries from FIT images

2019-12-16 Thread Cristian Ciocaltea
This test verifies the implementation of the 'bootm' extension that handles UEFI binaries inside FIT images (enabled via CONFIG_BOOTM_EFI). Signed-off-by: Cristian Ciocaltea --- test/py/tests/test_efi_fit.py | 233 ++ 1 file changed, 233 insertions(

[PATCH v3 4/5] doc: uefi.rst: Document launching UEFI binaries from FIT images

2019-12-16 Thread Cristian Ciocaltea
This patch adds a new section "Launching a UEFI binary from a FIT image" documenting the usage of the CONFIG_BOOTM_EFI extension to bootm command that offers a verified boot alternative for UEFI binaries such as GRUB2. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt

[PATCH v3 0/5] Add support for booting EFI FIT images

2019-12-16 Thread Cristian Ciocaltea
efi_loader: prepare for FIT images https://lists.denx.de/pipermail/u-boot/2019-December/393192.html * Add sample configuration: doc/uImage.FIT/uefi.its * Update uefi documentation: doc/uefi/uefi.rst Cristian Ciocaltea (5): image: Add IH_OS_EFI for EFI chain-load boot bootm: Add a bootm command

[PATCH v3 1/5] image: Add IH_OS_EFI for EFI chain-load boot

2019-12-16 Thread Cristian Ciocaltea
config-grub { kernel = "efi-grub"; signature-1 { algo = "sha256,rsa2048"; sign-images = "kernel"; }; }; }; }; Signed-off-by: Cristian Ciocaltea --- common/image-fit.c | 3 ++- commo

[PATCH v3 3/5] doc: Add sample uefi.its image description file

2019-12-16 Thread Cristian Ciocaltea
This patch adds an example FIT image description file demonstrating the usage of bootm command to securely launch UEFI binaries. Signed-off-by: Cristian Ciocaltea Reviewed-by: Heinrich Schuchardt --- doc/uImage.FIT/uefi.its | 67 + 1 file changed, 67

[PATCH v3 2/5] bootm: Add a bootm command for type IH_OS_EFI

2019-12-16 Thread Cristian Ciocaltea
Add support for booting EFI binaries contained in FIT images. A typical usage scenario is chain-loading GRUB2 in a verified boot environment. Signed-off-by: Cristian Ciocaltea --- cmd/Kconfig | 7 ++ common/bootm_os.c | 56 +++ 2 files

Re: [PATCH v2 2/4] bootm: Add a bootm command for type IH_OS_EFI

2019-12-11 Thread Cristian Ciocaltea
On Wed, Dec 11, 2019 at 10:57:48AM +0100, Heinrich Schuchardt wrote: > On 12/11/19 9:54 AM, Cristian Ciocaltea wrote: > > On Tue, Dec 10, 2019 at 08:32:17PM +0100, Heinrich Schuchardt wrote: > > > On 12/10/19 9:56 AM, Cristian Ciocaltea wrote: > > > > Add support for

Re: [PATCH v2 2/4] bootm: Add a bootm command for type IH_OS_EFI

2019-12-11 Thread Cristian Ciocaltea
On Wed, Dec 11, 2019 at 11:13:28AM +0100, Heinrich Schuchardt wrote: > On 12/11/19 9:54 AM, Cristian Ciocaltea wrote: > > 1. Create a public/private key pair > > $ openssl genpkey -algorithm RSA -out ${DEV_KEY} \ > > -pkeyopt rsa_keygen_bits:2048 -pkeyopt rs

Re: [PATCH v2 1/4] image: Add IH_OS_EFI for EFI chain-load boot

2019-12-11 Thread Cristian Ciocaltea
On Tue, Dec 10, 2019 at 10:49:09PM +, Peter Robinson wrote: > On Tue, Dec 10, 2019 at 6:30 PM Heinrich Schuchardt > wrote: > > > > On 12/10/19 9:56 AM, Cristian Ciocaltea wrote: > > > Add a new OS type to be used for chain-loading an EFI compatible > > >

Re: [PATCH v2 2/4] bootm: Add a bootm command for type IH_OS_EFI

2019-12-11 Thread Cristian Ciocaltea
On Tue, Dec 10, 2019 at 08:32:17PM +0100, Heinrich Schuchardt wrote: > On 12/10/19 9:56 AM, Cristian Ciocaltea wrote: > > Add support for booting EFI binaries contained in FIT images. > > A typical usage scenario is chain-loading GRUB2 in a verified > > boot environment. &

[PATCH v2 4/4] doc: uefi.rst: Document launching UEFI binaries from FIT images

2019-12-10 Thread Cristian Ciocaltea
This patch adds a new section "Launching a UEFI binary from a FIT image" documenting the usage of the CONFIG_BOOTM_EFI extension to bootm command that offers a secure boot alternative for UEFI binaries such as GRUB2. Signed-off-by: Cristian Ciocaltea --- doc/uefi/uef

[PATCH v2 3/4] doc: Add sample uefi.its image description file

2019-12-10 Thread Cristian Ciocaltea
This patch adds an example FIT image description file demonstrating the usage of bootm command to securely launch UEFI binaries. Signed-off-by: Cristian Ciocaltea --- doc/uImage.FIT/uefi.its | 67 + 1 file changed, 67 insertions(+) create mode 100644 doc

[PATCH v2 2/4] bootm: Add a bootm command for type IH_OS_EFI

2019-12-10 Thread Cristian Ciocaltea
Add support for booting EFI binaries contained in FIT images. A typical usage scenario is chain-loading GRUB2 in a verified boot environment. Signed-off-by: Cristian Ciocaltea --- cmd/Kconfig | 7 ++ common/bootm_os.c | 61 +++ 2 files

[PATCH v2 0/4] Add support for booting EFI FIT images

2019-12-10 Thread Cristian Ciocaltea
grub Changes since v1: * Rebase patches on Heinrich Schuchardt's patch series: efi_loader: prepare for FIT images https://lists.denx.de/pipermail/u-boot/2019-December/393192.html * Add sample configuration: doc/uImage.FIT/uefi.its * Update uefi documentation: doc/uefi/uefi.rst Crist

[PATCH v2 1/4] image: Add IH_OS_EFI for EFI chain-load boot

2019-12-10 Thread Cristian Ciocaltea
quot;; config-grub { kernel = "efi-grub"; signature-1 { algo = "sha256,rsa2048"; sign-images = "kernel"; }; }; }; }; Signed-off-by: Cristian Ciocaltea --- common/image-fit.c |

Re: [U-Boot] [PATCH 0/2] Add support for booting EFI FIT images

2019-12-09 Thread Cristian Ciocaltea
On Sun, Dec 08, 2019 at 01:25:27AM +0100, Heinrich Schuchardt wrote: > On 11/28/19 8:20 AM, Heinrich Schuchardt wrote: > > On 11/27/19 8:45 PM, Cristian Ciocaltea wrote: > > > On Tue, Nov 26, 2019 at 07:31:39PM +0100, Heinrich Schuchardt wrote: > > > > On 11/24/19 9:1

Re: [U-Boot] [PATCH 0/2] Add support for booting EFI FIT images

2019-11-27 Thread Cristian Ciocaltea
On Tue, Nov 26, 2019 at 07:31:39PM +0100, Heinrich Schuchardt wrote: > On 11/24/19 9:11 PM, Cristian Ciocaltea wrote: > > Currently the only way to run an EFI binary like GRUB2 is via the > > 'bootefi' command, which cannot be used in a verified boot scenario. > > &

[U-Boot] [PATCH 1/1] buildman: Improve [make-flags] section parser to allow quoted strings

2019-11-24 Thread Cristian Ciocaltea
atch replaces the simple string splitter based on the delimiter with a regex tokenizer that preserves spaces inside double quoted strings. Signed-off-by: Cristian Ciocaltea --- tools/buildman/toolchain.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/buildman/toolchai

[U-Boot] [PATCH 1/2] image: Add IH_OS_EFI for EFI chain-load boot

2019-11-24 Thread Cristian Ciocaltea
quot;; config-grub { kernel = "efi-grub"; signature-1 { algo = "sha256,rsa2048"; sign-images = "kernel"; }; }; }; }; Signed-off-by: Cristian Ciocaltea --- common/image-fit.c |

[U-Boot] [PATCH 0/2] Add support for booting EFI FIT images

2019-11-24 Thread Cristian Ciocaltea
}; The bootm command has been extended to handle the IH_OS_EFI images. To enable this feature, a new configuration option has been added: BOOTM_EFI I tested the solution using the 'qemu_arm' board: => load scsi 0:1 ${kernel_addr_r} efi-image.fit => bootm ${kernel_addr_r}#config-gr

[U-Boot] [PATCH 2/2] bootm: Add a bootm command for type IH_OS_EFI

2019-11-24 Thread Cristian Ciocaltea
Add support for booting EFI binaries contained in FIT images. A typical usage scenario is chain-loading GRUB2 in a verified boot environment. Signed-off-by: Cristian Ciocaltea --- cmd/Kconfig | 9 - cmd/bootefi.c | 2 +- common/bootm_os.c | 44

[U-Boot] [PATCH] api: storage: Add the missing write operation support

2019-01-11 Thread Cristian Ciocaltea
triggered with GRUB's save_env command. Signed-off-by: Cristian Ciocaltea --- api/api.c | 59 ++- api/api_private.h | 1 + api/api_storage.c | 24 +++ 3 files changed, 63 insertions(+), 21 deletions(-) diff --git a/api/ap

[U-Boot] [PATCH] api: storage: Fix enumeration of storage devices

2018-12-18 Thread Cristian Ciocaltea
so that the following search for the next available device will start from beginning. The issue has been identified by loading and booting GRUB in a QEMU vexpress-a9 environment. Signed-off-by: Cristian Ciocaltea --- api/api_storage.c | 5 + 1 file changed, 5 insertions(+) diff --git a