On Mon, Nov 07, 2011 at 02:46:47PM -0800, Michael Eager wrote:
The declaration of clk_setup is missing in
qemu/hw/virtex_ml507.c:
static void virtex_init(ram_addr_t ram_size,
const char *boot_device,
const char *kernel_filename,
const char *kernel_cmdline,
const char *initrd_filename, const char *cpu_model)
{
...
<no decl for clk_setup>
...
memset(clk_setup, 0, sizeof(clk_setup));
This SEGVs because clk_setup is an inline function defined in ppc.h.
(I presume that the linker generates an out-of-line copy.)
It isn't clear what the declaration should be. In ppc405_uc.c, there
is a decl:
clk_setup_t clk_setup[PPC405EP_CLK_NB];
Hi
the following patch seems to work on my side. Fabien, could
you please see if this was your intention? It seems to be
commit ddd1055b07fdfe488a22c2275adaca75f4206d30 that
introduced the segfault.
Cheers
commit 8e95771e4afb6e91c30a53943118afef4631b919
Author: Edgar E. Iglesias<address@hidden>
Date: Tue Nov 8 00:00:55 2011 +0100
virtex: Remove memset of clk_setup
clk_setup is now a function.
Signed-off-by: Edgar E. Iglesias<address@hidden>
diff --git a/hw/virtex_ml507.c b/hw/virtex_ml507.c
index d31a204..5ea0e60 100644
--- a/hw/virtex_ml507.c
+++ b/hw/virtex_ml507.c
@@ -202,7 +202,6 @@ static void virtex_init(ram_addr_t ram_size,
cpu_model = "440-Xilinx";
}
- memset(clk_setup, 0, sizeof(clk_setup));
env = ppc440_init_xilinx(&ram_size, 1, cpu_model, 400000000);
qemu_register_reset(main_cpu_reset, env);