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

[Peter Stuge]

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

This still holds.

//Peter