Hi,
On Wed, Sep 17, 2008 at 05:05:03PM +0200, Stefan Reinauer wrote:
I have a bit of a hick-up with removing the coreboot table, as this renders utility like flashrom (in some cases) and nvramtool (in all cases) pretty much unusable.
I don't propose removing the coreboot table. In most cases it is highly desireable, but in a few ones it's not that useful and is just taking space. A build option that defaults to 'yes' sounds like a sane way to support all use cases.
Note that the code removed by my last patch doesn't disable any functionality, it just stops passing a pointer around which wasn't being used for anything.
Also, I think bootloader stuff is libpayload/filo/grub2 territory.
How do you define "bootloader stuff"? coreboot looks a lot like a simplified bootloader to me. And it does support loading things that are usually loaded by GRUB, like Invaders, Memtest86 or even Linux.
Glad to see your patch is able to reduce the code size though; even if it comes at some cost.
Indeed, things come at a price. Such cost would be bearable for some and unacceptable for others. I really think having the user evaluate it is the best way to serve everyone.
Hey, at least your users are system builders and have a clue on what they're doing ;-)