Skip to content

Conversation

@orangecms
Copy link
Contributor

Rather positive wording, putting the book 📖 at the top.

Signed-off-by: Daniel Maslowski <info@orangecms.org>
@@ -1,29 +1,32 @@
LinuxBoot is the concept of using a Linux kernel and initramfs as a bootloader.
LinuxBoot is the concept of [using a Linux kernel and initramfs as a
bootloader](#benefits).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link text doesn't match the link. Benefits of something are not equal to that thing. Anyway, the benefits section is only a few sentences away so no need to link to it.


The official documentation for LinuxBoot is the [LinuxBoot Book 📖](https://book.linuxboot.org),
where you can find information on how LinuxBoot works and implementing it with
various approaches, such as based on coreboot or UEFI PEI.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Implementing it with various approaches" would be how you configure the kernel or what you use for userspace. coreboot and UEFI PEI are two examples of loaders of LinuxBoot, not strictly examples of implementation approaches, although they can impact the implementation. I'd also either remove the explicit examples or at least add U-Boot SPL. This is just two of the 4 examples already in linuxboot_info.png.

* Improved boot time by excluding unnecessary code, typically 20 times faster
than other solutions
* Runtime customization to support site-specific needs, covering both device
drivers as well as executables
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd personally just move this to the book and mention that in the sentence about the book above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants