Re: [U-Boot] [PATCH v4 00/18] warp7: Enable automated OPTEE/HAB boot flow
On 24/04/2018 19:46, Bryan O'Donoghue wrote:
> This tree here is pullable
>
> http://git.linaro.org/landing-teams/working/mbl/u-boot.git/log/?h=linaro-mbl%2bbod-nouart
>
> v4:
>
> - Add Tested-by and Reviewed-by from Fabio and Breno as indicated.
> Thanks very much guys for taking the time to do that :)
>
> - Adds patch tools/imximage: Fix fruity lack of 0x prefix in DCD Blocks
> Previously sent this patch.
> Reviewed-by Fabio added for completeness.
>
> v3:
> - Reword commit message of patch #16 - Breno
>
> - This patchset now relies on five in-flight patch-sets the first four of
> which should be applied first
>
> 1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
>Already has a Reviewed-by from Fabio
>
> 2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
>Has a Reviewed-by: from Breno
>
> 3. [PATCH v3 0/2] WaRP7 unify secure and non-secure defconfigs
>
> 4. Pierre-Jean's generic load patches
>
>[U-Boot] [PATCH v3 1/2] warp7: include/configs: use generic fs commands
>in CONFIG_EXTRA_ENV_SETTINGS
>
>[U-Boot] [PATCH v3 2/2] warp7: configs: enable CONFIG_CMD_FS_GENERIC
>
> 5. [PATCH] bootm: Align cache flush begin address
>This last patch can be applied in any order
>
> v2:
> - Ensure warp7_defconfig boots existing yocto with this change plus the
> automated HAB layer being added here following on from "[PATCH v3 0/2]
> WaRP7 unify secure and non-secure defconfigs"
>
> - Fix reference to partition #1 versus partition #2 in select uuidpart
> patch
>
> - Rebase on top of Pierre-Jean Texier generic load patches
>
> - Drop my patch which did the same thing as Pierre-Jean's patch via
> ${loadcmd}
>
> - Update example boot.scr from v1 to reflect use of generic 'load' command
>
> - This patchset now relies on four in-flight patch-sets which all have the
> relevant Reviewed-by tags from the board Maintainer Fabio.
>
> 1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
>Already has a Reviewed-by from Fabio
>
> 2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
>Has a Reviewed-by: from Breno
>
> 3. [PATCH v3 0/2] WaRP7 unify secure and non-secure defconfigs
>
> 4. Pierre-Jean's generic load patches
>
>[U-Boot] [PATCH v3 1/2] warp7: include/configs: use generic fs commands
>in CONFIG_EXTRA_ENV_SETTINGS
>
>[U-Boot] [PATCH v3 2/2] warp7: configs: enable CONFIG_CMD_FS_GENERIC
>
> v1:
> This series enables an automated HAB verified secure boot which chain-loads
> via OPTEE see `git show 5cf3251..c225e7c` for details.
>
> This set depends on three in-flight patchsets
>
> 1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
>Already has a Reviewed-by from Fabio
>
> 2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
>Has a Reviewed-by: from Breno
>
> 3. [PATCH] configs: warp7: Fix CAAM on boot with tip-of-tree
>
> I'm trying not to make this cover email too long. So - once this set is
> applied it is possible to boot from the BootROM using HAB to verify
>
> - u-boot
> - boot.scr
> - Kernel
> - DTB
>
> Chainload via OPTEE and boot up to Linux. If there is a HAB failure at any
> stage of the process we force-drop down to the USB HID failover mode, from
> which we can send up a recovery image to unblock.
>
> I've run the WaRP7 default u-boot and this new version on NXP's reference
> yocto image and verified that that yocto image boots with both versions of
> the WaRP7 -> warp7_defconfig and warp7_secure_defconfig.
>
> http://freescale.github.io/#download -> BoardsWaRPboard community - WaRP -
> Wearable Reference PlatformFSL Community BSP 2.3fsl-image-multimediawayland
>
> In addition the modifications targeting warp7_secure_defconfig mean it is
> possible to chain-load via OPTEE using scripted HAB to verify images prior
> to exiting the u-boot domain.
>
> Here is an example of the scripting we are doing which shows further reuse
> of shell functions introduced in previous patches.
>
> Example secure-boot boot.scr.imx-signed
>
> # This section is responsbile for loading a signed Linux kernel
> setenv image_signed zImage.imx-signed
> if test ${hab_enabled} -eq 1; then
> setexpr hab_ivt_addr ${loadaddr} - ${ivt_offset}
> load mmc ${mmcdev}:${mmcpart} ${hab_ivt_addr} ${image_signed}
> run warp7_auth_or_fail
> else
> run loadimage;
> fi
>
> # This section is responsbile for loading a signed FDT image
> setenv fdt_file_signed imx7s-warp.dtb.imx-signed
> if test ${hab_enabled} -eq 1; then
> setexpr hab_ivt_addr ${fdt_addr} - ${ivt_offset}
> load mmc ${mmcdev}:${mmcpart} ${hab_ivt_addr}
> ${fdt_file_signed}
> run warp7_auth_or_fail
> else
> run loadfdt;
> fi
>
> # Boot from rootfs1 by default
> setenv mmcpart 3
>
> # But if the rootfs2 file exists in partition 2, boot from rootfs2
> ext4size mmc 0:2 rootfs2 && setenv mmcpart 5
>
> # This section
[U-Boot] [PATCH v4 00/18] warp7: Enable automated OPTEE/HAB boot flow
This tree here is pullable
http://git.linaro.org/landing-teams/working/mbl/u-boot.git/log/?h=linaro-mbl%2bbod-nouart
v4:
- Add Tested-by and Reviewed-by from Fabio and Breno as indicated.
Thanks very much guys for taking the time to do that :)
- Adds patch tools/imximage: Fix fruity lack of 0x prefix in DCD Blocks
Previously sent this patch.
Reviewed-by Fabio added for completeness.
v3:
- Reword commit message of patch #16 - Breno
- This patchset now relies on five in-flight patch-sets the first four of
which should be applied first
1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
Already has a Reviewed-by from Fabio
2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
Has a Reviewed-by: from Breno
3. [PATCH v3 0/2] WaRP7 unify secure and non-secure defconfigs
4. Pierre-Jean's generic load patches
[U-Boot] [PATCH v3 1/2] warp7: include/configs: use generic fs commands
in CONFIG_EXTRA_ENV_SETTINGS
[U-Boot] [PATCH v3 2/2] warp7: configs: enable CONFIG_CMD_FS_GENERIC
5. [PATCH] bootm: Align cache flush begin address
This last patch can be applied in any order
v2:
- Ensure warp7_defconfig boots existing yocto with this change plus the
automated HAB layer being added here following on from "[PATCH v3 0/2]
WaRP7 unify secure and non-secure defconfigs"
- Fix reference to partition #1 versus partition #2 in select uuidpart
patch
- Rebase on top of Pierre-Jean Texier generic load patches
- Drop my patch which did the same thing as Pierre-Jean's patch via
${loadcmd}
- Update example boot.scr from v1 to reflect use of generic 'load' command
- This patchset now relies on four in-flight patch-sets which all have the
relevant Reviewed-by tags from the board Maintainer Fabio.
1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
Already has a Reviewed-by from Fabio
2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
Has a Reviewed-by: from Breno
3. [PATCH v3 0/2] WaRP7 unify secure and non-secure defconfigs
4. Pierre-Jean's generic load patches
[U-Boot] [PATCH v3 1/2] warp7: include/configs: use generic fs commands
in CONFIG_EXTRA_ENV_SETTINGS
[U-Boot] [PATCH v3 2/2] warp7: configs: enable CONFIG_CMD_FS_GENERIC
v1:
This series enables an automated HAB verified secure boot which chain-loads
via OPTEE see `git show 5cf3251..c225e7c` for details.
This set depends on three in-flight patchsets
1. [PATCH v3 0/3] NXP WaARP7 set serial# from OTP fuses for USB iSerial
Already has a Reviewed-by from Fabio
2. [PATCH v3 0/2] imx: hab: Add helper functions for scripted HAB auth
Has a Reviewed-by: from Breno
3. [PATCH] configs: warp7: Fix CAAM on boot with tip-of-tree
I'm trying not to make this cover email too long. So - once this set is
applied it is possible to boot from the BootROM using HAB to verify
- u-boot
- boot.scr
- Kernel
- DTB
Chainload via OPTEE and boot up to Linux. If there is a HAB failure at any
stage of the process we force-drop down to the USB HID failover mode, from
which we can send up a recovery image to unblock.
I've run the WaRP7 default u-boot and this new version on NXP's reference
yocto image and verified that that yocto image boots with both versions of
the WaRP7 -> warp7_defconfig and warp7_secure_defconfig.
http://freescale.github.io/#download -> BoardsWaRPboard community - WaRP -
Wearable Reference PlatformFSL Community BSP 2.3fsl-image-multimediawayland
In addition the modifications targeting warp7_secure_defconfig mean it is
possible to chain-load via OPTEE using scripted HAB to verify images prior
to exiting the u-boot domain.
Here is an example of the scripting we are doing which shows further reuse
of shell functions introduced in previous patches.
Example secure-boot boot.scr.imx-signed
# This section is responsbile for loading a signed Linux kernel
setenv image_signed zImage.imx-signed
if test ${hab_enabled} -eq 1; then
setexpr hab_ivt_addr ${loadaddr} - ${ivt_offset}
load mmc ${mmcdev}:${mmcpart} ${hab_ivt_addr} ${image_signed}
run warp7_auth_or_fail
else
run loadimage;
fi
# This section is responsbile for loading a signed FDT image
setenv fdt_file_signed imx7s-warp.dtb.imx-signed
if test ${hab_enabled} -eq 1; then
setexpr hab_ivt_addr ${fdt_addr} - ${ivt_offset}
load mmc ${mmcdev}:${mmcpart} ${hab_ivt_addr}
${fdt_file_signed}
run warp7_auth_or_fail
else
run loadfdt;
fi
# Boot from rootfs1 by default
setenv mmcpart 3
# But if the rootfs2 file exists in partition 2, boot from rootfs2
ext4size mmc 0:2 rootfs2 && setenv mmcpart 5
# This section is responsbile for loading a signed OPTEE image
setenv optee_file /lib/firmware/uTee.optee
setenv optee_file_signed /lib/firmware/uTee.optee.imx-signed
setenv loadoptee "load mmc ${mmcdev}:${mmcpart} ${optee_addr}
${optee_file}"
if test ${hab_enabled} -eq 1; then
setexpr hab_ivt_addr ${optee_addr} -
