From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1LdQig-0002dX-6E for qemu-devel@nongnu.org; Sat, 28 Feb 2009 10:03:14 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1LdQib-0002ZM-Jq for qemu-devel@nongnu.org; Sat, 28 Feb 2009 10:03:13 -0500 Received: from [199.232.76.173] (port=38665 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1LdQib-0002ZJ-Hj for qemu-devel@nongnu.org; Sat, 28 Feb 2009 10:03:09 -0500 Received: from cantor2.suse.de ([195.135.220.15]:50629 helo=mx2.suse.de) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1LdQia-00033o-P0 for qemu-devel@nongnu.org; Sat, 28 Feb 2009 10:03:09 -0500 From: Alexander Graf Date: Sat, 28 Feb 2009 16:03:06 +0100 Message-Id: <1235833386-30272-1-git-send-email-agraf@suse.de> Subject: [Qemu-devel] [PATCH] Make the ELF loader aware of backwards compatibility Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Alexander Graf Most 64 bit architectures I'm aware of support running 32 bit code of the same architecture as well. So x86_64 can run i386 code easily and ppc64 can run ppc code. Unfortunately, the current checks are pretty strict. So you can only load e.g. an x86_64 elf binary on qemu-system-x86_64, but no i386 one. This can get really annoying. I first encountered this issue with my multiboot patch, where qemu-system-x86_64 was unable to load an i386 elf binary because the elf loader rejected it. The same thing happened again on PPC64 now. The firmware we're loading is a PPC32 elf binary, as it's shared with PPC32. But the platform is PPC64. Right now there is a hack for this in the ppc cpu.h definition, that simply sets the type to PPC32 in system emulation mode. While that works fine for the firmware, it's no good if you also want to load a PPC64 kernel with -kernel. So in order to solve this mess, I figured the easiest way is to make the elf loader aware of platforms that are backwards compatible. For now I was only sure that x86_64 does i386 and ppc64 does ppc32, but maybe there are other combinations too. This patch is a prerequisite for having a working -kernel option on PPC64. Signed-off-by: Alexander Graf --- elf_ops.h | 17 +++++++++++++++-- target-ppc/cpu.h | 2 +- 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/elf_ops.h b/elf_ops.h index feea12f..485442a 100644 --- a/elf_ops.h +++ b/elf_ops.h @@ -194,8 +194,21 @@ static int glue(load_elf, SZ)(int fd, int64_t address_offset, glue(bswap_ehdr, SZ)(&ehdr); } - if (ELF_MACHINE != ehdr.e_machine) - goto fail; + switch (ELF_MACHINE) { + case EM_PPC64: + if (EM_PPC64 != ehdr.e_machine) + if (EM_PPC != ehdr.e_machine) + goto fail; + break; + case EM_X86_64: + if (EM_X86_64 != ehdr.e_machine) + if (EM_386 != ehdr.e_machine) + goto fail; + break; + default: + if (ELF_MACHINE != ehdr.e_machine) + goto fail; + } if (pentry) *pentry = (uint64_t)(elf_sword)ehdr.e_entry; diff --git a/target-ppc/cpu.h b/target-ppc/cpu.h index 80ee76c..4fc06c3 100644 --- a/target-ppc/cpu.h +++ b/target-ppc/cpu.h @@ -67,7 +67,7 @@ #define TARGET_HAS_ICE 1 /* Load a 32 bit BIOS also on 64 bit machines */ -#if defined (TARGET_PPC64) && defined(CONFIG_USER_ONLY) +#if defined (TARGET_PPC64) #define ELF_MACHINE EM_PPC64 #else #define ELF_MACHINE EM_PPC -- 1.5.3.1