loader: Allow ELF loader to auto-detect the ELF arch
authorAlistair Francis <alistair.francis@xilinx.com>
Thu, 22 Sep 2016 17:13:07 +0000 (18:13 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Thu, 22 Sep 2016 17:13:07 +0000 (18:13 +0100)
If the caller didn't specify an architecture for the ELF machine
the load_elf() function will auto detect it based on the ELF file.

Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: f2d70b47fcad31445f947f8817a0e146d80a046b.1474331683.git.alistair.francis@xilinx.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
include/hw/elf_ops.h
include/hw/loader.h

index f510e7ec2a1c6af234ec0cc801f856dd483f9017..5038c7f058bad43c15e4dee519855f2a34ff53dd 100644 (file)
@@ -280,6 +280,11 @@ static int glue(load_elf, SZ)(const char *name, int fd,
         glue(bswap_ehdr, SZ)(&ehdr);
     }
 
+    if (elf_machine <= EM_NONE) {
+        /* The caller didn't specify an ARCH, we can figure it out */
+        elf_machine = ehdr.e_machine;
+    }
+
     switch (elf_machine) {
         case EM_PPC64:
             if (ehdr.e_machine != EM_PPC64) {
index 4879b63a2f550a52a39a730886453d26b8b1549d..c59673dfc559c79f6a67f74f4312bcdff45e3e61 100644 (file)
@@ -68,6 +68,8 @@ const char *load_elf_strerror(int error);
  * load will fail if the target ELF does not match. Some architectures
  * have some architecture-specific behaviours that come into effect when
  * their particular values for @elf_machine are set.
+ * If @elf_machine is EM_NONE then the machine type will be read from the
+ * ELF header and no checks will be carried out against the machine type.
  */
 
 int load_elf(const char *filename, uint64_t (*translate_fn)(void *, uint64_t),