From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Thu, 13 May 2021 07:36:16 -0700
Subject: [PATCH 1/1] sandbox: ensure that state->ram_buf is in low memory
In-Reply-To: <B9E36C6A-A3CA-4318-9E71-AEB2770E7FFB@gmx.de>
References: <20210511190316.29710-1-xypron.glpk@gmx.de>
 <CAPnjgZ1RfPGAw7-XHWnuonSCijuvJKH643MEs9SqEzeapoJF8A@mail.gmail.com>
 <B9E36C6A-A3CA-4318-9E71-AEB2770E7FFB@gmx.de>
Message-ID: <CAPnjgZ2Ekkj=_beHukL5zPLOdnjAxVBOOuveCb38sKzQoe9DJw@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Heinrich,

On Wed, 12 May 2021 at 15:28, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> Am 12. Mai 2021 18:01:17 MESZ schrieb Simon Glass <sjg@chromium.org>:
> >Hi Heinrich,
> >
> >On Tue, 11 May 2021 at 13:03, Heinrich Schuchardt <xypron.glpk@gmx.de>
> >wrote:
> >>
> >> Addresses in state->ram_buf must be in the low 4 GiB of the address
> >space.
> >> Otherwise we cannot correctly fill SMBIOS tables. This shows up in
> >warnings
> >> like:
> >>
> >>     WARNING: SMBIOS table_address overflow 7f752735e020
> >
> >This sounds like a bug in the smbios-table code. For sandbox it should
> >perhaps use addresses instead of pointers.
> >
> >I think that code (that I unfortunately wrote) was an expeditious way
> >of getting it running, but is not correct.
>
> The field you are filling is only 32bit wide. I wonder how that table is meant to work on systems where the lowest memory address is above 4 GiB. Such ARMv8 systems exist.

map_to_sysmem() will give you a 32-bit wide address. Yes SMBIOS is
legacy and designed for 4GB.

Regards,
Simon