On 02/20/2019 01:36 PM, Alexander Graf wrote:
On 02/20/2019 12:19 PM, Andre Przywara wrote:
Hi,
I was looking for a solution to some linker problem and was hoping that
some linker cracks here on the list have some idea:
The generic ARMv8 ("arm64") code provides an exception vector table,
which architecturally requires a 2KB alignment. At the moment we don't
take any special measures about it, so we end up with a random padding
between 0 and 2044 bytes before the table:
0 bytes n * 2KB
| |
<code before table><padding>|<vector table> <code after table>
The vector table takes up a bit more than 1KB, so we happily use the
space after that and can't afford to just waste the rest of the 2KB slot
(by putting it at the end of the code, for instance).
Depending on the link order some code ends up before the table, other
behind it, with seemingly random effects on the resulting code size, due
to the padding.
This is quite annoying with the Allwinner SPLs, which are quite tight.
On the H6 it seems like most code additions lead to a build break at the
moment due to this, which needs fixing. Actually it should affect all
ARMv8 boards, but it might just not be critical for most of them.
I was wondering if we can tell the linker in the (SPL) linker script to
drop code into *two* possible regions, so that we can use a fixed
location for the vector table somewhere in the middle, and tell the
linker to fill up the space before *and* after that at will, without
requiring us to specify the exact link order of code.
I can *manually* arrange the code to make best use of the space, but
this is very fragile and breaks every so often, and is thus not
sustainable.
Another idea would be to put the exception table at the end of the image
(at least for sunxi), then fill the remaining <1KB with some data. But I
didn't find a low hanging fruit data block which fills the gap nicely.
If anyone has any ideas on how to overcome this, I would be very
grateful
to hear about it.
Can't you just put the exception table at 0x0 and make the first
instruction be the "b reset" from start.S? Then (if we need that
vector, I don't know) overwrite it with the real exception vector down
the road.
A few more ideas:
* Set CONFIG_ARMV8_SPL_EXCEPTION_VECTORS=n for H6
* Move "init" code to the lower 2kb, then dynamically generate vectors
at a post-init step and set vbar to _start
Realistically, CONFIG_ARMV8_SPL_EXCEPTION_VECTORS is currently broken in
the tree though (ifdefs on ...ARMV8__SPL...), so we may as well default
it to n :). That should save quite some space across the board.
Alex
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
