On Thu, 2013-02-07 at 20:48 -0500, Kevin O'Connor wrote:
Kconfig - I'm not sure we should have a qemu vs coreboot vs csm selection as it's possible we can have qemu+csm (or even qemu+coreboot+csm) and I think Kconfig should reflect that reality. I'll put together a patch to demonstrate what I'm thinking.
CSM is theoretically supposed to be hardware-agnostic, so I was happy with it being orthogonal to QEMU. Wasn't that fairly much the idea with COREBOOT vs. QEMU too? It's more about "direct/native boot on QEMU" rather than "running on QEMU at all". Which is why the virtio drivers shouldn't depend on QEMU alone.
You could perhaps introduce a separate config option for 'running on QEMU at all', and *that* one wouldn't in part of the choice.
pmtimer - the patches are fine - the coreboot/xen parts just need to be tested before merging. I may be able to test the coreboot part on my e350m1.
UmbStart/UmbEnd - I'd like to hear back from the OVMF team before merging this.
Yes, that makes sense. I've chased them up. Not really the OVMF team but the people who own the *spec* within Intel. Once the spec is fixed, OVMF can happily implement it.
unaligned PCIR - I'm not sure we can merge this as it would break QEMU users with a current "lgpl vgabios".
That's why I made it optional ;)