Hi Jan,
Sorry for the late answer.
On 01/10/2024 16:16, Jan Beulich wrote:
No functional change, albeit all globals now become hidden, and aliasing
symbols (__aeabi_{u,}idiv) as well as __memzero lose their function-ness
and size.
Signed-off-by: Jan Beulich <jbeul...@suse.com>
---
If the function-ness is important, some new construct would need
inventing. Not setting size for the aliases may even be desirable, as
I'm uncertain whether it is really legal in ELF that two entities
overlap in space.
I can't think of a reason where we would need the "function-ness".
I fear I'm breaking __memzero(), as I don't understand the purpose of
the ".word 0" next to where the FUNC_LOCAL() appears.
I am not entirely sure either. AFAIK, "0" is not a valid instruction.
This code was taken from Linux, the history doesn't give much clue
because it seems the ".word 0" was added before Linux used git.
However, it looks like Linux replace __memzero with memset() 6 years ago
on arm32. So maybe we should get rid of it? This would at least avoid
worrying on the purpose of ".word 0".
The rest of the patch looks good to me.
Cheers,
--
Julien Grall
