W dniu 4.12.2024 o 21:50, Philippe Mathieu-Daudé pisze:
On 4/12/24 13:17, Leif Lindholm wrote:
On 2024-12-02 10:53, Marcin Juszkiewicz wrote:
W dniu 26.11.2024 o 14:14, Peter Maydell pisze:
On Tue, 26 Nov 2024 at 08:49, Marcin Juszkiewicz
<marcin.juszkiew...@linaro.org> wrote:
We are working on adding RME support to SBSA Reference Platform.
When RME is enabled then RMM (Realm Managment Monitor) takes 1072MB of
memory for own use. Which ends with firmware panic on 1GB machine.
Reasonable change, but isn't it also a bug in the RMM that it
grabs 1GB of RAM regardless of how much RAM the machine
actually has?
I think that the goal is "get it working" first and then optimize.
I agree on a different platform this could feel quite hacky, but in
reality even 2GB falls within "ridiculously low for an SBSA platform".
If we're worried about overhead for CI jobs that do not require the
feature, we could always conditionalize it on RME being enabled. But
I'd be happy to wait and see.
I'd rather do that, since it is as simple as:
This is sbsa-ref not sbc-ref. Let it just have that 2GB of ram. None of
existing SBSA systems comes with such low amount.
If there is a use which is memory limited then it can set other values
with "-m" argument as always.
That's my opinion. I am fine with whatever option gets merged.
