On Sun, Sep 09, 2018 at 02:13:31PM +1000, Damien Zammit wrote:
Hi folks,
I want to put forth this patch for your review.
In my opinion, it is very useful to know just after your machine has POSTed how much system RAM the machine has access to.
This patch prints out to the screen just under the UUID the following line: System RAM: X MB
where X is the number of whole megabytes available to the system.
Since the memory map is already available to SeaBIOS, I think it makes sense to total up the sizes of regions marked as E820_RAM and print the grand total.
I have tested this on a coreboot QEMU bios with custom SeaBIOS built:
SeaBIOS (version rel-1.11.0-46-g18e193d) System RAM: 1022 MB
Thanks. However, in my view, the goal of SeaBIOS is to boot an operating system - not to display system information. There's a ton of information that could be provided (cpus, cpu types, cpu speeds, disks, disk types, disk sizes, etc.), and there are plenty of applications available which will display that system information. I fear trying to provide system information in SeaBIOS would become a maintenance burden and clutter the boot process.
-Kevin