On 1 February 2012 01:10, Alexander Graf <ag...@suse.de> wrote: > On 01.02.2012, at 01:11, Grant Likely wrote: >> @@ -312,7 +397,6 @@ void arm_load_kernel(CPUState *env, struct arm_boot_info >> *info) >> if (info->nb_cpus > 1) { >> info->write_secondary_boot(env, info); >> } >> - info->initrd_size = initrd_size; > > This seems to be an unrelated change. Any reason for moving that line up?
I've just discovered the reason for this, which is that load_dtb() reads info->initrd_size, so without this change it all works fine except that we silently don't pass the initrd to the kernel :-) -- PMM