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().
> 
> -Kevin

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/*" ?

3. add option to disable builtin tables at compile time

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.


_______________________________________________
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios

Reply via email to