[SeaBIOS] [PATCH 2/2] accept MADT over fw_cfg
Michael S. Tsirkin
mst at redhat.com
Sun Apr 21 22:41:48 CEST 2013
On Sun, Mar 24, 2013 at 05:55:22PM -0400, Kevin O'Connor wrote:
> On Sun, Mar 24, 2013 at 09:45:54PM +0200, Michael S. Tsirkin wrote:
> > On Sun, Mar 24, 2013 at 03:21:08PM -0400, Kevin O'Connor wrote:
> > > On Sun, Mar 24, 2013 at 09:14:47PM +0200, Michael S. Tsirkin wrote:
> > > > I don't exactly understand what do you mean by "file".
> > >
> > > A fw_cfg entry added using QEMU's fw_cfg_add_file() instead of
> > > fw_cfg_add_bytes().
> >
> > Looks good overall. Some further proposals:
> >
> > 1. I think we need runtime patching so we'll need
> > another one that adds stuff from memory but yes it
> > can look like a file for seabios.
> > 2. paths exposed to seabios are relative at the moment.
> > So "acpi/*" ?
>
> fw_cfg_add_file() does read from memory. It's not tied to the host
> filesystem or guest filesystem.
>
> > 3. add option to disable builtin tables at compile time
>
> Agreed.
>
> > 4. to keep the ability to develop seabios code
> > in-tree, make it depend on compile option
> > disabled by default.
> >
> > 5. similarly in QEMU, put code there disabled first,
> > so developers can play with it.
> >
> > 6. when all is ready, flip the switch to enable in both places.
>
> Sure - we can do that if needed.
>
> -Kevin
Okay I'm pretty close to posting some patches
that advance this project further, but wanted to
check something beforehand: there are several tables
that point to other tables (for example: FADT points
to DSDT). What I did is provide a list of fixups
such that bios can patch in pointers without
any need to understand what's what.
Thoughts?
--
MST
More information about the SeaBIOS
mailing list