Blame SOURCES/0185-Make-elfload-not-use-hooks.-Opt-for-flags-and-iterat.patch

f96e0b
From 6b0582fa0af9c5b2c3537198b3b142222137978c Mon Sep 17 00:00:00 2001
f96e0b
From: Vladimir 'phcoder' Serbinenko <phcoder@gmail.com>
f96e0b
Date: Sat, 2 Mar 2013 16:45:57 +0100
f96e0b
Subject: [PATCH 185/482] 	Make elfload not use hooks. Opt for flags and
f96e0b
 iterators instead.
f96e0b
f96e0b
---
f96e0b
 ChangeLog                                    |   4 +
f96e0b
 grub-core/kern/elf.c                         | 455 +++------------------------
f96e0b
 grub-core/kern/elfXX.c                       | 144 +++++++++
f96e0b
 grub-core/loader/i386/bsd.c                  | 137 +++-----
f96e0b
 grub-core/loader/i386/coreboot/chainloader.c |  69 ++--
f96e0b
 grub-core/loader/mips/linux.c                |  37 +--
f96e0b
 grub-core/loader/powerpc/ieee1275/linux.c    |  34 +-
f96e0b
 grub-core/loader/sparc64/ieee1275/linux.c    |  18 +-
f96e0b
 include/grub/elfload.h                       |  35 ++-
f96e0b
 9 files changed, 301 insertions(+), 632 deletions(-)
f96e0b
 create mode 100644 grub-core/kern/elfXX.c
f96e0b
f96e0b
diff --git a/ChangeLog b/ChangeLog
f96e0b
index d55dd8f..7c80ed6 100644
f96e0b
--- a/ChangeLog
f96e0b
+++ b/ChangeLog
f96e0b
@@ -1,5 +1,9 @@
f96e0b
 2013-03-02  Vladimir Serbinenko  <phcoder@gmail.com>
f96e0b
 
f96e0b
+	Make elfload not use hooks. Opt for flags and iterators instead.
f96e0b
+
f96e0b
+2013-03-02  Vladimir Serbinenko  <phcoder@gmail.com>
f96e0b
+
f96e0b
 	* grub-core/lib/ia64/longjmp.S: Fix the name of longjmp function.
f96e0b
 	* grub-core/lib/ia64/setjmp.S: Fix the name of setjmp function.
f96e0b
 
f96e0b
diff --git a/grub-core/kern/elf.c b/grub-core/kern/elf.c
f96e0b
index f52ca21..5f99c43 100644
f96e0b
--- a/grub-core/kern/elf.c
f96e0b
+++ b/grub-core/kern/elf.c
f96e0b
@@ -51,6 +51,7 @@ grub_elf_close (grub_elf_t elf)
f96e0b
   grub_file_t file = elf->file;
f96e0b
 
f96e0b
   grub_free (elf->phdrs);
f96e0b
+  grub_free (elf->filename);
f96e0b
   grub_free (elf);
f96e0b
 
f96e0b
   if (file)
f96e0b
@@ -85,9 +86,14 @@ grub_elf_file (grub_file_t file, const char *filename)
f96e0b
   if (grub_elf_check_header (elf))
f96e0b
     goto fail;
f96e0b
 
f96e0b
+  elf->filename = grub_strdup (filename);
f96e0b
+  if (!elf->filename)
f96e0b
+    goto fail;
f96e0b
+
f96e0b
   return elf;
f96e0b
 
f96e0b
 fail:
f96e0b
+  grub_free (elf->filename);
f96e0b
   grub_free (elf->phdrs);
f96e0b
   grub_free (elf);
f96e0b
   return 0;
f96e0b
@@ -112,420 +118,41 @@ grub_elf_open (const char *name)
f96e0b
 
f96e0b
 
f96e0b
 /* 32-bit */
