Peter Stuge wrote:
Hi,
On Sun, Jan 20, 2008 at 10:31:23PM +0100, Carl-Daniel Hailfinger wrote:
New plan:
Scratch all "special" LAR member stuff and simply add a normal file called "foo" or "placeholder" with the nocompress: parameter to the LAR. No new code needed, gives us almost everything we need right now.
What do you think?
I have been thinking about the problem but I can't come up with any really simple solution that is without drawbacks.
So far I still prefer making sure that coreboot does not look for files that do not exist, rather than adding special files. But there are problems with that approache too. We want the lar to be dynamic so things could change after coreboot has been built and I agree with Ron that missing files, though we should make sure it's not a common case, shall still be pretty efficient.
We should distinguish here between the files before and after raminit. I don't think we should have any false file lookups before we can start caching the rom.
I like the plan Carl-Daniel, it's a simple workaround that buys us time to think. Someone may come up with a new solution to all of this once flashrom can work with individual sectors.
//Peter