Dear Peter,

In message 1279658019.5685.125.camel@thunk you wrote:

...

I think the problem is that before relocation we have to use getenv_r(), which is done for example to read the baudrate, see arch/powerpc/lib/board.c: init_baudrate()

How hard is it to detect this case and generate a stackdump if someone calls getenv() before relocation is complete? That would help to spot places where people are breaking the rules...

It's trivial to check - all we need to addd to getenv() is something like

if (!(gd->flags & GD_FLG_RELOC)) bail_out();

But then, we could as well try to be friendly and eliminate getenv_r(). We would just need a static buffer for the result.

Best regards,

Wolfgang Denk