[Qemu-devel] Re: [PATCH 0/2][RFC][PPC] openbios support

[Laurent Vivier]

Le 19 déc. 08 à 18:50, Blue Swirl a écrit :

> On 12/19/08, address@hidden <address@hidden> wrote:
> > > On Fri, Dec 19, 2008 at 09:54:54AM +0100, address@hidden  
> > > wrote:
> > > > > Laurent Vivier a écrit :
> > > > > > These two patches allows to load an openbios elf image instead  
> > > > > > of an
> > > > > > openhackware image.
> > > > > >
> > > > > > [PATCH 1/2][RFC] Modify hw/ppc_oldword.c to use qemu_ram_alloc().
> > > > > > [PATCH 2/2][RFC] Load an OpenBios ELF image instead of  
> > > > > > OpenHackware
> > > binary
> > > > > Great job! We are closer to get rid of OpenHackware.
> > > > >
> > > > > However I have tried those patches with the latest SVN from  
> > > > > OpenBios,
> > > > > and I am unable to get the machine booting. Do we need other  
> > > > > patches for
> > > > > OpenBios or QEMU? It would be nice to make them available, even  
> > > > > if they
> > > > > are not ready for merge.
> > > >
> > > > Yes, we need more patches:
> > > >
> > > > - for qemu, an nvram cleanup I already sent some month ago, and a  
> > > > little PCI
> > > cleanup patch.
> > >
> > > Could you please resent it? I will apply it if everything is ok.
> >
> >
> > Here it is.
> >
> > Note that the NVRAM is stored into a file.
>
> One problem with this is that the file is written to current
> directory, so running Qemu leaves  these files everywhere. Maybe the
> user should explicitly request using a NVRAM file and with that also
> specify the location of it?

Yes, I think it's a good idea, if no parameters is given the nvram is  
only stored in memory. I rework this patch

Thank you,
Laurent
----------------------- Laurent Vivier ----------------------
"The best way to predict the future is to invent it."
- Alan Kay