linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: roy.franz@linaro.org (Roy Franz)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V3 RFC 00/16]  EFI stub for ARM
Date: Fri,  9 Aug 2013 16:26:01 -0700	[thread overview]
Message-ID: <1376090777-20090-1-git-send-email-roy.franz@linaro.org> (raw)


This patch series adds EFI stub support for the ARM architecture.
Some code that was previously only used by x86/x86_64 is now shared
and has been made more general.  The stub for ARM is implemented in
a similar manner to x86 in that it is a shim layer between EFI and
the normal zImage/bzImage boot process, and that an image with the
stub configured is bootable as both a zImage and EFI application.

This patch now (new for v3) series depends on the 
"arm: Add [U]EFI runtime services support" patches by leif.lindholm at linaro.org.
The Kconfig option now depends on the "CONFIG_EFI" option that his series
adds, and this option will ensure a little endian configuration.  Also, the
EFI support is used to handle the EFI memory map the stub passes to the kernel.
There are some minor changes to be coordinated with the EFI runtime services
patch series, so I have put this back to RFC status.  These changes should be
minor and relate to final device tree bindings.

Changes since v2:
* EFI bugfix "correct call to free_pages" that patch series
  depends on now in mainline
* remove "-fno-stack-protector" from decompressor Makefile.  The current code doesn't
  trigger the stack protection, so the decompressor now compiles with it still
  on.  Note that there has never been any stack protection in the decompressor
  since the stack usage doesn't trigger the heuristic in GCC, so right now
  the "-fno-stack-protector" is a noop.
* Changed EFI command line handling to not have a fixed limit.
* Change FDT memory allocation to retry with a larger allocation if
  first educated guess is inadequate.
* Correctly set 'SizeOfCode' in PE/COFF header.
* Reviewed ".setup" section that is in x86 PE/COFF header.  This is used for x86
  to account for code that is not in the .text section.  We don't need this
  for ARM, as all of our code is in the .text section, or in the PE/COFF header
  itself.
* Moved EFI_STUB_ERROR #define to header file to share between stub C and ASM.
* Variety of cleanups and fixes in head.S.
* Changed update_fdt_and_exit_boot() to just update the device tree, and
  renamed appropriately.  Memory allocations moved out of this function as
  well, which enables the retries if the initial FDT size is too small.
  Note that in order to do the retried allocations, the original FDT and 
  command line memory regions are left allocated.  This is OK since the kernel
  has the memory map and will free these allocations along with the initrd
  and new fdt allocations.
* Added prefix to all prints, reduced number of prints, and reviewed all
  messages.
* Change mixed usage of dtb/fdt to all be fdt or "device tree" in efi-stub.c
* remove unnecessary zimage_size variable from relocate_kernel()
* correct return types on EFI functions - should be efi_status_t, not int.



Changes since V1:
* Updated head.S based on feedback from Dave Martin.  ARM/THUMB
  switches now much cleaner.
* Broke up changes to x86 and common code into more patches.
  10 more patches in this series.

Roy Franz (16):
  Move common EFI stub code from x86 arch code to common location
  Add system pointer argument to shared EFI stub related     functions
    so they no longer use global system table     pointer as they did
    when part of eboot.c.     This code is now shared, so using a
    global variable     as part of the interface is not that nice.
    Also,     by avoiding any global variables in the ARM EFI stub,    
    this allows the code to be position independent without    
    requiring GOT fixups.
  Rename memory allocation/free functions
  Add minimum address parameter to efi_low_alloc()
  rename __get_map() to efi_get_memory_map(), add parameter     to
    optionally return mmap key. The mmap key is required     to exit
    EFI boot services, and allows efi_get_memory_map()     to be used
    for getting final memory map.
  Enforce minimum alignment of 1 page on allocations.     The
    efi_high_alloc() and efi_low_alloc() functions     use the
    EFI_ALLOCATE_ADDRESS option to the EFI     function
    allocate_pages(), which requires a minimum     of page alignment,
    and rejects all other requests.
  Allow efi_free() to be called with size of 0, and do nothing in that
    case.
  Generalize handle_ramdisks() and rename to handle_cmdline_files().
  Renames in handle_cmdline_files() to complete generalization.
  Move EFI_READ_CHUNK_SIZE define to shared location.
  Add proper definitions for some EFI function pointers.
  Fix types in EFI calls to match EFI function definitions.
  resolve warnings found on ARM compile
  Add strstr to compressed string.c for ARM.
  Add EFI stub for ARM
  Add config EFI_STUB for ARM to Kconfig

 arch/arm/Kconfig                       |   11 +
 arch/arm/boot/compressed/Makefile      |   15 +-
 arch/arm/boot/compressed/efi-header.S  |  111 +++++++
 arch/arm/boot/compressed/efi-stub.c    |  448 ++++++++++++++++++++++++++++
 arch/arm/boot/compressed/efi-stub.h    |    5 +
 arch/arm/boot/compressed/head.S        |   90 +++++-
 arch/arm/boot/compressed/string.c      |   21 ++
 arch/x86/boot/compressed/eboot.c       |  490 ++-----------------------------
 arch/x86/boot/compressed/eboot.h       |    9 -
 drivers/firmware/efi/efi-stub-helper.c |  505 ++++++++++++++++++++++++++++++++
 include/linux/efi.h                    |   51 +++-
 11 files changed, 1263 insertions(+), 493 deletions(-)
 create mode 100644 arch/arm/boot/compressed/efi-header.S
 create mode 100644 arch/arm/boot/compressed/efi-stub.c
 create mode 100644 arch/arm/boot/compressed/efi-stub.h
 create mode 100644 drivers/firmware/efi/efi-stub-helper.c

