Hi Masami,
On Thu, 3 Feb 2022 at 02:23, Masami Hiramatsu masami.hiramatsu@linaro.org wrote:
Add a cold reset soon after processing capsule update on disk. This is required in UEFI specification 2.9 Section 8.5.5 "Delivery of Capsules via file on Mass Storage device" as;
In all cases that a capsule is identified for processing the system is restarted after capsule processing is completed.This also reports the result of each capsule update so that the user can notice that the capsule update has been succeeded or not from console log.
Signed-off-by: Masami Hiramatsu masami.hiramatsu@linaro.org
Changes in v4:
- Do not use sysreset because that is a warm reset.
I don't get that.
You should use sysreset and the SYSRESET_COLD type.
- Fix patch description.
lib/efi_loader/efi_capsule.c | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/lib/efi_loader/efi_capsule.c b/lib/efi_loader/efi_capsule.c index 1ec7ea29ff..20d9490dbd 100644 --- a/lib/efi_loader/efi_capsule.c +++ b/lib/efi_loader/efi_capsule.c @@ -14,6 +14,7 @@ #include <env.h> #include <fdtdec.h> #include <fs.h> +#include <hang.h> #include <malloc.h> #include <mapmem.h> #include <sort.h> @@ -1120,8 +1121,11 @@ efi_status_t efi_launch_capsules(void) if (ret == EFI_SUCCESS) { ret = efi_capsule_update_firmware(capsule); if (ret != EFI_SUCCESS)
log_err("Applying capsule %ls failed\n",
log_err("Applying capsule %ls failed.\n", files[i]);elselog_info("Applying capsule %ls succeeded.\n",files[i]); /* create CapsuleXXXX */ set_capsule_result(index, capsule, ret);@@ -1142,6 +1146,16 @@ efi_status_t efi_launch_capsules(void) free(files[i]); free(files);
return ret;
/** UEFI spec requires to reset system after complete processing capsule* update on the storage.*/log_info("Reboot after firmware update");/* Cold reset is required for loading the new firmware. */do_reset(NULL, 0, 0, NULL);
No! This should use sysreset.
hang();/* not reach here */return 0;} #endif /* CONFIG_EFI_CAPSULE_ON_DISK */
Regards, Simon