Re: [PATCH 1/1] hw/arm/sbsa-ref: bump default memory size to 2GB

[Peter Maydell]

On Thu, 5 Dec 2024 at 15:10, Marcin Juszkiewicz
<marcin.juszkiewicz@linaro.org> wrote:
>
> W dniu 5.12.2024 o 15:53, Peter Maydell pisze:
> >> 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.
>
> > Yes, I think I agree here. Changing the default RAM size based on
> > whether the CPU does or does not have a particular feature is
> > unusual and not something we do on other board types, and it
> > definitely doesn't match the general intention that sbsa-ref is a
> > "looks like real hardware" machine.
> >
> > The problem with using 2GB, though, is that it doesn't work on 32-
> > bit hosts, which have a max of 2047MB (and
>
> Do we do CI on 32-bit hosts?

We do at least some. In particular the qom-test tests run
by 'make check' include "start every machine in its default
configuration and check it doesn't fall over". This will fail
for a machine that defaults to 2GB RAM on a 32-bit host.

> RME tests (queued to testing/next by Alex) already use 2GB so in such
> environment we would have failure anyway.

I think we probably don't run the 'check-functional' tests on
32-bit hosts.

-- PMM