[coreboot-gerrit] Patch set updated for coreboot: northbridge/amd/amdmct/mct_ddr3: Fix S3 suspend overrunning the stack size limit
Timothy Pearson (tpearson@raptorengineeringinc.com)
gerrit at coreboot.org
Sat Oct 17 22:23:24 CEST 2015
Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11959
-gerrit
commit d441ec18c1c200a5cc6d1f412fd5ed230f66c91c
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date: Sun May 31 18:46:40 2015 -0500
northbridge/amd/amdmct/mct_ddr3: Fix S3 suspend overrunning the stack size limit
Change-Id: Id7441dacef2e46e283d1fc99d5e5fa3f20e0d097
Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
src/northbridge/amd/amdmct/mct_ddr3/s3utils.c | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/s3utils.c b/src/northbridge/amd/amdmct/mct_ddr3/s3utils.c
index 78523e8..1dcbea0 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/s3utils.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/s3utils.c
@@ -543,10 +543,17 @@ int8_t save_mct_information_to_nvram(void)
struct spi_flash *flash;
ssize_t s3nv_offset;
- struct amd_s3_persistent_data persistent_data;
+ struct amd_s3_persistent_data *persistent_data;
+
+ /* Allocate temporary data structures */
+ persistent_data = malloc(sizeof(struct amd_s3_persistent_data));
+ if (!persistent_data) {
+ printk(BIOS_DEBUG, "Could not allocate S3 data structure in RAM\n");
+ return -1;
+ }
/* Obtain MCT configuration data */
- copy_mct_data_to_save_variable(&persistent_data);
+ copy_mct_data_to_save_variable(persistent_data);
/* Obtain CBFS file offset */
s3nv_offset = get_s3nv_file_offset();
@@ -576,7 +583,10 @@ int8_t save_mct_information_to_nvram(void)
/* Erase and write data structure */
flash->erase(flash, s3nv_offset, CONFIG_S3_DATA_SIZE);
- flash->write(flash, s3nv_offset, sizeof(struct amd_s3_persistent_data), &persistent_data);
+ flash->write(flash, s3nv_offset, sizeof(struct amd_s3_persistent_data), persistent_data);
+
+ /* Deallocate temporary data structures */
+ free(persistent_data);
/* Tear down SPI flash access */
flash->spi->rw = SPI_WRITE_FLAG;
More information about the coreboot-gerrit
mailing list