[coreboot-gerrit] Change in coreboot[master]: cpu/qemu-x86/cache_as_ram_bootblock: Fix wrong instruction

[Patrick Rudolph (Code Review)]

Hello build bot (Jenkins), Patrick Georgi, Christian Walter, Paul Menzel, Angel Pons, Arthur Heymans,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/45820

to look at the new patch set (#2).

Change subject: cpu/qemu-x86/cache_as_ram_bootblock: Fix wrong instruction
......................................................................

cpu/qemu-x86/cache_as_ram_bootblock: Fix wrong instruction

The shld instruction does an arithmetic shift left on 64bit operants,
but it's not the instruction we want, because what it actually does is
shifting by cl, and storing the result in address 32.

This wasn't noticed with QEMU as the DRAM is up and address 32 is valid.
On real hardware when CAR is running this instruction causes a crash.

Replace the instruction with the correct 64bit arithmetic left shift.

Change-Id: Iedad9f4b693b1ea05898456eac2050a9389f6f19
Signed-off-by: Patrick Rudolph 
---
M src/cpu/qemu-x86/cache_as_ram_bootblock.S
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/45820/2