[U-Boot] Issue in u-boot; TFTP error: trying to overwrite reserved memory...

[Simon Goldschmidt]

On Mon, Sep 9, 2019 at 11:29 AM Moses Christopher
<moseschristopherb at gmail.com> wrote:
>
> Hi Simon,
>
>
> Thanks for the prompt reply.
>
> On Fri, 6 Sep, 2019, 8:13 AM Simon Goldschmidt, <simon.k.r.goldschmidt at gmail.com> wrote:
>
> Hi,
>
> On Thu, Sep 5, 2019 at 4:14 PM Moses Christopher
> <moseschristopherb at gmail.com> wrote:
> > Hello together,
> >
> > I was trying to build u-boot and spl for the arm target and tried to boot via usb-ethernet.
> > I found an issue with one of the commit made in the early 2019,
> > http://patchwork.ozlabs.org/patch/1024795/
> >
> > When using this CONFIG_LMB the max_size or the lmb_get_free_size(&lmb, load_addr); returns 0, no matter what.
> > And it triggers the following error,
> > TFTP error: trying to overwrite reserved memory...
> > I did a quick fix by adding #undef CONFIG_LMB in the file, net/tftp.c
> > So, I would like to know why this doesn’t work as it was working before applying this patch ?
>
> Can you add "#define DEBUG" as the first line in 'lib/lmb.c'? That
> should give you debug
> output when lmb is used.
>
>
> I did add DEBUG macro to lmb.c but the function having the debug messages isn't getting called. I suppose it was from fs/fs.c

Right, tftp.c is missing the call to that funcftion. Could you add the
call to 'lmb_dump_all(&lmb);'
right below 'lmb_init_and_reserve()' in tftp.c?

That should give you the output required. And while at it, tell us
what 'load_addr' is set to
(by adding a printf in tftp.c, too).

Thanks,
Simon

>
> FYI,
> I'm trying to load SPL and uboot on RAM, using USB-ETH. Also the environment is not stored separately, neither the device tree.
>
>
> The lmb code works by getting the RAM size, adding reserved areas and then only
> allowing allocations in non-reserved areay. However, the RAM size is
> not fully used
> depending on some config options and/or environment variables. There's possibly
> something wrong in your configuration around that.
>
>
> Because, earlier to this patch, net/tftp.c isn't actually checking for the reserved memory regions and is able to download the files properly on the RAM and it works. I know, that's not a good approach, hence you've made the necessary changes to correct it.
>
> Could you kindly provide me some information, where I can read more about the reserved memory regions and how exactly some region is treated as reserved region ?
>
> Also, it'd be great if you could provide some information related to the configuration of Reserved and free addresses of RAM.
>
> Thank you for your patience and time.
>
> Regards,
> Simon
>
> >
> > Best regards,
> > Moses Christopher
>
> Best regards,
> Moses Christopher