[SeaBIOS] [PATCH] Seabios - read e820 table from qemu_cfg
kevin at koconnor.net
Sat Jan 30 04:35:08 CET 2010
On Fri, Jan 29, 2010 at 10:03:55AM +0100, Jes Sorensen wrote:
> On 01/28/10 05:39, Kevin O'Connor wrote:
> >The advantage being that it should be possible to write one set of
> >helper functions in both qemu and seabios that can then be used to
> >pass arbitrary content.
> The only issue here is that I designed the Seabios portion to not rely
> on the size of the struct, to avoid having to statically reserve it like
> in your example. Having the qemu_cfg_get_file() function return a
> pointer to a file descriptor and then have a qemu_cfg_read() helper that
> takes the descriptor as it's first argument would avoid this problem.
SeaBIOS already has a maximum size for the e820 map (32) - see
> >As a side note, it should probably do the e820 map check even for qemu
> >users (ie, not just kvm).
> Ah I didn't realize Seabios would try to use the fw_cfg interface if it
> wasn't running on top of QEMU. That would be good to do.
Your patch only used it for kvm. SeaBIOS will use fw_cfg on both qemu
More information about the SeaBIOS