f96e0b
-
f96e0b
-int
f96e0b
-grub_elf_is_elf32 (grub_elf_t elf)
f96e0b
-{
f96e0b
-  return elf->ehdr.ehdr32.e_ident[EI_CLASS] == ELFCLASS32;
f96e0b
-}
f96e0b
-
f96e0b
-static grub_err_t
f96e0b
-grub_elf32_load_phdrs (grub_elf_t elf, const char *filename)
f96e0b
-{
f96e0b
-  grub_ssize_t phdrs_size;
f96e0b
-
f96e0b
-  phdrs_size = elf->ehdr.ehdr32.e_phnum * elf->ehdr.ehdr32.e_phentsize;
f96e0b
-
f96e0b
-  grub_dprintf ("elf", "Loading program headers at 0x%llx, size 0x%lx.\n",
f96e0b
-		(unsigned long long) elf->ehdr.ehdr32.e_phoff,
f96e0b
-		(unsigned long) phdrs_size);
f96e0b
-
f96e0b
-  elf->phdrs = grub_malloc (phdrs_size);
f96e0b
-  if (! elf->phdrs)
f96e0b
-    return grub_errno;
f96e0b
-
f96e0b
-  if ((grub_file_seek (elf->file, elf->ehdr.ehdr32.e_phoff) == (grub_off_t) -1)
f96e0b
-      || (grub_file_read (elf->file, elf->phdrs, phdrs_size) != phdrs_size))
f96e0b
-    {
f96e0b
-      if (!grub_errno)
f96e0b
-	grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
-		    filename);
f96e0b
-      return grub_errno;
f96e0b
-    }
f96e0b
-
f96e0b
-  return GRUB_ERR_NONE;
f96e0b
-}
f96e0b
-
f96e0b
-grub_err_t
f96e0b
-grub_elf32_phdr_iterate (grub_elf_t elf,
f96e0b
-			 const char *filename,
f96e0b
-			 grub_elf32_phdr_iterate_hook_t hook, void *hook_arg)
f96e0b
-{
f96e0b
-  Elf32_Phdr *phdrs;
f96e0b
-  unsigned int i;
f96e0b
-
f96e0b
-  if (! elf->phdrs)
f96e0b
-    if (grub_elf32_load_phdrs (elf, filename))
f96e0b
-      return grub_errno;
f96e0b
-  phdrs = elf->phdrs;
f96e0b
-
f96e0b
-  for (i = 0; i < elf->ehdr.ehdr32.e_phnum; i++)
f96e0b
-    {
f96e0b
-      Elf32_Phdr *phdr = phdrs + i;
f96e0b
-      grub_dprintf ("elf",
f96e0b
-		    "Segment %u: type 0x%x paddr 0x%lx memsz 0x%lx "
f96e0b
-		    "filesz %lx\n",
f96e0b
-		    i, phdr->p_type,
f96e0b
-		    (unsigned long) phdr->p_paddr,
f96e0b
-		    (unsigned long) phdr->p_memsz,
f96e0b
-		    (unsigned long) phdr->p_filesz);
f96e0b
-      if (hook (elf, phdr, hook_arg))
f96e0b
-	break;
f96e0b
-    }
f96e0b
-
f96e0b
-  return grub_errno;
f96e0b
-}
f96e0b
-
f96e0b
-struct grub_elf32_size_ctx
f96e0b
-{
f96e0b
-  Elf32_Addr segments_start, segments_end;
f96e0b
-  int nr_phdrs;
f96e0b
-  grub_uint32_t curr_align;
f96e0b
-};
f96e0b
-
f96e0b
-/* Run through the program headers to calculate the total memory size we
f96e0b
- * should claim.  */
f96e0b
-static int
f96e0b
-grub_elf32_calcsize (grub_elf_t _elf  __attribute__ ((unused)),
f96e0b
-		     Elf32_Phdr *phdr, void *data)
f96e0b
-{
f96e0b
-  struct grub_elf32_size_ctx *ctx = data;
f96e0b
-
f96e0b
-  /* Only consider loadable segments.  */
f96e0b
-  if (phdr->p_type != PT_LOAD)
f96e0b
-    return 0;
f96e0b
-  ctx->nr_phdrs++;
f96e0b
-  if (phdr->p_paddr < ctx->segments_start)
f96e0b
-    ctx->segments_start = phdr->p_paddr;
f96e0b
-  if (phdr->p_paddr + phdr->p_memsz > ctx->segments_end)
f96e0b
-    ctx->segments_end = phdr->p_paddr + phdr->p_memsz;
f96e0b
-  if (ctx->curr_align < phdr->p_align)
f96e0b
-    ctx->curr_align = phdr->p_align;
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-/* Calculate the amount of memory spanned by the segments.  */
f96e0b
-grub_size_t
f96e0b
-grub_elf32_size (grub_elf_t elf, const char *filename,
f96e0b
-		 Elf32_Addr *base, grub_uint32_t *max_align)
f96e0b
-{
f96e0b
-  struct grub_elf32_size_ctx ctx = {
f96e0b
-    .segments_start = (Elf32_Addr) -1,
f96e0b
-    .segments_end = 0,
f96e0b
-    .nr_phdrs = 0,
f96e0b
-    .curr_align = 1
f96e0b
-  };
f96e0b
-
f96e0b
-  grub_elf32_phdr_iterate (elf, filename, grub_elf32_calcsize, &ctx;;
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = 0;
f96e0b
-
f96e0b
-  if (ctx.nr_phdrs == 0)
f96e0b
-    {
f96e0b
-      grub_error (GRUB_ERR_BAD_OS, "no program headers present");
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  if (ctx.segments_end < ctx.segments_start)
f96e0b
-    {
f96e0b
-      /* Very bad addresses.  */
f96e0b
-      grub_error (GRUB_ERR_BAD_OS, "bad program header load addresses");
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = ctx.segments_start;
f96e0b
-  if (max_align)
f96e0b
-    *max_align = ctx.curr_align;
f96e0b
-  return ctx.segments_end - ctx.segments_start;
f96e0b
-}
f96e0b
-
f96e0b
-struct grub_elf32_load_ctx
f96e0b
-{
f96e0b
-  const char *filename;
f96e0b
-  grub_elf32_load_hook_t load_hook;
f96e0b
-  grub_addr_t load_base;
f96e0b
-  grub_size_t load_size;
f96e0b
-};
f96e0b
-
f96e0b
-static int
f96e0b
-grub_elf32_load_segment (grub_elf_t elf, Elf32_Phdr *phdr, void *data)
f96e0b
-{
f96e0b
-  struct grub_elf32_load_ctx *ctx = data;
f96e0b
-  grub_addr_t load_addr;
f96e0b
-  int do_load = 1;
f96e0b
-
f96e0b
-  load_addr = phdr->p_paddr;
f96e0b
-  if (ctx->load_hook && ctx->load_hook (phdr, &load_addr, &do_load))
f96e0b
-    return 1;
f96e0b
-
f96e0b
-  if (! do_load)
f96e0b
-    return 0;
f96e0b
-
f96e0b
-  if (load_addr < ctx->load_base)
f96e0b
-    ctx->load_base = load_addr;
f96e0b
-
f96e0b
-  grub_dprintf ("elf", "Loading segment at 0x%llx, size 0x%llx\n",
f96e0b
-		(unsigned long long) load_addr,
f96e0b
-		(unsigned long long) phdr->p_memsz);
f96e0b
-
f96e0b
-  if (grub_file_seek (elf->file, phdr->p_offset) == (grub_off_t) -1)
f96e0b
-    return grub_errno;
f96e0b
-
f96e0b
-  if (phdr->p_filesz)
f96e0b
-    {
f96e0b
-      grub_ssize_t read;
f96e0b
-      read = grub_file_read (elf->file, (void *) load_addr, phdr->p_filesz);
f96e0b
-      if (read != (grub_ssize_t) phdr->p_filesz)
f96e0b
-	{
f96e0b
-	  /* XXX How can we free memory from `ctx->load_hook'? */
f96e0b
-	  if (!grub_errno)
f96e0b
-	    grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
-			ctx->filename);
f96e0b
-	  return grub_errno;
f96e0b
-	}
f96e0b
-    }
f96e0b
-
f96e0b
-  if (phdr->p_filesz < phdr->p_memsz)
f96e0b
-    grub_memset ((void *) (long) (load_addr + phdr->p_filesz),
f96e0b
-		 0, phdr->p_memsz - phdr->p_filesz);
f96e0b
-
f96e0b
-  ctx->load_size += phdr->p_memsz;
f96e0b
-
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-/* Load every loadable segment into memory specified by `_load_hook'.  */
f96e0b
-grub_err_t
f96e0b
-grub_elf32_load (grub_elf_t elf, const char *filename,
f96e0b
-		 grub_elf32_load_hook_t load_hook,
f96e0b
-		 grub_addr_t *base, grub_size_t *size)
f96e0b
-{
f96e0b
-  struct grub_elf32_load_ctx ctx = {
f96e0b
-    .filename = filename,
f96e0b
-    .load_hook = load_hook,
f96e0b
-    .load_base = (grub_addr_t) -1ULL,
f96e0b
-    .load_size = 0
f96e0b
-  };
f96e0b
-  grub_err_t err;
f96e0b
-
f96e0b
-  err = grub_elf32_phdr_iterate (elf, filename, grub_elf32_load_segment, &ctx;;
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = ctx.load_base;
f96e0b
-  if (size)
f96e0b
-    *size = ctx.load_size;
f96e0b
-
f96e0b
-  return err;
f96e0b
-}
f96e0b
+#define ehdrXX ehdr32
f96e0b
+#define ELFCLASSXX ELFCLASS32
f96e0b
+#define ElfXX_Addr Elf32_Addr
f96e0b
+#define grub_elfXX_size grub_elf32_size
f96e0b
+#define grub_elfXX_load grub_elf32_load
f96e0b
+#define FOR_ELFXX_PHDRS FOR_ELF32_PHDRS
f96e0b
+#define grub_elf_is_elfXX grub_elf_is_elf32
f96e0b
+#define grub_elfXX_load_phdrs grub_elf32_load_phdrs
f96e0b
+#define ElfXX_Phdr Elf32_Phdr
f96e0b
+#define grub_uintXX_t grub_uint32_t
f96e0b
+
f96e0b
+#include "elfXX.c"
f96e0b
+
f96e0b
+#undef ehdrXX
f96e0b
+#undef ELFCLASSXX
f96e0b
+#undef ElfXX_Addr
f96e0b
+#undef grub_elfXX_size
f96e0b
+#undef grub_elfXX_load
f96e0b
+#undef FOR_ELFXX_PHDRS
f96e0b
+#undef grub_elf_is_elfXX
f96e0b
+#undef grub_elfXX_load_phdrs
f96e0b
+#undef ElfXX_Phdr
f96e0b
+#undef grub_uintXX_t
f96e0b
 
f96e0b
 
f96e0b
 /* 64-bit */
f96e0b
-
f96e0b
-int
f96e0b
-grub_elf_is_elf64 (grub_elf_t elf)
f96e0b
-{
f96e0b
-  return elf->ehdr.ehdr64.e_ident[EI_CLASS] == ELFCLASS64;
f96e0b
-}
f96e0b
-
f96e0b
-static grub_err_t
f96e0b
-grub_elf64_load_phdrs (grub_elf_t elf, const char *filename)
f96e0b
-{
f96e0b
-  grub_ssize_t phdrs_size;
f96e0b
-
f96e0b
-  phdrs_size = elf->ehdr.ehdr64.e_phnum * elf->ehdr.ehdr64.e_phentsize;
f96e0b
-
f96e0b
-  grub_dprintf ("elf", "Loading program headers at 0x%llx, size 0x%lx.\n",
f96e0b
-		(unsigned long long) elf->ehdr.ehdr64.e_phoff,
f96e0b
-		(unsigned long) phdrs_size);
f96e0b
-
f96e0b
-  elf->phdrs = grub_malloc (phdrs_size);
f96e0b
-  if (! elf->phdrs)
f96e0b
-    return grub_errno;
f96e0b
-
f96e0b
-  if ((grub_file_seek (elf->file, elf->ehdr.ehdr64.e_phoff) == (grub_off_t) -1)
f96e0b
-      || (grub_file_read (elf->file, elf->phdrs, phdrs_size) != phdrs_size))
f96e0b
-    {
f96e0b
-      if (!grub_errno)
f96e0b
-	grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
-		    filename);
f96e0b
-      return grub_errno;
f96e0b
-    }
f96e0b
-
f96e0b
-  return GRUB_ERR_NONE;
f96e0b
-}
f96e0b
-
f96e0b
-grub_err_t
f96e0b
-grub_elf64_phdr_iterate (grub_elf_t elf,
f96e0b
-			 const char *filename,
f96e0b
-			 grub_elf64_phdr_iterate_hook_t hook, void *hook_arg)
f96e0b
-{
f96e0b
-  Elf64_Phdr *phdrs;
f96e0b
-  unsigned int i;
f96e0b
-
f96e0b
-  if (! elf->phdrs)
f96e0b
-    if (grub_elf64_load_phdrs (elf, filename))
f96e0b
-      return grub_errno;
f96e0b
-  phdrs = elf->phdrs;
f96e0b
-
f96e0b
-  for (i = 0; i < elf->ehdr.ehdr64.e_phnum; i++)
f96e0b
-    {
f96e0b
-      Elf64_Phdr *phdr = phdrs + i;
f96e0b
-      grub_dprintf ("elf",
f96e0b
-		    "Segment %u: type 0x%x paddr 0x%lx memsz 0x%lx "
f96e0b
-		    "filesz %lx\n",
f96e0b
-		    i, phdr->p_type,
f96e0b
-		    (unsigned long) phdr->p_paddr,
f96e0b
-		    (unsigned long) phdr->p_memsz,
f96e0b
-		    (unsigned long) phdr->p_filesz);
f96e0b
-      if (hook (elf, phdr, hook_arg))
f96e0b
-	break;
f96e0b
-    }
f96e0b
-
f96e0b
-  return grub_errno;
f96e0b
-}
f96e0b
-
f96e0b
-struct grub_elf64_size_ctx
f96e0b
-{
f96e0b
-  Elf64_Addr segments_start, segments_end;
f96e0b
-  int nr_phdrs;
f96e0b
-  grub_uint64_t curr_align;
f96e0b
-};
f96e0b
-
f96e0b
-/* Run through the program headers to calculate the total memory size we
f96e0b
- * should claim.  */
f96e0b
-static int
f96e0b
-grub_elf64_calcsize (grub_elf_t _elf  __attribute__ ((unused)),
f96e0b
-		     Elf64_Phdr *phdr, void *data)
f96e0b
-{
f96e0b
-  struct grub_elf64_size_ctx *ctx = data;
f96e0b
-
f96e0b
-  /* Only consider loadable segments.  */
f96e0b
-  if (phdr->p_type != PT_LOAD)
f96e0b
-    return 0;
f96e0b
-  ctx->nr_phdrs++;
f96e0b
-  if (phdr->p_paddr < ctx->segments_start)
f96e0b
-    ctx->segments_start = phdr->p_paddr;
f96e0b
-  if (phdr->p_paddr + phdr->p_memsz > ctx->segments_end)
f96e0b
-    ctx->segments_end = phdr->p_paddr + phdr->p_memsz;
f96e0b
-  if (ctx->curr_align < phdr->p_align)
f96e0b
-    ctx->curr_align = phdr->p_align;
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-/* Calculate the amount of memory spanned by the segments.  */
f96e0b
-grub_size_t
f96e0b
-grub_elf64_size (grub_elf_t elf, const char *filename,
f96e0b
-		 Elf64_Addr *base, grub_uint64_t *max_align)
f96e0b
-{
f96e0b
-  struct grub_elf64_size_ctx ctx = {
f96e0b
-    .segments_start = (Elf64_Addr) -1,
f96e0b
-    .segments_end = 0,
f96e0b
-    .nr_phdrs = 0,
f96e0b
-    .curr_align = 1
f96e0b
-  };
f96e0b
-
f96e0b
-  grub_elf64_phdr_iterate (elf, filename, grub_elf64_calcsize, &ctx;;
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = 0;
f96e0b
-
f96e0b
-  if (ctx.nr_phdrs == 0)
f96e0b
-    {
f96e0b
-      grub_error (GRUB_ERR_BAD_OS, "no program headers present");
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  if (ctx.segments_end < ctx.segments_start)
f96e0b
-    {
f96e0b
-      /* Very bad addresses.  */
f96e0b
-      grub_error (GRUB_ERR_BAD_OS, "bad program header load addresses");
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = ctx.segments_start;
f96e0b
-  if (max_align)
f96e0b
-    *max_align = ctx.curr_align;
f96e0b
-  return ctx.segments_end - ctx.segments_start;
f96e0b
-}
f96e0b
-
f96e0b
-struct grub_elf64_load_ctx
f96e0b
-{
f96e0b
-  const char *filename;
f96e0b
-  grub_elf64_load_hook_t load_hook;
f96e0b
-  grub_addr_t load_base;
f96e0b
-  grub_size_t load_size;
f96e0b
-};
f96e0b
-
f96e0b
-static int
f96e0b
-grub_elf64_load_segment (grub_elf_t elf, Elf64_Phdr *phdr, void *data)
f96e0b
-{
f96e0b
-  struct grub_elf64_load_ctx *ctx = data;
f96e0b
-  grub_addr_t load_addr;
f96e0b
-  int do_load = 1;
f96e0b
-
f96e0b
-  load_addr = phdr->p_paddr;
f96e0b
-  if (ctx->load_hook && ctx->load_hook (phdr, &load_addr, &do_load))
f96e0b
-    return 1;
f96e0b
-
f96e0b
-  if (! do_load)
f96e0b
-    return 0;
f96e0b
-
f96e0b
-  if (load_addr < ctx->load_base)
f96e0b
-    ctx->load_base = load_addr;
f96e0b
-
f96e0b
-  grub_dprintf ("elf", "Loading segment at 0x%llx, size 0x%llx\n",
f96e0b
-		(unsigned long long) load_addr,
f96e0b
-		(unsigned long long) phdr->p_memsz);
f96e0b
-
f96e0b
-  if (grub_file_seek (elf->file, phdr->p_offset) == (grub_off_t) -1)
f96e0b
-    return grub_errno;
f96e0b
-
f96e0b
-  if (phdr->p_filesz)
f96e0b
-    {
f96e0b
-      grub_ssize_t read;
f96e0b
-      read = grub_file_read (elf->file, (void *) load_addr, phdr->p_filesz);
f96e0b
-      if (read != (grub_ssize_t) phdr->p_filesz)
f96e0b
-	{
f96e0b
-	  /* XXX How can we free memory from `ctx->load_hook'?  */
f96e0b
-	  if (!grub_errno)
f96e0b
-	    grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
-			ctx->filename);
f96e0b
-	  return grub_errno;
f96e0b
-	}
f96e0b
-    }
f96e0b
-
f96e0b
-  if (phdr->p_filesz < phdr->p_memsz)
f96e0b
-    grub_memset ((void *) (long) (load_addr + phdr->p_filesz),
f96e0b
-		 0, phdr->p_memsz - phdr->p_filesz);
f96e0b
-
f96e0b
-  ctx->load_size += phdr->p_memsz;
f96e0b
-
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-/* Load every loadable segment into memory specified by `_load_hook'.  */
f96e0b
-grub_err_t
f96e0b
-grub_elf64_load (grub_elf_t elf, const char *filename,
f96e0b
-		 grub_elf64_load_hook_t load_hook,
f96e0b
-		 grub_addr_t *base, grub_size_t *size)
f96e0b
-{
f96e0b
-  struct grub_elf64_load_ctx ctx = {
f96e0b
-    .filename = filename,
f96e0b
-    .load_hook = load_hook,
f96e0b
-    .load_base = (grub_addr_t) -1ULL,
f96e0b
-    .load_size = 0
f96e0b
-  };
f96e0b
-  grub_err_t err;
f96e0b
-
f96e0b
-  err = grub_elf64_phdr_iterate (elf, filename, grub_elf64_load_segment, &ctx;;
f96e0b
-
f96e0b
-  if (base)
f96e0b
-    *base = ctx.load_base;
f96e0b
-  if (size)
f96e0b
-    *size = ctx.load_size;
f96e0b
-
f96e0b
-  return err;
f96e0b
-}
f96e0b
+#define ehdrXX ehdr64
f96e0b
+#define ELFCLASSXX ELFCLASS64
f96e0b
+#define ElfXX_Addr Elf64_Addr
f96e0b
+#define grub_elfXX_size grub_elf64_size
f96e0b
+#define grub_elfXX_load grub_elf64_load
f96e0b
+#define FOR_ELFXX_PHDRS FOR_ELF64_PHDRS
f96e0b
+#define grub_elf_is_elfXX grub_elf_is_elf64
f96e0b
+#define grub_elfXX_load_phdrs grub_elf64_load_phdrs
f96e0b
+#define ElfXX_Phdr Elf64_Phdr
f96e0b
+#define grub_uintXX_t grub_uint64_t
f96e0b
+
f96e0b
+#include "elfXX.c"
f96e0b
diff --git a/grub-core/kern/elfXX.c b/grub-core/kern/elfXX.c
f96e0b
new file mode 100644
f96e0b
index 0000000..b35e235
f96e0b
--- /dev/null
f96e0b
+++ b/grub-core/kern/elfXX.c
f96e0b
@@ -0,0 +1,144 @@
f96e0b
+int
f96e0b
+grub_elf_is_elfXX (grub_elf_t elf)
f96e0b
+{
f96e0b
+  return elf->ehdr.ehdrXX.e_ident[EI_CLASS] == ELFCLASSXX;
f96e0b
+}
f96e0b
+
f96e0b
+grub_err_t
f96e0b
+grub_elfXX_load_phdrs (grub_elf_t elf)
f96e0b
+{
f96e0b
+  grub_ssize_t phdrs_size;
f96e0b
+
f96e0b
+  if (elf->phdrs)
f96e0b
+    return GRUB_ERR_NONE;
f96e0b
+
f96e0b
+  phdrs_size = elf->ehdr.ehdrXX.e_phnum * elf->ehdr.ehdrXX.e_phentsize;
f96e0b
+
f96e0b
+  grub_dprintf ("elf", "Loading program headers at 0x%llx, size 0x%lx.\n",
f96e0b
+		(unsigned long long) elf->ehdr.ehdrXX.e_phoff,
f96e0b
+		(unsigned long) phdrs_size);
f96e0b
+
f96e0b
+  elf->phdrs = grub_malloc (phdrs_size);
f96e0b
+  if (! elf->phdrs)
f96e0b
+    return grub_errno;
f96e0b
+
f96e0b
+  if ((grub_file_seek (elf->file, elf->ehdr.ehdrXX.e_phoff) == (grub_off_t) -1)
f96e0b
+      || (grub_file_read (elf->file, elf->phdrs, phdrs_size) != phdrs_size))
f96e0b
+    {
f96e0b
+      if (!grub_errno)
f96e0b
+	grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
+		    elf->filename);
f96e0b
+      return grub_errno;
f96e0b
+    }
f96e0b
+
f96e0b
+  return GRUB_ERR_NONE;
f96e0b
+}
f96e0b
+
f96e0b
+/* Calculate the amount of memory spanned by the segments.  */
f96e0b
+grub_size_t
f96e0b
+grub_elfXX_size (grub_elf_t elf,
f96e0b
+		 ElfXX_Addr *base, grub_uintXX_t *max_align)
f96e0b
+{
f96e0b
+  ElfXX_Addr segments_start = (ElfXX_Addr) -1;
f96e0b
+  ElfXX_Addr segments_end = 0;
f96e0b
+  int nr_phdrs = 0;
f96e0b
+  grub_uint32_t curr_align = 1;
f96e0b
+  ElfXX_Phdr *phdr;
f96e0b
+
f96e0b
+  /* Run through the program headers to calculate the total memory size we
f96e0b
+   * should claim.  */
f96e0b
+  FOR_ELFXX_PHDRS (elf, phdr)
f96e0b
+    {
f96e0b
+      /* Only consider loadable segments.  */
f96e0b
+      if (phdr->p_type != PT_LOAD)
f96e0b
+	continue;
f96e0b
+      nr_phdrs++;
f96e0b
+      if (phdr->p_paddr < segments_start)
f96e0b
+	segments_start = phdr->p_paddr;
f96e0b
+      if (phdr->p_paddr + phdr->p_memsz > segments_end)
f96e0b
+	segments_end = phdr->p_paddr + phdr->p_memsz;
f96e0b
+      if (curr_align < phdr->p_align)
f96e0b
+	curr_align = phdr->p_align;
f96e0b
+    }
f96e0b
+
f96e0b
+  if (base)
f96e0b
+    *base = 0;
f96e0b
+
f96e0b
+  if (nr_phdrs == 0)
f96e0b
+    {
f96e0b
+      grub_error (GRUB_ERR_BAD_OS, "no program headers present");
f96e0b
+      return 0;
f96e0b
+    }
f96e0b
+
f96e0b
+  if (segments_end < segments_start)
f96e0b
+    {
f96e0b
+      /* Very bad addresses.  */
f96e0b
+      grub_error (GRUB_ERR_BAD_OS, "bad program header load addresses");
f96e0b
+      return 0;
f96e0b
+    }
f96e0b
+
f96e0b
+  if (base)
f96e0b
+    *base = segments_start;
f96e0b
+  if (max_align)
f96e0b
+    *max_align = curr_align;
f96e0b
+  return segments_end - segments_start;
f96e0b
+}
f96e0b
+
f96e0b
+grub_err_t
f96e0b
+grub_elfXX_load (grub_elf_t elf, const char *filename,
f96e0b
+		 void *load_offset, enum grub_elf_load_flags load_flags,
f96e0b
+		 grub_addr_t *base, grub_size_t *size)
f96e0b
+{
f96e0b
+  grub_addr_t load_base = (grub_addr_t) -1ULL;
f96e0b
+  grub_size_t load_size = 0;
f96e0b
+  ElfXX_Phdr *phdr;
f96e0b
+
f96e0b
+  FOR_ELFXX_PHDRS(elf, phdr)
f96e0b
+  {
f96e0b
+    grub_addr_t load_addr;
f96e0b
+
f96e0b
+    if (phdr->p_type != PT_LOAD && !((load_flags & GRUB_ELF_LOAD_FLAGS_LOAD_PT_DYNAMIC) && phdr->p_type == PT_DYNAMIC))
f96e0b
+      continue;
f96e0b
+
f96e0b
+    load_addr = (grub_addr_t) phdr->p_paddr;
f96e0b
+    if (load_flags & GRUB_ELF_LOAD_FLAGS_28BITS)
f96e0b
+      load_addr &= 0xFFFFFFF;
f96e0b
+    load_addr += (grub_addr_t) load_offset;
f96e0b
+
f96e0b
+    if (load_addr < load_base)
f96e0b
+      load_base = load_addr;
f96e0b
+
f96e0b
+    grub_dprintf ("elf", "Loading segment at 0x%llx, size 0x%llx\n",
f96e0b
+		  (unsigned long long) load_addr,
f96e0b
+		  (unsigned long long) phdr->p_memsz);
f96e0b
+
f96e0b
+    if (grub_file_seek (elf->file, phdr->p_offset) == (grub_off_t) -1)
f96e0b
+      return grub_errno;
f96e0b
+
f96e0b
+    if (phdr->p_filesz)
f96e0b
+      {
f96e0b
+	grub_ssize_t read;
f96e0b
+	read = grub_file_read (elf->file, (void *) load_addr, phdr->p_filesz);
f96e0b
+	if (read != (grub_ssize_t) phdr->p_filesz)
f96e0b
+	  {
f96e0b
+	    if (!grub_errno)
f96e0b
+	      grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
+			  filename);
f96e0b
+	    return grub_errno;
f96e0b
+	  }
f96e0b
+      }
f96e0b
+
f96e0b
+    if (phdr->p_filesz < phdr->p_memsz)
f96e0b
+      grub_memset ((void *) (long) (load_addr + phdr->p_filesz),
f96e0b
+		   0, phdr->p_memsz - phdr->p_filesz);
f96e0b
+
f96e0b
+    load_size += phdr->p_memsz;
f96e0b
+  }
f96e0b
+
f96e0b
+  if (base)
f96e0b
+    *base = load_base;
f96e0b
+  if (size)
f96e0b
+    *size = load_size;
f96e0b
+
f96e0b
+  return grub_errno;
f96e0b
+}
f96e0b
diff --git a/grub-core/loader/i386/bsd.c b/grub-core/loader/i386/bsd.c
f96e0b
index 9b86158..6199609 100644
f96e0b
--- a/grub-core/loader/i386/bsd.c
f96e0b
+++ b/grub-core/loader/i386/bsd.c
f96e0b
@@ -1311,89 +1311,6 @@ grub_bsd_load_aout (grub_file_t file, const char *filename)
f96e0b
 			 bss_size);
f96e0b
 }
f96e0b
 
f96e0b
-static int
f96e0b
-grub_bsd_elf32_size_hook (grub_elf_t elf __attribute__ ((unused)),
f96e0b
-			  Elf32_Phdr *phdr, void *arg __attribute__ ((unused)))
f96e0b
-{
f96e0b
-  Elf32_Addr paddr;
f96e0b
-
f96e0b
-  if (phdr->p_type != PT_LOAD
f96e0b
-      && phdr->p_type != PT_DYNAMIC)
f96e0b
-      return 0;
f96e0b
-
f96e0b
-  paddr = phdr->p_paddr & 0xFFFFFFF;
f96e0b
-
f96e0b
-  if (paddr < kern_start)
f96e0b
-    kern_start = paddr;
f96e0b
-
f96e0b
-  if (paddr + phdr->p_memsz > kern_end)
f96e0b
-    kern_end = paddr + phdr->p_memsz;
f96e0b
-
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-static grub_err_t
f96e0b
-grub_bsd_elf32_hook (Elf32_Phdr * phdr, grub_addr_t * addr, int *do_load)
f96e0b
-{
f96e0b
-  Elf32_Addr paddr;
f96e0b
-
f96e0b
-  if (phdr->p_type != PT_LOAD
f96e0b
-      && phdr->p_type != PT_DYNAMIC)
f96e0b
-    {
f96e0b
-      *do_load = 0;
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  *do_load = 1;
f96e0b
-  phdr->p_paddr &= 0xFFFFFFF;
f96e0b
-  paddr = phdr->p_paddr;
f96e0b
-
f96e0b
-  *addr = (grub_addr_t) (paddr - kern_start + (grub_uint8_t *) kern_chunk_src);
f96e0b
-
f96e0b
-  return GRUB_ERR_NONE;
f96e0b
-}
f96e0b
-
f96e0b
-static int
f96e0b
-grub_bsd_elf64_size_hook (grub_elf_t elf __attribute__ ((unused)),
f96e0b
-			  Elf64_Phdr *phdr, void *arg __attribute__ ((unused)))
f96e0b
-{
f96e0b
-  Elf64_Addr paddr;
f96e0b
-
f96e0b
-  if (phdr->p_type != PT_LOAD
f96e0b
-      && phdr->p_type != PT_DYNAMIC)
f96e0b
-    return 0;
f96e0b
-
f96e0b
-  paddr = phdr->p_paddr & 0xfffffff;
f96e0b
-
f96e0b
-  if (paddr < kern_start)
f96e0b
-    kern_start = paddr;
f96e0b
-
f96e0b
-  if (paddr + phdr->p_memsz > kern_end)
f96e0b
-    kern_end = paddr + phdr->p_memsz;
f96e0b
-
f96e0b
-  return 0;
f96e0b
-}
f96e0b
-
f96e0b
-static grub_err_t
f96e0b
-grub_bsd_elf64_hook (Elf64_Phdr * phdr, grub_addr_t * addr, int *do_load)
f96e0b
-{
f96e0b
-  Elf64_Addr paddr;
f96e0b
-
f96e0b
-  if (phdr->p_type != PT_LOAD
f96e0b
-      && phdr->p_type != PT_DYNAMIC)
f96e0b
-    {
f96e0b
-      *do_load = 0;
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  *do_load = 1;
f96e0b
-  paddr = phdr->p_paddr & 0xfffffff;
f96e0b
-
f96e0b
-  *addr = (grub_addr_t) (paddr - kern_start + (grub_uint8_t *) kern_chunk_src);
f96e0b
-
f96e0b
-  return GRUB_ERR_NONE;
f96e0b
-}
f96e0b
-
f96e0b
 static grub_err_t
f96e0b
 grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
 {
f96e0b
@@ -1405,12 +1322,29 @@ grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
   if (grub_elf_is_elf32 (elf))
f96e0b
     {
f96e0b
       grub_relocator_chunk_t ch;
f96e0b
+      Elf32_Phdr *phdr;
f96e0b
 
f96e0b
       entry = elf->ehdr.ehdr32.e_entry & 0xFFFFFFF;
f96e0b
-      err = grub_elf32_phdr_iterate (elf, filename,
f96e0b
-				     grub_bsd_elf32_size_hook, NULL);
f96e0b
-      if (err)
f96e0b
-	return err;
f96e0b
+
f96e0b
+      FOR_ELF32_PHDRS (elf, phdr)
f96e0b
+	{
f96e0b
+	  Elf32_Addr paddr;
f96e0b
+
f96e0b
+	  if (phdr->p_type != PT_LOAD
f96e0b
+	      && phdr->p_type != PT_DYNAMIC)
f96e0b
+	    continue;
f96e0b
+
f96e0b
+	  paddr = phdr->p_paddr & 0xFFFFFFF;
f96e0b
+
f96e0b
+	  if (paddr < kern_start)
f96e0b
+	    kern_start = paddr;
f96e0b
+
f96e0b
+	  if (paddr + phdr->p_memsz > kern_end)
f96e0b
+	    kern_end = paddr + phdr->p_memsz;
f96e0b
+	}
f96e0b
+
f96e0b
+      if (grub_errno)
f96e0b
+	return grub_errno;
f96e0b
       err = grub_relocator_alloc_chunk_addr (relocator, &ch,
f96e0b
 					     kern_start, kern_end - kern_start);
f96e0b
       if (err)
f96e0b
@@ -1418,7 +1352,7 @@ grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
 
f96e0b
       kern_chunk_src = get_virtual_current_address (ch);
f96e0b
 
f96e0b
-      err = grub_elf32_load (elf, filename, grub_bsd_elf32_hook, 0, 0);
f96e0b
+      err = grub_elf32_load (elf, filename, (grub_uint8_t *) kern_chunk_src - kern_start, GRUB_ELF_LOAD_FLAGS_LOAD_PT_DYNAMIC | GRUB_ELF_LOAD_FLAGS_28BITS, 0, 0);
f96e0b
       if (err)
f96e0b
 	return err;
f96e0b
       if (kernel_type != KERNEL_TYPE_OPENBSD)
f96e0b
@@ -1428,6 +1362,8 @@ grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
     }
f96e0b
   else if (grub_elf_is_elf64 (elf))
f96e0b
     {
f96e0b
+      Elf64_Phdr *phdr;
f96e0b
+
f96e0b
       is_64bit = 1;
f96e0b
 
f96e0b
       if (! grub_cpuid_has_longmode)
f96e0b
@@ -1445,10 +1381,25 @@ grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
 	  entry_hi = 0;
f96e0b
 	}
f96e0b
 
f96e0b
-      err = grub_elf64_phdr_iterate (elf, filename,
f96e0b
-				     grub_bsd_elf64_size_hook, NULL);
f96e0b
-      if (err)
f96e0b
-	return err;
f96e0b
+      FOR_ELF64_PHDRS (elf, phdr)
f96e0b
+	{
f96e0b
+	  Elf64_Addr paddr;
f96e0b
+
f96e0b
+	  if (phdr->p_type != PT_LOAD
f96e0b
+	      && phdr->p_type != PT_DYNAMIC)
f96e0b
+	    continue;
f96e0b
+
f96e0b
+	  paddr = phdr->p_paddr & 0xFFFFFFF;
f96e0b
+
f96e0b
+	  if (paddr < kern_start)
f96e0b
+	    kern_start = paddr;
f96e0b
+
f96e0b
+	  if (paddr + phdr->p_memsz > kern_end)
f96e0b
+	    kern_end = paddr + phdr->p_memsz;
f96e0b
+	}
f96e0b
+
f96e0b
+      if (grub_errno)
f96e0b
+	return grub_errno;
f96e0b
 
f96e0b
       grub_dprintf ("bsd", "kern_start = %lx, kern_end = %lx\n",
f96e0b
 		    (unsigned long) kern_start, (unsigned long) kern_end);
f96e0b
@@ -1463,7 +1414,7 @@ grub_bsd_load_elf (grub_elf_t elf, const char *filename)
f96e0b
       }
f96e0b
 
f96e0b
       err = grub_elf64_load (elf, filename,
f96e0b
-			     grub_bsd_elf64_hook, 0, 0);
f96e0b
+			     (grub_uint8_t *) kern_chunk_src - kern_start, GRUB_ELF_LOAD_FLAGS_LOAD_PT_DYNAMIC | GRUB_ELF_LOAD_FLAGS_28BITS, 0, 0);
f96e0b
       if (err)
f96e0b
 	return err;
f96e0b
       if (kernel_type != KERNEL_TYPE_OPENBSD)
f96e0b
diff --git a/grub-core/loader/i386/coreboot/chainloader.c b/grub-core/loader/i386/coreboot/chainloader.c
f96e0b
index df4e276..505aa0b 100644
f96e0b
--- a/grub-core/loader/i386/coreboot/chainloader.c
f96e0b
+++ b/grub-core/loader/i386/coreboot/chainloader.c
f96e0b
@@ -56,36 +56,13 @@ grub_chain_unload (void)
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
-grub_chain_elf32_hook (Elf32_Phdr * phdr, grub_addr_t * addr, int *do_load)
f96e0b
-{
f96e0b
-  grub_err_t err;
f96e0b
-  grub_relocator_chunk_t ch;
f96e0b
-
f96e0b
-  if (phdr->p_type != PT_LOAD)
f96e0b
-    {
f96e0b
-      *do_load = 0;
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-
f96e0b
-  *do_load = 1;
f96e0b
-  err = grub_relocator_alloc_chunk_addr (relocator, &ch,
f96e0b
-					 phdr->p_paddr, phdr->p_memsz);
f96e0b
-  if (err)
f96e0b
-    return err;
f96e0b
-
f96e0b
-  *addr = (grub_addr_t) get_virtual_current_address (ch);
f96e0b
-
f96e0b
-  return GRUB_ERR_NONE;
f96e0b
-}
f96e0b
-
f96e0b
-
f96e0b
-static grub_err_t
f96e0b
 grub_cmd_chain (grub_command_t cmd __attribute__ ((unused)),
f96e0b
 		int argc, char *argv[])
f96e0b
 {
f96e0b
   grub_err_t err;
f96e0b
   grub_file_t file;
f96e0b
   grub_elf_t elf;
f96e0b
+  Elf32_Phdr *phdr;
f96e0b
 
f96e0b
   if (argc != 1)
f96e0b
     return grub_error (GRUB_ERR_BAD_ARGUMENT, N_("filename expected"));
f96e0b
@@ -118,13 +95,47 @@ grub_cmd_chain (grub_command_t cmd __attribute__ ((unused)),
f96e0b
       grub_elf_close (elf);
f96e0b
     }
f96e0b
 
f96e0b
-  entry = elf->ehdr.ehdr32.e_entry & 0xFFFFFF;
f96e0b
-  
f96e0b
-  err = grub_elf32_load (elf, argv[0], grub_chain_elf32_hook, 0, 0);
f96e0b
+  entry = elf->ehdr.ehdr32.e_entry;
f96e0b
+
f96e0b
+  FOR_ELF32_PHDRS(elf, phdr)
f96e0b
+    {
f96e0b
+      grub_uint8_t *load_addr;
f96e0b
+      grub_relocator_chunk_t ch;
f96e0b
+
f96e0b
+      if (phdr->p_type != PT_LOAD)
f96e0b
+	continue;
f96e0b
+
f96e0b
+      err = grub_relocator_alloc_chunk_addr (relocator, &ch,
f96e0b
+					     phdr->p_paddr, phdr->p_memsz);
f96e0b
+      if (err)
f96e0b
+	break;
f96e0b
+
f96e0b
+      load_addr = get_virtual_current_address (ch);
f96e0b
+
f96e0b
+      if (grub_file_seek (elf->file, phdr->p_offset) == (grub_off_t) -1)
f96e0b
+	return grub_errno;
f96e0b
+
f96e0b
+      if (phdr->p_filesz)
f96e0b
+	{
f96e0b
+	  grub_ssize_t read;
f96e0b
+	  read = grub_file_read (elf->file, load_addr, phdr->p_filesz);
f96e0b
+	  if (read != (grub_ssize_t) phdr->p_filesz)
f96e0b
+	    {
f96e0b
+	      if (!grub_errno)
f96e0b
+		grub_error (GRUB_ERR_FILE_READ_ERROR, N_("premature end of file %s"),
f96e0b
+			    argv[0]);
f96e0b
+	      break;
f96e0b
+	    }
f96e0b
+	}
f96e0b
+
f96e0b
+      if (phdr->p_filesz < phdr->p_memsz)
f96e0b
+	grub_memset ((load_addr + phdr->p_filesz),
f96e0b
+		     0, phdr->p_memsz - phdr->p_filesz);
f96e0b
+    }
f96e0b
 
f96e0b
   grub_elf_close (elf);
f96e0b
-  if (err)
f96e0b
-    return err;
f96e0b
+  if (grub_errno)
f96e0b
+    return grub_errno;
f96e0b
 
f96e0b
   grub_loader_set (grub_chain_boot, grub_chain_unload, 0);
f96e0b
   return GRUB_ERR_NONE;
f96e0b
diff --git a/grub-core/loader/mips/linux.c b/grub-core/loader/mips/linux.c
f96e0b
index f2cf7cc..653f8a2 100644
f96e0b
--- a/grub-core/loader/mips/linux.c
f96e0b
+++ b/grub-core/loader/mips/linux.c
f96e0b
@@ -144,7 +144,7 @@ grub_linux_load32 (grub_elf_t elf, const char *filename,
f96e0b
   /* Linux's entry point incorrectly contains a virtual address.  */
f96e0b
   entry_addr = elf->ehdr.ehdr32.e_entry;
f96e0b
 
f96e0b
-  linux_size = grub_elf32_size (elf, filename, &base, 0);
f96e0b
+  linux_size = grub_elf32_size (elf, &base, 0);
f96e0b
   if (linux_size == 0)
f96e0b
     return grub_errno;
f96e0b
   target_addr = base;
f96e0b
@@ -171,22 +171,7 @@ grub_linux_load32 (grub_elf_t elf, const char *filename,
f96e0b
   *extra_mem = playground + extraoff;
f96e0b
 
f96e0b
   /* Now load the segments into the area we claimed.  */
f96e0b
-  auto grub_err_t offset_phdr (Elf32_Phdr *phdr, grub_addr_t *addr, int *do_load);
f96e0b
-  grub_err_t offset_phdr (Elf32_Phdr *phdr, grub_addr_t *addr, int *do_load)
f96e0b
-    {
f96e0b
-      if (phdr->p_type != PT_LOAD)
f96e0b
-	{
f96e0b
-	  *do_load = 0;
f96e0b
-	  return 0;
f96e0b
-	}
f96e0b
-      *do_load = 1;
f96e0b
-
f96e0b
-      /* Linux's program headers incorrectly contain virtual addresses.
f96e0b
-       * Translate those to physical, and offset to the area we claimed.  */
f96e0b
-      *addr = (grub_addr_t) (phdr->p_paddr - base + playground);
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-  return grub_elf32_load (elf, filename, offset_phdr, 0, 0);
f96e0b
+  return grub_elf32_load (elf, filename, playground - base, GRUB_ELF_LOAD_FLAGS_NONE, 0, 0);
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
@@ -200,7 +185,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename,
f96e0b
   /* Linux's entry point incorrectly contains a virtual address.  */
f96e0b
   entry_addr = elf->ehdr.ehdr64.e_entry;
f96e0b
 
f96e0b
-  linux_size = grub_elf64_size (elf, filename, &base, 0);
f96e0b
+  linux_size = grub_elf64_size (elf, &base, 0);
f96e0b
   if (linux_size == 0)
f96e0b
     return grub_errno;
f96e0b
   target_addr = base;
f96e0b
@@ -227,21 +212,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename,
f96e0b
   *extra_mem = playground + extraoff;
f96e0b
 
f96e0b
   /* Now load the segments into the area we claimed.  */
f96e0b
-  auto grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load);
f96e0b
-  grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load)
f96e0b
-    {
f96e0b
-      if (phdr->p_type != PT_LOAD)
f96e0b
-	{
f96e0b
-	  *do_load = 0;
f96e0b
-	  return 0;
f96e0b
-	}
f96e0b
-      *do_load = 1;
f96e0b
-      /* Linux's program headers incorrectly contain virtual addresses.
f96e0b
-       * Translate those to physical, and offset to the area we claimed.  */
f96e0b
-      *addr = (grub_addr_t) (phdr->p_paddr - base + playground);
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-  return grub_elf64_load (elf, filename, offset_phdr, 0, 0);
f96e0b
+  return grub_elf64_load (elf, filename, playground - base, GRUB_ELF_LOAD_FLAGS_NONE, 0, 0);
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
diff --git a/grub-core/loader/powerpc/ieee1275/linux.c b/grub-core/loader/powerpc/ieee1275/linux.c
f96e0b
index c977941..9055399 100644
f96e0b
--- a/grub-core/loader/powerpc/ieee1275/linux.c
f96e0b
+++ b/grub-core/loader/powerpc/ieee1275/linux.c
f96e0b
@@ -179,7 +179,7 @@ grub_linux_load32 (grub_elf_t elf, const char *filename)
f96e0b
   grub_uint32_t offset;
f96e0b
   Elf32_Addr entry;
f96e0b
 
f96e0b
-  linux_size = grub_elf32_size (elf, filename, &base_addr, &align);
f96e0b
+  linux_size = grub_elf32_size (elf, &base_addr, &align);
f96e0b
   if (linux_size == 0)
f96e0b
     return grub_errno;
f96e0b
   /* Pad it; the kernel scribbles over memory beyond its load address.  */
f96e0b
@@ -203,20 +203,7 @@ grub_linux_load32 (grub_elf_t elf, const char *filename)
f96e0b
   linux_addr = seg_addr;
f96e0b
 
f96e0b
   /* Now load the segments into the area we claimed.  */
f96e0b
-  auto grub_err_t offset_phdr (Elf32_Phdr *phdr, grub_addr_t *addr, int *do_load);
f96e0b
-  grub_err_t offset_phdr (Elf32_Phdr *phdr, grub_addr_t *addr, int *do_load)
f96e0b
-    {
f96e0b
-      if (phdr->p_type != PT_LOAD)
f96e0b
-	{
f96e0b
-	  *do_load = 0;
f96e0b
-	  return 0;
f96e0b
-	}
f96e0b
-      *do_load = 1;
f96e0b
-
f96e0b
-      *addr = (phdr->p_paddr - base_addr) + seg_addr;
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-  return grub_elf32_load (elf, filename, offset_phdr, 0, 0);
f96e0b
+  return grub_elf32_load (elf, filename, (void *) (seg_addr - base_addr), GRUB_ELF_LOAD_FLAGS_NONE, 0, 0);
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
@@ -228,7 +215,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename)
f96e0b
   grub_uint64_t offset;
f96e0b
   Elf64_Addr entry;
f96e0b
 
f96e0b
-  linux_size = grub_elf64_size (elf, filename, &base_addr, &align);
f96e0b
+  linux_size = grub_elf64_size (elf, &base_addr, &align);
f96e0b
   if (linux_size == 0)
f96e0b
     return grub_errno;
f96e0b
   /* Pad it; the kernel scribbles over memory beyond its load address.  */
f96e0b
@@ -250,20 +237,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename)
f96e0b
   linux_addr = seg_addr;
f96e0b
 
f96e0b
   /* Now load the segments into the area we claimed.  */
f96e0b
-  auto grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load);
f96e0b
-  grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load)
f96e0b
-    {
f96e0b
-      if (phdr->p_type != PT_LOAD)
f96e0b
-	{
f96e0b
-	  *do_load = 0;
f96e0b
-	  return 0;
f96e0b
-	}
f96e0b
-      *do_load = 1;
f96e0b
-
f96e0b
-      *addr = (phdr->p_paddr - base_addr) + seg_addr;
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-  return grub_elf64_load (elf, filename, offset_phdr, 0, 0);
f96e0b
+  return grub_elf64_load (elf, filename, (void *) (grub_addr_t) (seg_addr - base_addr), GRUB_ELF_LOAD_FLAGS_NONE, 0, 0);
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
diff --git a/grub-core/loader/sparc64/ieee1275/linux.c b/grub-core/loader/sparc64/ieee1275/linux.c
f96e0b
index c85fcfd..d203377 100644
f96e0b
--- a/grub-core/loader/sparc64/ieee1275/linux.c
f96e0b
+++ b/grub-core/loader/sparc64/ieee1275/linux.c
f96e0b
@@ -261,7 +261,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename)
f96e0b
   linux_entry = elf->ehdr.ehdr64.e_entry;
f96e0b
   linux_addr = 0x40004000;
f96e0b
   off = 0x4000;
f96e0b
-  linux_size = grub_elf64_size (elf, filename, 0, 0);
f96e0b
+  linux_size = grub_elf64_size (elf, 0, 0);
f96e0b
   if (linux_size == 0)
f96e0b
     return grub_errno;
f96e0b
 
f96e0b
@@ -286,21 +286,7 @@ grub_linux_load64 (grub_elf_t elf, const char *filename)
f96e0b
   base = linux_entry - off;
f96e0b
 
f96e0b
   /* Now load the segments into the area we claimed.  */
f96e0b
-  auto grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load);
f96e0b
-  grub_err_t offset_phdr (Elf64_Phdr *phdr, grub_addr_t *addr, int *do_load)
f96e0b
-    {
f96e0b
-      if (phdr->p_type != PT_LOAD)
f96e0b
-	{
f96e0b
-	  *do_load = 0;
f96e0b
-	  return 0;
f96e0b
-	}
f96e0b
-      *do_load = 1;
f96e0b
-
f96e0b
-      /* Adjust the program load address to linux_addr.  */
f96e0b
-      *addr = (phdr->p_paddr - base) + (linux_addr - off);
f96e0b
-      return 0;
f96e0b
-    }
f96e0b
-  return grub_elf64_load (elf, filename, offset_phdr, 0, 0);
f96e0b
+  return grub_elf64_load (elf, filename, (void *) (linux_addr - off - base), GRUB_ELF_LOAD_FLAGS_NONE, 0, 0);
f96e0b
 }
f96e0b
 
f96e0b
 static grub_err_t
f96e0b
diff --git a/include/grub/elfload.h b/include/grub/elfload.h
f96e0b
index d1a8d54..f854d0b 100644
f96e0b
--- a/include/grub/elfload.h
f96e0b
+++ b/include/grub/elfload.h
f96e0b
@@ -33,14 +33,10 @@ struct grub_elf_file
f96e0b
     Elf32_Ehdr ehdr32;
f96e0b
   } ehdr;
f96e0b
   void *phdrs;
f96e0b
+  char *filename;
f96e0b
 };
f96e0b
 typedef struct grub_elf_file *grub_elf_t;
f96e0b
 
f96e0b
-typedef grub_err_t (*grub_elf32_load_hook_t)
f96e0b
-  (Elf32_Phdr *phdr, grub_addr_t *addr, int *load);
f96e0b
-typedef grub_err_t (*grub_elf64_load_hook_t)
f96e0b
-  (Elf64_Phdr *phdr, grub_addr_t *addr, int *load);
f96e0b
-
f96e0b
 typedef int (*grub_elf32_phdr_iterate_hook_t)
f96e0b
   (grub_elf_t elf, Elf32_Phdr *phdr, void *arg);
f96e0b
 typedef int (*grub_elf64_phdr_iterate_hook_t)
f96e0b
@@ -52,26 +48,31 @@ grub_err_t grub_elf_close (grub_elf_t);
f96e0b
 
f96e0b
 int grub_elf_is_elf32 (grub_elf_t);
f96e0b
 grub_size_t grub_elf32_size (grub_elf_t,
f96e0b
-			     const char *filename,
f96e0b
 			     Elf32_Addr *, grub_uint32_t *);
f96e0b
+enum grub_elf_load_flags
f96e0b
+  {
f96e0b
+    GRUB_ELF_LOAD_FLAGS_NONE = 0,
f96e0b
+    GRUB_ELF_LOAD_FLAGS_LOAD_PT_DYNAMIC = 1,
f96e0b
+    GRUB_ELF_LOAD_FLAGS_28BITS = 2,
f96e0b
+  };
f96e0b
 grub_err_t grub_elf32_load (grub_elf_t, const char *filename,
f96e0b
-			    grub_elf32_load_hook_t, grub_addr_t *,
f96e0b
+			    void *load_offset, enum grub_elf_load_flags flags, grub_addr_t *,
f96e0b
 			    grub_size_t *);
f96e0b
 
f96e0b
 int grub_elf_is_elf64 (grub_elf_t);
f96e0b
 grub_size_t grub_elf64_size (grub_elf_t,
f96e0b
-			     const char *filename,
f96e0b
 			     Elf64_Addr *, grub_uint64_t *);
f96e0b
 grub_err_t grub_elf64_load (grub_elf_t, const char *filename,
f96e0b
-			    grub_elf64_load_hook_t, grub_addr_t *,
f96e0b
+			    void *load_offset, enum grub_elf_load_flags flags, grub_addr_t *,
f96e0b
 			    grub_size_t *);
f96e0b
-grub_err_t
f96e0b
-grub_elf32_phdr_iterate (grub_elf_t elf,
f96e0b
-			 const char *filename,
f96e0b
-			 grub_elf32_phdr_iterate_hook_t hook, void *hook_arg);
f96e0b
-grub_err_t
f96e0b
-grub_elf64_phdr_iterate (grub_elf_t elf,
f96e0b
-			 const char *filename,
f96e0b
-			 grub_elf64_phdr_iterate_hook_t hook, void *hook_arg);
f96e0b
+grub_err_t grub_elf32_load_phdrs (grub_elf_t elf);
f96e0b
+grub_err_t grub_elf64_load_phdrs (grub_elf_t elf);
f96e0b
+
f96e0b
+#define FOR_ELF32_PHDRS(elf, phdr) \
f96e0b
+  for (grub_elf32_load_phdrs (elf), phdr = elf->phdrs; \
f96e0b
+       phdr && phdr < (Elf32_Phdr *) elf->phdrs + elf->ehdr.ehdr32.e_phnum; phdr++)
f96e0b
+#define FOR_ELF64_PHDRS(elf, phdr) \
f96e0b
+  for (grub_elf64_load_phdrs (elf), phdr = elf->phdrs;		\
f96e0b
+       phdr && phdr < (Elf64_Phdr *) elf->phdrs + elf->ehdr.ehdr64.e_phnum; phdr++)
f96e0b
 
f96e0b
 #endif /* ! GRUB_ELFLOAD_HEADER */
f96e0b
-- 
f96e0b
1.8.2.1
f96e0b