Skip to content

Add Secure Boot signing guide#87

Open
Cuda-Chen wants to merge 1 commit into
sysprog21:masterfrom
Cuda-Chen:add-secure-boot-signing-guide
Open

Add Secure Boot signing guide#87
Cuda-Chen wants to merge 1 commit into
sysprog21:masterfrom
Cuda-Chen:add-secure-boot-signing-guide

Conversation

@Cuda-Chen

@Cuda-Chen Cuda-Chen commented Jun 17, 2026

Copy link
Copy Markdown

Add Secure Boot signing guide.


Summary by cubic

Add a Secure Boot signing guide and link it from the README so users can load simplefs.ko on UEFI Secure Boot systems. The guide shows Ubuntu commands to install linux-headers-$(uname -r), mokutil, kmod, and openssl; generate and enroll a MOK key via mokutil/MOK Manager; sign the module and verify with modinfo—with the README noting disabling Secure Boot as an alternative.

Written for commit 06da008. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

2 issues found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="README.md">

<violation number="1" location="README.md:36">
P3: New Secure Boot sentence contains grammar errors that hurt readability</violation>
</file>

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread secure-boot-signing-guide.md Outdated
Comment thread README.md Outdated
Generate test image via `make test.img`, which creates a zeroed file of 50 MiB.

You can then mount this image on a system with the simplefs kernel module installed.
As nowadays many computer enables UEFI Secure Boot, you can either disable Secure Boot

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3: New Secure Boot sentence contains grammar errors that hurt readability

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At README.md, line 36:

<comment>New Secure Boot sentence contains grammar errors that hurt readability</comment>

<file context>
@@ -33,6 +33,9 @@ You can build the kernel module and tool with `make`.
 Generate test image via `make test.img`, which creates a zeroed file of 50 MiB.
 
 You can then mount this image on a system with the simplefs kernel module installed.
+As nowadays many computer enables UEFI Secure Boot, you can either disable Secure Boot
+or sign the kernel module. To sign the module, refer to [secure-boot-signing-guide.md](./secure-boot-signing-guide.md).
+
</file context>
Suggested change
As nowadays many computer enables UEFI Secure Boot, you can either disable Secure Boot
+As many computers now have UEFI Secure Boot enabled, you can either disable Secure Boot
or sign the kernel module. To sign the module, refer to [secure-boot-signing-guide.md](./secure-boot-signing-guide.md).

@Cuda-Chen Cuda-Chen force-pushed the add-secure-boot-signing-guide branch from fba532c to 06da008 Compare June 17, 2026 15:16
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.

1 participant