Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update releasenotes for x86_64 ......................................................................
Documentation: Update releasenotes for x86_64
Change-Id: I7d8d39bde3b3364ff6ce93030aa2bab34598acd8 Signed-off-by: Patrick Rudolph siro@das-labor.org --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/44781/1
diff --git a/Documentation/releases/coreboot-4.13-relnotes.md b/Documentation/releases/coreboot-4.13-relnotes.md index dcc8bf4..626e9a1 100644 --- a/Documentation/releases/coreboot-4.13-relnotes.md +++ b/Documentation/releases/coreboot-4.13-relnotes.md @@ -49,4 +49,13 @@ CPU threads as possible limited only by SMRAM space and not by 64K. By default this loader version is disabled. Please see cpu/x86/Kconfig for more info.
+### Initial support for x86_64 support + +The x86_64 code support has been revived and enabled for qemu. While it started +as PoC and the only supported platform is an emulator, there's interrest in +enabling additional platforms. It would allow to access more than 4GiB of memory +at runtime and possibly brings optimised code for faster execution times. +It still needs changes in assembly, fixed integer to pointer conversions in C, +wrappers for BLOBs, support fpr running Option ROMs ... + ### Add significant changes here
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update releasenotes for x86_64 ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 52: ### Initial support for x86_64 support *support* is there twice.
Initial x86_64 support
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 55: interrest interest
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: fpr for
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: BLOBs blobs
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: ... Use ellipse character: ….
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update releasenotes for x86_64 ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/44781/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44781/1//COMMIT_MSG@7 PS1, Line 7: releasenotes The term is spelled with a space in the document *release notes*.
Patrick Rudolph has uploaded a new patch set (#2) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Documentation: Update release notes for x86_64
Change-Id: I7d8d39bde3b3364ff6ce93030aa2bab34598acd8 Signed-off-by: Patrick Rudolph siro@das-labor.org --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/44781/2
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Patch Set 1:
(6 comments)
https://review.coreboot.org/c/coreboot/+/44781/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44781/1//COMMIT_MSG@7 PS1, Line 7: releasenotes
The term is spelled with a space in the document *release notes*.
Done
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 52: ### Initial support for x86_64 support
*support* is there twice. […]
Done
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 55: interrest
interest
Done
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: fpr
for
Done
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: ...
Use ellipse character: ….
That's not in UTF-8. Which encoding is used in documentation?
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: BLOBs
blobs
Done
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Patch Set 2: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: ...
That's not in UTF-8. […]
Alternatively, use something like:
... running Option ROMs, among other things.
Hello build bot (Jenkins), Patrick Rudolph, Paul Menzel, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/44781
to look at the new patch set (#3).
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Documentation: Update release notes for x86_64
Change-Id: I7d8d39bde3b3364ff6ce93030aa2bab34598acd8 Signed-off-by: Patrick Rudolph siro@das-labor.org --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/44781/3
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... File Documentation/releases/coreboot-4.13-relnotes.md:
https://review.coreboot.org/c/coreboot/+/44781/1/Documentation/releases/core... PS1, Line 59: ...
Alternatively, use something like: […]
Done
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Patch Set 3: Code-Review+1
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Patch Set 3: Code-Review+2
Patrick Rudolph has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44781 )
Change subject: Documentation: Update release notes for x86_64 ......................................................................
Documentation: Update release notes for x86_64
Change-Id: I7d8d39bde3b3364ff6ce93030aa2bab34598acd8 Signed-off-by: Patrick Rudolph siro@das-labor.org Reviewed-on: https://review.coreboot.org/c/coreboot/+/44781 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Reviewed-by: Arthur Heymans arthur@aheymans.xyz --- M Documentation/releases/coreboot-4.13-relnotes.md 1 file changed, 9 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve 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 dcc8bf4..8d19067 100644 --- a/Documentation/releases/coreboot-4.13-relnotes.md +++ b/Documentation/releases/coreboot-4.13-relnotes.md @@ -49,4 +49,13 @@ CPU threads as possible limited only by SMRAM space and not by 64K. By default this loader version is disabled. Please see cpu/x86/Kconfig for more info.
+### Initial support for x86_64 + +The x86_64 code support has been revived and enabled for qemu. While it started +as PoC and the only supported platform is an emulator, there's interest in +enabling additional platforms. It would allow to access more than 4GiB of memory +at runtime and possibly brings optimised code for faster execution times. +It still needs changes in assembly, fixed integer to pointer conversions in C, +wrappers for blobs, support for running Option ROMs, among other things. + ### Add significant changes here
-
Angel Pons (Code Review) -
Arthur Heymans (Code Review)
-
Patrick Rudolph (Code Review)
-
Paul Menzel (Code Review)