On Tue, Jan 06, 2026 at 02:10:02PM +0100, Ahmad Fatoum wrote:
> Hi,
> 
> On 1/6/26 1:53 PM, Sascha Hauer wrote:
> > For use by the ELF loader in PBL to relocate barebox proper, export a
> > new relocate_image capable of relocating barebox and implement
> > relocate_to_current_adr() in terms of it.
> > 
> > Signed-off-by: Sascha Hauer <[email protected]>
> 
> Reviewed-by: Ahmad Fatoum <[email protected]>
> 
> > -   dstart = runtime_address(__rel_dyn_start);
> > -   dend = runtime_address(__rel_dyn_end);
> > -   dynsym = runtime_address(__dynsym_start);
> 
> Interestingly dynend wasn't used here and no memset, unlike for ARM and
> also it needs dynsym even with RELA...

Yes, as said, the memset is only needed for multiple relocations of the
same image. Likely nobody trapped into this on RiscV.

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Reply via email to