[coreboot-gerrit] New patch to review for coreboot: 1809bdf Do CAR variable migration only once

[Kyösti Mälkki]

Kyösti Mälkki ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3512

-gerrit

commit 1809bdfaa4bf6be0b0729b02f28445c16a06b28a
Author: Aaron Durbin 
Date:   Fri Jun 21 18:02:26 2013 +0300

Do CAR variable migration only once

Non-S3 resume paths of sandy/ivybridge call cbmem_initialize()
    more than once. Doing car_migrate_variables() more than twice caused
    at least loss of some lines in CBMEM console.

Change-Id: Idd14aba9384984aa3a7d38937a4b3572aa5dc088
    Signed-off-by: Kyösti Mälkki 
---
 src/cpu/x86/car.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/cpu/x86/car.c b/src/cpu/x86/car.c
index 31fc67c..88f2807 100644
--- a/src/cpu/x86/car.c
+++ b/src/cpu/x86/car.c
@@ -80,6 +80,10 @@ void car_migrate_variables(void)
    car_migration_func_t *migrate_func;
    size_t car_data_size = &_car_data_end[0] - &_car_data_start[0];

+	/* Check if already migrated. */
+	if (car_migrated)
+		return;
+
    migrated_base = cbmem_add(CBMEM_ID_CAR_GLOBALS, car_data_size);

if (migrated_base == NULL) {