On Mon, Feb 08, 2010 at 11:31:40AM +0100, Jes Sorensen wrote:
On 01/28/10 05:39, Kevin O'Connor wrote:
As a side note, it should probably do the e820 map check even for qemu users (ie, not just kvm).
Hi Kevin,
Here is an updated version of the patch which does the e820 read unconditionally of the return from kvm_para_available() so it should work for coreboot too.
I haven't touched the file descriptor issue as I find it's a different discussion.
I'd prefer to use the "file" method - but I wont hold up your patch for it. If the host part of your patch is committed to qemu, I'll commit the SeaBIOS part.
[...]
+struct e820_entry {
- u64 address;
- u64 length;
- u32 type;
+};
I find this struct to be easily confused with 'struct e820entry' in memmap.h. Both code should use the same struct, or the new struct should clearly indicate it's for qemu (eg, "qemu_e820_entry").
Thanks, -Kevin