From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leif Lindholm Subject: Re: [PATCH 0/3 v2] arm64/efi: improve TEXT_OFFSET handling Date: Wed, 13 Aug 2014 18:29:58 +0100 Message-ID: <20140813172958.GA22224@bivouac.eciton.net> References: <1406717944-24725-1-git-send-email-ard.biesheuvel@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1406717944-24725-1-git-send-email-ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ard Biesheuvel Cc: linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, msalter-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, will.deacon-5wv7dgnIgG8@public.gmane.org, matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org List-Id: linux-efi@vger.kernel.org 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=next&id=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 Tested-by: Leif Lindholm / Leif