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 <[email protected]>

This still holds.


//Peter

--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to