Attention is currently required from: Johannes Hahn, Mario Scheithauer, Werner Zeh.
Uwe Poeche has posted comments on this change by Johannes Hahn. ( https://review.coreboot.org/c/coreboot/+/85606?usp=email )
Change subject: soc/intel/common/block/power_limit: Disable RAPL via MSR completely
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
Patchset:
PS3:
I started our ci (with activated apl1/4 and ehl1 mainboards) with that patch on top and checked on ehl1 MSR 0x610 in OS before and after the patch. Result is as regarded. The effort to set ehl1 to power limit and also verify that MSR 0x196 Bit 10 was set before and after the patch not set was interrupted.
The error which shows siemens-bot via his submit is caused by the known problematic upstream patch CB:84833 and has no correlation with the patch here.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/85606?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I8272339a991667d5ba177f4755ec40e1961d729e
Gerrit-Change-Number: 85606
Gerrit-PatchSet: 3
Gerrit-Owner: Johannes Hahn
johannes-hahn@siemens.com
Gerrit-Reviewer: Mario Scheithauer
mario.scheithauer@siemens.com
Gerrit-Reviewer: Uwe Poeche
uwe.poeche@siemens.com
Gerrit-Reviewer: Werner Zeh
werner.zeh@siemens.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: siemens-bot
Gerrit-Attention: Mario Scheithauer
mario.scheithauer@siemens.com
Gerrit-Attention: Johannes Hahn
johannes-hahn@siemens.com
Gerrit-Attention: Werner Zeh
werner.zeh@siemens.com
Gerrit-Comment-Date: Tue, 17 Dec 2024 10:21:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes