[coreboot] [PATCH] Warn if we run out of MTRRs

Peter Stuge peter at stuge.se
Thu Feb 12 17:24:14 CET 2009


Carl-Daniel Hailfinger wrote:
> > Would it work to add a third negative MTRR for that region?
> 
> It would be two negative MTRRs in that case.

Yes. And the new one would be the third.


> > If yes, that's what I would like, and then I say to Stefan's patch:
> 
> You do realize that solving this problem means either adding another
> special case for SMM or discarding Stefan's patch?

The former. KISS.


Stefan Reinauer wrote:
> > Would it work to add a third negative MTRR for that region?
> 
> Yes, that is also what other BIOSes do.

Go for it.


> > Acked-by: Peter Stuge <peter at stuge.se>

This still holds.


//Peter




More information about the coreboot mailing list