[coreboot-gerrit] Change in coreboot[master]: emulation/riscv: Don't probe for total DRAM

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36445 )

Change subject: emulation/riscv: Don't probe for total DRAM
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36445/1/src/mainboard/emulation/qem... 
File src/mainboard/emulation/qemu-riscv/Kconfig:

https://review.coreboot.org/c/coreboot/+/36445/1/src/mainboard/emulation/qem... 
PS1, Line 58: 	int "size of DRAM. Run qemu with the argument -m ***M"
> Isn't this the main problem? Ensuring the correct arguments are passed to qemu?

Partially, trying to access memory where there is no dram throws in an exception, so probing cannot be done.