On Wed, 31 Jul 2024 at 20:08, Simon Glass <s...@chromium.org> wrote: > > Hi Sughosh, > > On Wed, 31 Jul 2024 at 01:26, Sughosh Ganu <sughosh.g...@linaro.org> wrote: > > > > On Mon, 29 Jul 2024 at 20:58, Simon Glass <s...@chromium.org> wrote: > > > > > > Hi Sughosh, > > > > > > On Mon, 29 Jul 2024 at 02:53, Sughosh Ganu <sughosh.g...@linaro.org> > > > wrote: > > > > > > > > On Fri, 26 Jul 2024 at 05:02, Simon Glass <s...@chromium.org> wrote: > > > > > > > > > > Hi Sughosh, > > > > > > > > > > On Wed, 24 Jul 2024 at 00:05, Sughosh Ganu <sughosh.g...@linaro.org> > > > > > wrote: > > > > > > > > > > > > Instead of a randomly selected address, use an LMB allocated one for > > > > > > reading the file into memory. With the LMB map now being persistent > > > > > > and global, the address used for reading the file might be already > > > > > > allocated as non-overwritable, resulting in a failure. Get a valid > > > > > > address from LMB and then read the file to that address. > > > > > > > > > > > > Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org> > > > > > > --- > > > > > > Changes since rfc: None > > > > > > > > > > > > test/boot/cedit.c | 6 +++++- > > > > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > > > > > > > > > > > No, we should not start putting lmb into tests like this. > > > > > > > > The test fails if the address is not allocated through an lmb api. > > > > Although I can check why. > > > > > > Neither am I, but you did put the tcg in the same region so perhaps > > > that is an issue? > > > > I now remember the issue. The cedit_fdt() function uses a random > > address(0x1000) and tries to load the settings.dtb file to the > > address. The load command uses the lmb_alloc_addr() API function to > > reserve and use this address for loading the file. This worked earlier > > because of the local nature of the lmb memory map. But it does not > > work anymore, as this clashes with some existing reservation resulting > > in the test failure. So this change is very much needed. > > OK I see. Do you know what is clashing? I wonder if we should have > strings associated with lmb records, to make this easier, if there are > going to be so many?
Regarding your point about making use of strings for lmb records, I have something similar on my todo list. I plan to use the strings for the lmb flags associated with a memory region. The current flag values shown as part of the bdinfo command are not very helpful from the ease of understanding pov. -sughosh > > > > > > -sughosh > > > > > > > > Sandbox is designed so that low memory regions can be used in tests. I > > > pointed you to the memory-map docs. > > Regards, > Simon