Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41471 )
Change subject: soc/intel/broadwell: Mask lower 20 bits of TOLUD and TOLM in systemagent.asl ......................................................................
Patch Set 6:
Patch Set 6:
Patch Set 6:
Patch Set 6:
Why was this not fixed in the whole tree?
Was any platform missed? I had done a grep for TLUD in *.asl files:
Um, yes, `git grep TLUD reveals them. :)
Were there more platforms that required changes?
Haswell (which is most likely exactly the same as Broadwell in separate dirs) and Sandy Bridge at least.
Ah yes. I see those now. Pushed changes here: https://review.coreboot.org/c/coreboot/+/41976
I would like to have some conclusion about the ME range, though, before we continue. Should we drop that? It seems safe enough, to me...
I have pushed the changes for sandybridge and haswell since they fix a known issue. Let me revisit your comment on the ME range.