Skip to content

Conversation

Jordymalone
Copy link
Contributor

@Jordymalone Jordymalone commented Sep 27, 2025

This commit includes two distinct changes to the documentation:

  1. Corrected the rmmod command for the hello-5 example from hello-5 to hello_5. The kernel renames hyphens to underscores on module load, which will cause the original command to fail.
  2. Improved formatting in the LaTeX document by re-wrapping lines, enhancing maintainability.

Summary by cubic

Fixes the rmmod command in the hello-5 example (use hello_5) and rewraps LaTeX lines to make the docs easier to edit and review.

  • Bug Fixes

    • Use rmmod hello_5; the kernel converts hyphens to underscores on module load, so hello-5 fails.
  • Refactors

    • Re-wrapped long lines in lkmpg.tex to improve readability and produce cleaner diffs.

This commit includes two distinct changes to the documentation:

1. Corrected the `rmmod` command for the `hello-5` example from
   `hello-5` to `hello_5`. The kernel renames hyphens to underscores on
   module load, which will cause the original command to fail.
2. Improved formatting in the LaTeX document by re-wrapping lines,
   enhancing maintainability.

Signed-off-by: Chia-Hao Chiu <[email protected]>
Copy link

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

Choose a reason for hiding this comment

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

No issues found across 1 file

Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

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

Split the proposed changes into two pull requests respectively.

@Jordymalone
Copy link
Contributor Author

Split the proposed changes into two pull requests respectively.

Split to #351 and #352 .

@Jordymalone Jordymalone deleted the improve-maintain branch September 29, 2025 06:00
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