[PATCH] arm: boot/compressed: fix decompressor header layout for v7-M

Dave Martin Dave.Martin at arm.com
Fri May 19 10:38:33 PDT 2017


On Fri, May 19, 2017 at 06:34:01PM +0100, Dave Martin wrote:
> On Fri, May 19, 2017 at 06:16:36PM +0100, Robin Murphy wrote:

[...]

> > Hmm, couldn't it be ISA-agnostic?
> > 
> > 	ldr pc, 1f
> > 	.ltorg
> > THUMB(	.thumb	)
> > 	...
> > 
> > given that that should be interworking on anything where THUMB() is
> > valid in the first place.
> 
> Did you mean
> 
> 	ldr	pc, =BSYM(1f)
> ?
> 
> Otherwise this is quite neat: providing there are no pending literals
> when the assembler reaches the ldr pc, and providing that the ldr pc is
> word-aligned (it certainly should be) then that sequence should always
> expand to two words.

Modulo Ard's comment about position-independence...  Curses.

Cheers
---Dave



More information about the linux-arm-kernel mailing list