grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH, RFC, RFT] ARM relocation fixes


From: Leif Lindholm
Subject: Re: [PATCH, RFC, RFT] ARM relocation fixes
Date: Tue, 3 Dec 2013 09:14:41 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

On Tue, Dec 03, 2013 at 06:37:33AM +0100, Vladimir 'φ-coder/phcoder' Serbinenko 
wrote:
> > No, BLX immediate has been supported since ARMv5T.
> > (Yes, that is effectively a 32-bit encoding in what is a 16-bit
> > instruction set - but so is the basic BL.)
> > 
> I've looked through encoding of those instructions and see how much it's
> a mess. b* and b*x don't have similar set of options which makes
> validating them a difficult error-prone task. So I think, I'll just add
> veneers to mkimage, just like we do on ia64 (either by making a
> pc-relative variant of veneers or adding fixup for them)

Not B, BL.
There is a 1-bit range difference between Thumb BL and BLX, which we
need to check for anyway. This check already exists (and must exist) in
the code. Adding veneers would be pure overhead.

/
    Leif



reply via email to

[Prev in Thread] Current Thread [Next in Thread]