Patch Set #7, Line 27:
This is not sufficient. When CAR is active it's microarchitecturally defined how a clflush is handled in CAR mode. You need a way for the chipset to indicate this is valid sequence that can be performed.
Aren't there defines for these bits instead of open coding them?
Patch Set #7, Line 60:
Why is a cbmem check in here? It doesn't immediately make sense why this would be the case. Likewise, this function is also active when not loading postcar, e.g. FSP. What about that? Or are you using cbmem to be an indirect proxy for postcar loading?
To view, visit change 37196. To unsubscribe, or for help writing mail filters, visit settings.