Re: [coreboot] [PATCH] Multiboot

[Carl-Daniel Hailfinger]

On 18.09.2008 23:59, Carl-Daniel Hailfinger wrote:
> On 18.09.2008 22:09, Robert Millan wrote:
> > --- arch/x86/stage1.c	(revision 867)
> > +++ arch/x86/stage1.c	(working copy)
> > @@ -139,6 +140,14 @@
> >  }
> >  #endif /* CONFIG_PAYLOAD_ELF_LOADER */
> >
> > +int run_address_multiboot(void *f)
> > +{
> > int ret;
> > __asm__ __volatile__ ("call *%3" : "=a" (ret) : "a" (MB_MAGIC2), "b" (0xf0000), "c" (f));
> > return ret;
> >
> > +}
>
> The function above belongs in multiboot.c

I forgot to mention that the clobber list is incomplete, possibly
leading to corruption if a payload returns.

Regards,
Carl-Daniel