Re: [PATCH 0/3 v2] arm64/efi: improve TEXT_OFFSET handling

2014-08-13 Thread Leif Lindholm
On Wed, Jul 30, 2014 at 12:59:01PM +0200, Ard Biesheuvel wrote:
 Resending this series sent out yesterday with only minor changes and acks etc
 added.
 
 In summary: patch #3 relaxes the requirements imposed by the EFI stub on where
 Image may be loaded, but this breaks APM Mustang (if booting via UEFI) if 
 patch
 #1 does not go in first. Patch #2 prevents potential boot issues when Image is
 loaded such that the stub does not have to relocate it.
 
 @Will: as discussed on the list yesterday, these patches should be kept in
 sequence when going upstream, so it is best to take them through a single 
 tree.
 However, patch #3 will not apply cleanly to the arm64 tree until after 
 3.16-rc1
 is released as it depends on a trivial change going in through x86/tip [efi].
 [https://git.kernel.org/cgit/linux/kernel/git/mfleming/efi.git/commit/?h=nextid=6091c9c447370c4717ec9975813c874af490eb36]
 
 If you are ok with these patches, how would you like to proceed? Patches #1 
 and
 #2 could go in straight away (through arm64), and I can send Catalin and you a
 gentle reminder once -rc1 is released to take #3? Or instead, ack them and ask
 Matt to queue them for 3.17-late? It would be nice if this makes 3.17 as it
 fixes actual boot problems on hardware that is under development.
 
 Changes in v2:
 - add (__force void *) cast to patch #1, as suggested in LAKML discussion
 - add tested-by/acked-by lines
 - rebased patch #3 onto efi-next
 
 Mark Rutland (1):
   arm64: spin-table: handle unmapped cpu-release-addrs
 
 Ard Biesheuvel (2):
   arm64/efi: efistub: cover entire static mem footprint in PE/COFF .text
   arm64/efi: efistub: don't abort if base of DRAM is occupied
 
  arch/arm64/kernel/efi-stub.c   | 18 ++
  arch/arm64/kernel/head.S   |  6 +++---
  arch/arm64/kernel/smp_spin_table.c | 21 -
  3 files changed, 25 insertions(+), 20 deletions(-)
 
 -- 
 1.8.3.2

Since this set fixes two separate bugs on actual hardware platforms,
for the entire set - for what it's worth:
Acked-by: Leif Lindholm leif.lindh...@linaro.org
Tested-by: Leif Lindholm leif.lindh...@linaro.org

/
Leif
--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 0/3 v2] arm64/efi: improve TEXT_OFFSET handling

2014-07-30 Thread Ard Biesheuvel
Resending this series sent out yesterday with only minor changes and acks etc
added.

In summary: patch #3 relaxes the requirements imposed by the EFI stub on where
Image may be loaded, but this breaks APM Mustang (if booting via UEFI) if patch
#1 does not go in first. Patch #2 prevents potential boot issues when Image is
loaded such that the stub does not have to relocate it.

@Will: as discussed on the list yesterday, these patches should be kept in
sequence when going upstream, so it is best to take them through a single tree.
However, patch #3 will not apply cleanly to the arm64 tree until after 3.16-rc1
is released as it depends on a trivial change going in through x86/tip [efi].
[https://git.kernel.org/cgit/linux/kernel/git/mfleming/efi.git/commit/?h=nextid=6091c9c447370c4717ec9975813c874af490eb36]

If you are ok with these patches, how would you like to proceed? Patches #1 and
#2 could go in straight away (through arm64), and I can send Catalin and you a
gentle reminder once -rc1 is released to take #3? Or instead, ack them and ask
Matt to queue them for 3.17-late? It would be nice if this makes 3.17 as it
fixes actual boot problems on hardware that is under development.

Changes in v2:
- add (__force void *) cast to patch #1, as suggested in LAKML discussion
- add tested-by/acked-by lines
- rebased patch #3 onto efi-next

Mark Rutland (1):
  arm64: spin-table: handle unmapped cpu-release-addrs

Ard Biesheuvel (2):
  arm64/efi: efistub: cover entire static mem footprint in PE/COFF .text
  arm64/efi: efistub: don't abort if base of DRAM is occupied

 arch/arm64/kernel/efi-stub.c   | 18 ++
 arch/arm64/kernel/head.S   |  6 +++---
 arch/arm64/kernel/smp_spin_table.c | 21 -
 3 files changed, 25 insertions(+), 20 deletions(-)

-- 
1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-efi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html