Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63491 )
Change subject: cpu/x86/mtrr: Use `need_restore_mtrr` to set put_back_original_solution ......................................................................
cpu/x86/mtrr: Use `need_restore_mtrr` to set put_back_original_solution
This patch calls into need_restore_mtrr() from the mtrr_use_temp_range function to set `put_back_original_solution` to discard any temporary MTRR range prior to boot to payload.
BUG=b:225766934 TEST=Able to build and boot google/brya to verify that `remove_temp_solution()` is able to discard any temporary MTRR range before booting to payload.
Signed-off-by: Subrata Banik subratabanik@google.com Change-Id: I2e00ec593847e1eb173d5ac77b15b50342860f89 --- M src/cpu/x86/mtrr/mtrr.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/63491/1
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c index e1bdf45..288f06d 100644 --- a/src/cpu/x86/mtrr/mtrr.c +++ b/src/cpu/x86/mtrr/mtrr.c @@ -908,7 +908,7 @@ (long long)begin, (long long)begin + size, (long long)size, type); else - put_back_original_solution = true; + need_restore_mtrr();
memranges_teardown(&addr_space); }