Patch Set 4:

My test results for KBL:
memsetting 1GiB of memory with different MTRR settings in NEM right after raminit:

do_fsp_post_memory_init: took 9180 msec to clear 1GiB UC
do_fsp_post_memory_init: took 9179 msec to clear 1GiB WPROT
do_fsp_post_memory_init: took 39 msec to clear 1GiB WRCOMB
do_fsp_post_memory_init: took 76 msec to clear 1GiB WRBACK


Appreciate your help Patrick for the data using this CL and analysis.

The use of WRBACK in NEM causes postcar stage loading to fail.

yes, me and Aaron are debugging this issue with cache architect on CML (behavior should be same across), the symptoms that we are seeing is that "invd" is clearing LLC cache line completely when marked as WB but WP/WC doesn't create problem although "invd" should clear entire LLC cache line. As per our understanding this is expected behavior of WB cache vs other type. Hence we are working on alternative ways to achieve the same. We have some sight but still under debug.

The use of WRCOMB seems fine, it still boots.

yes, WC and WP will work for sure. verified that part.

View Change

To view, visit change 34805. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iddafb573afb4799de64754a94816d7f3f2f4982f
Gerrit-Change-Number: 34805
Gerrit-PatchSet: 4
Gerrit-Owner: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: V Sowmya <v.sowmya@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-CC: Patrick Rudolph <siro@das-labor.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 15 Aug 2019 06:46:53 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment