Zoltan Baldaszti has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: added more payload links and descriptions ......................................................................
Documentation: added more payload links and descriptions
Change-Id: I13c10edbc7fbb7017641ab025982c658cbeb3610 Signed-off-by: Zoltan Baldaszti bztemail@gmail.com --- M Documentation/payloads.md 1 file changed, 35 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/45829/1
diff --git a/Documentation/payloads.md b/Documentation/payloads.md index eee841e..bee8393 100644 --- a/Documentation/payloads.md +++ b/Documentation/payloads.md @@ -23,17 +23,45 @@ coreboot payload, but these days this function is available directly in the CorebootPayloadPkg part of its source tree.
+## iPXE + +[iPXE](https://ipxe.org) is the leading open source network boot firmware. +It provides a full PXE Implementation enchaced with additional features such +as booting from HTTP, iSCSI, FCoE, AoE, wireless, WAN, infiniband. + ## GRUB2
-GRUB2 was originally written as a bootloader and that's its most popular -purpose, but it can also be compiled as a coreboot payload. +[GRUB2](https://www.gnu.org/software/grub) was originally written as a +bootloader and that's its most popular purpose, but it can also be compiled +as a coreboot payload. + +## U-Boot + +[U-Boot](https://www.denx.de/wiki/U-Boot) is a boot loader for Embedded boards +based on PowerPC, ARM, MIPS and several other processors, which can be +installed in a boot ROM and used to initialize and test the hardware or to +download and run application code. + +## FILO + +[FILO](https://www.coreboot.org/FILO) loads boot images from local filesystem, +without help from legacy BIOS services. Expected usage is to flash it into +the BIOS ROM together with coreboot. + +## BOOTBOOT + +[BOOTBOOT](https://gitlab.com/bztsrc/bootboot) is a multi-platform, +architecture agnostic boot protocol. The protocol describes how to boot an +ELF64 or PE32+ executable inside an initial ram disk image into clean 64 bit +mode. This version uses libpayload to do that. Initrd can be in ROM or on +disk a GPT partition or a file on a FAT formatted ESP partition.
## Linux
-There are several projects using Linux as a payload (which was the -configuration that gave coreboot its original name, LinuxBIOS). That kernel is -often rather small and serves to load a current kernel from somewhere, e.g. -disk or network, and run that through the kexec mechanism. +There are several projects using [Linux](https://kernel.org) as a payload +(which was the configuration that gave coreboot its original name, LinuxBIOS). +That kernel is often rather small and serves to load a current kernel from +somewhere, e.g. disk or network, and run that through the kexec mechanism.
Two aspects emphasized by proponents of Linux-as-a-payload are the availability of well-tested, battle-hardened drivers (as compared to @@ -51,4 +79,4 @@ For more details have a look at [heads-wiki].
[Heads]: https://github.com/osresearch/heads -[heads-wiki]: http://osresearch.net/ \ No newline at end of file +[heads-wiki]: http://osresearch.net/
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: added more payload links and descriptions ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/45829/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45829/1//COMMIT_MSG@7 PS1, Line 7: added Add
https://review.coreboot.org/c/coreboot/+/45829/1/Documentation/payloads.md File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/1/Documentation/payloads.md@2... PS1, Line 29: enchaced enhanced
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45829
to look at the new patch set (#2).
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Documentation: Add more payload links and descriptions
Change-Id: I13c10edbc7fbb7017641ab025982c658cbeb3610 Signed-off-by: Zoltan Baldaszti bztemail@gmail.com --- M Documentation/payloads.md 1 file changed, 35 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/45829/2
Zoltan Baldaszti has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/45829/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/45829/1//COMMIT_MSG@7 PS1, Line 7: added
Add
Fixed!
https://review.coreboot.org/c/coreboot/+/45829/1/Documentation/payloads.md File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/1/Documentation/payloads.md@2... PS1, Line 29: enchaced
enhanced
Ups, typo. Thanks, fixed!
Attention is currently required from: Zoltan Baldaszti. Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(3 comments)
File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/comment/ace55fc9_ef27a317 PS2, Line 41: which can be : installed in a boot ROM and used to initialize That doesn't refer to its use as a payload though, and neither does the URL above
https://review.coreboot.org/c/coreboot/+/45829/comment/b5726860_dde5cf34 PS2, Line 56: version implementation?
https://review.coreboot.org/c/coreboot/+/45829/comment/a985cc6d_4b84d8f2 PS2, Line 61: [Linux](https://kernel.org) I think the more important aspect would be links to the projects using Linux, not a link to the kernel sources without any further comment (similar issue as with the treatment of U-Boot above)
Attention is currently required from: Zoltan Baldaszti. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(1 comment)
File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/comment/335a388f_b0f813b5 PS2, Line 82: http https?
Attention is currently required from: Angel Pons. Zoltan Baldaszti has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(4 comments)
File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/comment/510a6e0c_7fdf2274 PS2, Line 41: which can be : installed in a boot ROM and used to initialize
That doesn't refer to its use as a payload though, and neither does the URL above
I'm no position to describe U-Boot, text taken from the repo's README verbatim. https://gitlab.denx.de/u-boot/u-boot/-/blob/master/README About the link, that's its official url. The doc url in payloads/external/U-Boot/Kconfig.name is a dead-link. If you have a better one, let me know.
https://review.coreboot.org/c/coreboot/+/45829/comment/fc2b9413_b36d998f PS2, Line 56: version
implementation?
Yeah, why not?
https://review.coreboot.org/c/coreboot/+/45829/comment/b44f9ff5_d22d4de2 PS2, Line 61: [Linux](https://kernel.org)
I think the more important aspect would be links to the projects using Linux, not a link to the kern […]
This is a link to the Linux kernel, because the context refers to the Linux kernel. Distros should be listed separately IMHO (just like Heads). But if there's a list of Linux distros compiled for coreboot, let me know the url.
https://review.coreboot.org/c/coreboot/+/45829/comment/aaad92a9_75da4d54 PS2, Line 82: http
https?
I haven't checked this (the diff is because of the newline), but now that you mentioned it I did. It has an invalid cert, SSL_ERROR_BAD_CERT_DOMAIN. I guess that's the reason why it was http in the first place. Do you still want https knowing it's not working? I can update if you want.
Attention is currently required from: Zoltan Baldaszti. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(1 comment)
File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/comment/3b8dd7bf_55d2b679 PS2, Line 82: http
I haven't checked this (the diff is because of the newline), but now that you mentioned it I did. […]
Duh. No point in updating it then. 😄
Zoltan Baldaszti has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2: I'm sorry, I've committed the changes into a new issue accidentally https://review.coreboot.org/c/coreboot/+/49676
I've found the current equivalent of the dead-link in KConfig.name. That is the doc file in the U-Boot repo that mentions coreboot and links back to here. I've also tried to rephrase the text a bit to use the words "coreboot" and "payload".
I've changed the Linux link to "https://www.coreboot.org/Linux". ATM that links to the obsolete wiki page, but that's the only one I could find about "Implementations of Linux payloads", and it is at least a coreboot.org link.
I haven't changed the heads link (yet), because of the SSL error.
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/45829?usp=email )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Abandoned
This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author.
-
Angel Pons (Code Review) -
Martin L Roth (Code Review)
-
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)
-
Zoltan Baldaszti (Code Review)