Hi,

> So from a "will this change stop my production VM from booting" perspective,
> there's no magic number here, though 64G certainly seems like a very 
> reasonable
> cutoff point - beyond having to fix up the VM config, there's no possible way
> that could affect any actual workload in practice.
> 
> Although this actually works at 63G, right, due to the memory layout?

I think it makes sense to check for "RamSizeOver4G > 60GB" then.

On q35 machine type this will actually be 62GB RAM + 2GB mmio window
(below 4G), on pc machine type this will indeed be 63GB RAM + 1GB mmio
window.

> I have a patch series that implements the above if you're interested...

Series?  I'd expect a single one-line patch updating the checks, but
sure, send it along.

take care,
  Gerd

_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-le...@seabios.org

Reply via email to