-- 
1.7.10.4

             reply	other threads:[~2013-08-09 23:26 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-09 23:26 Roy Franz [this message]
2013-08-09 23:26 ` [PATCH 01/16] Move common EFI stub code from x86 arch code to common location Roy Franz
2013-08-13 13:56   ` Mark Salter
2013-08-30 12:45     ` Grant Likely
2013-08-09 23:26 ` [PATCH 02/16] Add system pointer argument to shared EFI stub related functions so they no longer use global system table pointer as they did when part of eboot.c. This code is now shared, so using a global variable as part of the interface is not that nice. Also, by avoiding any global variables in the ARM EFI stub, this allows the code to be position independent without requiring GOT fixups Roy Franz
2013-08-13 13:56   ` Mark Salter
2013-08-30 12:53   ` Grant Likely
2013-08-30 21:46     ` Roy Franz
2013-08-09 23:26 ` [PATCH 03/16] Rename memory allocation/free functions Roy Franz
2013-08-13 13:56   ` Mark Salter
2013-08-30 12:54   ` Grant Likely
2013-08-09 23:26 ` [PATCH 04/16] Add minimum address parameter to efi_low_alloc() Roy Franz
2013-08-13 13:56   ` Mark Salter
2013-08-30 13:01   ` Grant Likely
2013-08-30 22:12     ` Roy Franz
2013-08-30 22:55       ` Grant Likely
2013-08-09 23:26 ` [PATCH 05/16] rename __get_map() to efi_get_memory_map(), add parameter to optionally return mmap key. The mmap key is required to exit EFI boot services, and allows efi_get_memory_map() to be used for getting final memory map Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:06   ` Grant Likely
2013-08-09 23:26 ` [PATCH 06/16] Enforce minimum alignment of 1 page on allocations. The efi_high_alloc() and efi_low_alloc() functions use the EFI_ALLOCATE_ADDRESS option to the EFI function allocate_pages(), which requires a minimum of page alignment, and rejects all other requests Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:08   ` Grant Likely
2013-08-09 23:26 ` [PATCH 07/16] Allow efi_free() to be called with size of 0, and do nothing in that case Roy Franz
2013-08-09 23:26 ` [PATCH 08/16] Generalize handle_ramdisks() and rename to handle_cmdline_files() Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:31   ` Grant Likely
2013-08-09 23:26 ` [PATCH 09/16] Renames in handle_cmdline_files() to complete generalization Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:32   ` Grant Likely
2013-08-09 23:26 ` [PATCH 10/16] Move EFI_READ_CHUNK_SIZE define to shared location Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:33   ` Grant Likely
2013-08-30 22:49     ` Roy Franz
2013-08-09 23:26 ` [PATCH 11/16] Add proper definitions for some EFI function pointers Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:34   ` Grant Likely
2013-08-09 23:26 ` [PATCH 12/16] Fix types in EFI calls to match EFI function definitions Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:39   ` Grant Likely
2013-08-30 23:12     ` Roy Franz
2013-08-30 23:14       ` Grant Likely
2013-08-30 23:16         ` Roy Franz
2013-08-09 23:26 ` [PATCH 13/16] resolve warnings found on ARM compile Roy Franz
2013-08-13 13:57   ` Mark Salter
2013-08-30 13:41   ` Grant Likely
2013-08-09 23:26 ` [PATCH 14/16] Add strstr to compressed string.c for ARM Roy Franz
2013-08-30 13:43   ` Grant Likely
2013-08-30 13:47     ` Russell King - ARM Linux
2013-08-30 14:02       ` Grant Likely
2013-08-30 14:16         ` Roy Franz
2013-08-09 23:26 ` [PATCH 15/16] Add EFI stub " Roy Franz
2013-08-12 14:51   ` Mark Salter
2013-08-13  0:01     ` Roy Franz
2013-08-13 14:19   ` Dave Martin
     [not found]     ` <CAFECyb_bZYE5tCss8u4RkO=pdpMApo=KV0L0ORMfOnYHuiNf6w@mail.gmail.com>
2013-08-14  2:07       ` Roy Franz
2013-08-30 14:36   ` Grant Likely
2013-08-31  0:35     ` Roy Franz
2013-08-09 23:26 ` [PATCH 16/16] Add config EFI_STUB for ARM to Kconfig Roy Franz
2013-08-13 13:18   ` Dave Martin
2013-08-13 17:37     ` Roy Franz
2013-08-30 11:17       ` Grant Likely
2013-08-12 14:02 ` [PATCH V3 RFC 00/16] EFI stub for ARM Mark Salter
2013-08-13  1:13   ` Roy Franz
2013-08-13 15:46     ` Mark Salter
2013-08-13 17:58 ` Roy Franz
2013-08-17  0:16   ` Roy Franz
2013-08-23 22:40     ` Roy Franz
2013-09-02  7:36       ` Matt Fleming
2013-09-02 10:33   ` Matt Fleming
2013-09-02 15:49     ` Roy Franz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1376090777-20090-1-git-send-email-roy.franz@linaro.org \
    --to=roy.franz@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).