[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: If psp_verstage is in RO, don't reset on error

[Martin Roth (Code Review)]

Martin Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44652 )

Change subject: soc/amd/picasso: If psp_verstage is in RO, don't reset on error
......................................................................

soc/amd/picasso: If psp_verstage is in RO, don't reset on error

If there's already been an error and PSP_verstage is booting to RO,
don't reset the system.  It may be that the error is fatal, but if the
system is stuck, don't intentionally force it into a reboot loop.

BUG=None
TEST=Force an error, still boots to RO instead of going into a boot loop

Signed-off-by: Martin Roth 
Change-Id: Ibb6794fefe9d482850ca31b1d3b0d145fcd8bb8f
---
M src/soc/amd/picasso/psp_verstage/psp_verstage.c
1 file changed, 10 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/52/44652/1

diff --git a/src/soc/amd/picasso/psp_verstage/psp_verstage.c b/src/soc/amd/picasso/psp_verstage/psp_verstage.c
index 85fd74b..be515b1 100644
--- a/src/soc/amd/picasso/psp_verstage/psp_verstage.c
+++ b/src/soc/amd/picasso/psp_verstage/psp_verstage.c
@@ -29,6 +29,12 @@
    subcode += PSP_VBOOT_ERROR_SUBCODE;
    svc_write_postcode(subcode);

+	/* Continue booting from RO */
+	if (ctx->flags & VB2_CONTEXT_RECOVERY_MODE) {
+		printk(BIOS_ERR, "Already in recovery mode. Staying in RO.\n");
+		return;
+	}
+
    vb2api_fail(ctx, VB2_RECOVERY_RO_UNSPECIFIED, (int)subcode);
    vboot_save_data(ctx);

@@ -231,21 +237,20 @@

verstage_main();

-	vb2api_relocate(_vboot2_work, _vboot2_work, VB2_FIRMWARE_WORKBUF_RECOMMENDED_SIZE,
-			&ctx);
+	ctx = vboot_get_context();
    retval = check_cmos_recovery();
    if (retval)
-		goto err;
+		reboot_into_recovery(ctx, retval);

post_code(POSTCODE_SAVE_BUFFERS);
    retval = save_buffers(&ctx);
    if (retval)
-		goto err;
+		reboot_into_recovery(ctx, retval);

post_code(POSTCODE_UPDATE_BOOT_REGION);
    retval = update_boot_region(ctx);
    if (retval)
-		goto err;
+		reboot_into_recovery(ctx, retval);

post_code(POSTCODE_UNMAP_SPI_ROM);
    if (boot_dev.base) {
@@ -260,9 +265,6 @@

printk(BIOS_DEBUG, "Leaving verstage on PSP\n");
    svc_exit(retval);
-
-err:
-	reboot_into_recovery(ctx, retval);
 }

const struct region_device *boot_device_ro(void)