Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Doc/relnotes/4.13: Add several relevant changes
While some of these have little impact, they are worth mentioning here.
Change-Id: Idbf629ae77b8918ff1d93edb7b6c4669bbbe17df Signed-off-by: Angel Pons th3fanbus@gmail.com --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/47444/1
diff --git a/Documentation/releases/coreboot-4.13-relnotes.md b/Documentation/releases/coreboot-4.13-relnotes.md index 8d19067..efb45b0 100644 --- a/Documentation/releases/coreboot-4.13-relnotes.md +++ b/Documentation/releases/coreboot-4.13-relnotes.md @@ -13,6 +13,27 @@ Significant changes -------------------
+### Native refcode implementation for Bay Trail + +Bay Trail no longer needs a refcode binary to function properly. The refcode +was reimplemented as coreboot code, which should be functionally equivalent. +Thus, MRC.bin is now the only required Bay Trail binary executed by coreboot. + +### Unusual config files to build test more code + +There's some new highly-unusual config files, whose only purpose is to coerce +Jenkins into build-testing several disabled-by-default coreboot config options. +This prevents them from silently decaying over time because of build failures. + +### Initial support for Intel Trusted eXecution Technology + +coreboot now supports enabling Intel TXT. Though it's not feature-complete yet, +the code allows successfully launching tboot, a Measured Launch Environment. It +was tested on Haswell using an Asrock B85M Pro4 mainboard with TPM 2.0 on LPC. +Support for other platforms is still work in progress, but is being worked on. +The Haswell MRC.bin needs to be patched so as to enable DPR. The only winning +move is not to play with the MRC.bin, and instead write a replacement for it. + ### Hidden PCI devices
This new functionality takes advantage of the existing 'hidden' keyword in the
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 1: Code-Review+2
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 20: Thus, MRC.bin is now the only required Bay Trail binary executed by coreboot. I'm not a native English speaker but this phrase looks a bit weird to me. Maybe: "Thus, the mrc.bin is now the only blob coreboot has to execute to initialize the Bay Trail platform."
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 33: still work in progress, but is being worked on That is more or less saying the same thing twice?
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 33: still work in progress, but is being worked on
That is more or less saying the same thing twice?
Uh, yes.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 1:
(1 comment)
No idea, if the change-sets could be referenced, but probably they can also be found with not too much work.
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 34: The only winning : move Winning move for what? Avoid patching?
Hello build bot (Jenkins), Patrick Georgi, Stefan Reinauer, Arthur Heymans,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/47444
to look at the new patch set (#2).
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Doc/relnotes/4.13: Add several relevant changes
While some of these have little impact, they are worth mentioning here.
Change-Id: Idbf629ae77b8918ff1d93edb7b6c4669bbbe17df Signed-off-by: Angel Pons th3fanbus@gmail.com --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/47444/2
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 33: still work in progress, but is being worked on
Uh, yes.
Done
https://review.coreboot.org/c/coreboot/+/47444/1/Documentation/releases/core... PS1, Line 34: The only winning : move
Winning move for what? Avoid patching?
Ack
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 2: Code-Review+2
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47444/2/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/47444/2/Documentation/releases/core... PS2, Line 61: https://review.coreboot.org/plugins/gitiles/coreboot/+/refs/heads/master/uti... unrelated: this link does not work.
Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47444 )
Change subject: Doc/relnotes/4.13: Add several relevant changes ......................................................................
Doc/relnotes/4.13: Add several relevant changes
While some of these have little impact, they are worth mentioning here.
Change-Id: Idbf629ae77b8918ff1d93edb7b6c4669bbbe17df Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/47444 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Arthur Heymans arthur@aheymans.xyz --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 21 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved
diff --git a/Documentation/releases/coreboot-4.13-relnotes.md b/Documentation/releases/coreboot-4.13-relnotes.md index ddbea81..139fa20 100644 --- a/Documentation/releases/coreboot-4.13-relnotes.md +++ b/Documentation/releases/coreboot-4.13-relnotes.md @@ -13,6 +13,27 @@ Significant changes -------------------
+### Native refcode implementation for Bay Trail + +Bay Trail no longer needs a refcode binary to function properly. The refcode +was reimplemented as coreboot code, which should be functionally equivalent. +Thus, coreboot only needs to run the MRC.bin to successfully boot Bay Trail. + +### Unusual config files to build test more code + +There's some new highly-unusual config files, whose only purpose is to coerce +Jenkins into build-testing several disabled-by-default coreboot config options. +This prevents them from silently decaying over time because of build failures. + +### Initial support for Intel Trusted eXecution Technology + +coreboot now supports enabling Intel TXT. Though it's not feature-complete yet, +the code allows successfully launching tboot, a Measured Launch Environment. It +was tested on Haswell using an Asrock B85M Pro4 mainboard with TPM 2.0 on LPC. +Though support for other platforms is still not ready, it is being worked on. +The Haswell MRC.bin needs to be patched so as to enable DPR. Given that the MRC +binary cannot be redistributed, the best long-term solution is to replace it. + ### Hidden PCI devices
This new functionality takes advantage of the existing 'hidden' keyword in the
-
Angel Pons (Code Review) -
Arthur Heymans (Code Review)
-
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)