Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43494 )
Change subject: soc/intel/common/cpu: Update COS mask calculation for NEM enhanced mode
......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/43494/3//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/43494/3//COMMIT_MSG@23
PS3, Line 23: mas
> maps
Done
https://review.coreboot.org/c/coreboot/+/43494/3//COMMIT_MSG@20
PS3, Line 20:
: Also the COS mask selection is mapped to bit 32:33 of MSR
: IA32_PQR_ASSOC(0xC8F) and need to be updated in edx(maps 63:32) before
: MSR write instead of eax(mas 31:0). This implementation corrects that
: as well.
> yes, MSR definition is not documented for TGL, CML, KBL. We can get it revised.
Done
https://review.coreboot.org/c/coreboot/+/43494/3/src/soc/intel/common/block…
File src/soc/intel/common/block/cpu/car/cache_as_ram.S:
PS3:
> yes, we configure the mask2 with the RW data mapped ways configured for eviction and start filling i […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/43494
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I54e047161853bfc70516c1d607aa479e68836d04
Gerrit-Change-Number: 43494
Gerrit-PatchSet: 7
Gerrit-Owner: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Shreesh Chhabbi <shreesh.chhabbi(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Shreesh Chhabbi <shreesh.chhabbi(a)intel.com>
Gerrit-Comment-Date: Wed, 09 Sep 2020 03:05:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Comment-In-Reply-To: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-MessageType: comment
Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43494 )
Change subject: soc/intel/common/cpu: Update COS mask calculation for NEM enhanced mode
......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/43494/1/src/soc/intel/common/block…
File src/soc/intel/common/block/cpu/car/cache_as_ram.S:
https://review.coreboot.org/c/coreboot/+/43494/1/src/soc/intel/common/block…
PS1, Line 371: * Data Size (RW) : Up to 256K
> Yes..however way size depends on the total LLC and number of ways that a platform supports. […]
Done
https://review.coreboot.org/c/coreboot/+/43494/1/src/soc/intel/common/block…
PS1, Line 429: movl $0x02, %edx
> COS is mapped to 32:33 of the MSR 0xCF8 , that is why assigning it to EDX.
Done
https://review.coreboot.org/c/coreboot/+/43494/1/src/soc/intel/common/block…
PS1, Line 445: movl $0x01, %edx
> same above
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/43494
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I54e047161853bfc70516c1d607aa479e68836d04
Gerrit-Change-Number: 43494
Gerrit-PatchSet: 7
Gerrit-Owner: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Shreesh Chhabbi <shreesh.chhabbi(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Shreesh Chhabbi <shreesh.chhabbi(a)intel.com>
Gerrit-Comment-Date: Wed, 09 Sep 2020 03:04:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subrata.banik(a)intel.com>
Comment-In-Reply-To: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-MessageType: comment
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35508 )
Change subject: trogdor: SoC makefile blob support
......................................................................
Patch Set 92:
Ravi, can you please rebase the whole patch train once more (to pick up CB:45057)? With that, fingers crossed, Jenkins should hopefully set Verified+1 here.
--
To view, visit https://review.coreboot.org/c/coreboot/+/35508
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I85a20ef31ec91c6f22221d16fd4c3097c5cb97d1
Gerrit-Change-Number: 35508
Gerrit-PatchSet: 92
Gerrit-Owner: mturney mturney <mturney(a)codeaurora.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Ravi kumar <rbokka(a)codeaurora.org>
Gerrit-Reviewer: ashk(a)codeaurora.org
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: mturney mturney <mturney(a)codeaurora.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Ravi Kumar Bokka <c_rbokka(a)qualcomm.corp-partner.google.com>
Gerrit-CC: Saurabh Gorecha <sgorecha(a)codeaurora.org>
Gerrit-Comment-Date: Wed, 09 Sep 2020 00:03:08 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello Patrick Georgi,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/45057
to review the following change.
Change subject: 3rdparty/qc_blobs: Uprev to new HEAD (c564412)
......................................................................
3rdparty/qc_blobs: Uprev to new HEAD (c564412)
Signed-off-by: Julius Werner <jwerner(a)chromium.org>
Change-Id: I2de0c13000e5b1e32e9c1a6de3daa09acf6c321b
---
M 3rdparty/qc_blobs
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/45057/1
diff --git a/3rdparty/qc_blobs b/3rdparty/qc_blobs
index 126fef6..c564412 160000
--- a/3rdparty/qc_blobs
+++ b/3rdparty/qc_blobs
@@ -1 +1 @@
-Subproject commit 126fef6b996237403039aa603945fc4caa75c8d6
+Subproject commit c56441253e7a6e3d74226aafdf307b8882484ebd
--
To view, visit https://review.coreboot.org/c/coreboot/+/45057
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2de0c13000e5b1e32e9c1a6de3daa09acf6c321b
Gerrit-Change-Number: 45057
Gerrit-PatchSet: 1
Gerrit-Owner: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-MessageType: newchange
Hello Jason Glenesk,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/45114
to review the following change.
Change subject: vc/amd/fsp/picasso: Add a GUID for reporting NB IOAPIC base
......................................................................
vc/amd/fsp/picasso: Add a GUID for reporting NB IOAPIC base
The GUID will be used in an upcoming patch to identify a HOB.
BUG=b:167421913, b:166519072
TEST=none
BRANCH=zork
Signed-off-by: Jason Glenesk <jason.glenesk(a)amd.corp-partner.google.com>
Signed-off-by: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Change-Id: I499e0ef801e4e98a63510a326d1134dd98305238
---
M src/vendorcode/amd/fsp/picasso/FspGuids.h
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/45114/1
diff --git a/src/vendorcode/amd/fsp/picasso/FspGuids.h b/src/vendorcode/amd/fsp/picasso/FspGuids.h
index 70bbe74..277fce1 100644
--- a/src/vendorcode/amd/fsp/picasso/FspGuids.h
+++ b/src/vendorcode/amd/fsp/picasso/FspGuids.h
@@ -33,4 +33,8 @@
GUID_INIT(0xf2784616, 0xb9bf, 0x4e1e, \
0x99, 0xe0, 0x96, 0x26, 0xda, 0x7e, 0xa5, 0xf5)
+#define AMD_FSP_NB_IOAPIC_BASE_HOB_GUID \
+ GUID_INIT(0xdd9738b3, 0x679e, 0x4e19, \
+ 0x80, 0xd1, 0xe4, 0x99, 0xed, 0x0a, 0x6d, 0xa1)
+
#endif /* __FSP_GUIDS__ */
--
To view, visit https://review.coreboot.org/c/coreboot/+/45114
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I499e0ef801e4e98a63510a326d1134dd98305238
Gerrit-Change-Number: 45114
Gerrit-PatchSet: 1
Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)amd.corp-partner.google.com>
Gerrit-MessageType: newchange