Hi Sughosh,
On Wed, 14 Aug 2024 at 05:02, Sughosh Ganu sughosh.ganu@linaro.org wrote:
The TCG event log buffer is being set at the end of ram memory. This region of memory is to be reserved as LMB_NOMAP memory in the LMB memory map. The current location of this buffer overlaps with the memory region reserved for the U-Boot image, which is at the top of the usable memory. This worked earlier as the LMB memory map was not global but caller specific, but fails now because of the overlap.
Move the TCG event log buffer to the start of the ram memory region instead. Move the location of the bloblist address up by 8KB to accomodate the event log buffer, and update the sandbox memory map specified in the documentation accordingly.
Signed-off-by: Sughosh Ganu sughosh.ganu@linaro.org
Changes since V1:
- Rejig the memory map to have the event log at 0xb000 and bloblist at 0xd000.
- Update the memory map in the document accordingly.
arch/sandbox/dts/test.dts | 2 +- common/Kconfig | 2 +- doc/arch/sandbox/sandbox.rst | 3 ++- test/lib/kconfig.c | 4 ++-- 4 files changed, 6 insertions(+), 5 deletions(-)
I think the TCG should be in a bloblist. I can do a patch to do that.
So if you are OK with that:
Reviewed-by: Simon Glass sjg@chromium.org
and I will send a patch later.
Regards, SImon