feat(kernel): add kernel module loading helper with static kernel port#97
Merged
feat(kernel): add kernel module loading helper with static kernel port#97
Conversation
…port Add kernel_module.rs to cryptpilot-core with helper functions for loading kernel modules. The new implementation: - Checks /sys/module/<name> before attempting modprobe to support statically linked kernels (built-in modules) - Converts hyphens to underscores for module name compatibility - Provides ensure_module_loaded() for async module loading with params Update all modprobe calls across the codebase to use the new helper: - cryptpilot-fde: dm_mod, dm-linear, dm-verity, dm-snapshot, dm-zero, overlay, zram modules - cryptpilot-core: nbd, loop modules - luks2: dm_crypt, dm-integrity modules Add documentation for statically linked kernel configuration in boot.md and boot_zh.md, listing required kernel config options.
Change ensure_module_loaded() from returning Result<()> to () to make module loading failures non-fatal. This improves compatibility with containerized environments where modprobe may not be available. Key changes: - Check for modprobe binary existence before attempting to load - Convert all errors to warnings using tracing::warn!() - Remove anyhow dependency from kernel_module.rs - Update all call sites to remove ? error handling - Add dm_crypt and dm_integrity to static kernel config docs This allows CryptPilot to continue execution even when: - modprobe is not installed (containers, minimal systems) - Module loading fails (module may be built-in) - Kernel modules are statically linked
Correct the UKI parameter from --uki-mode to --uki in both English and Chinese quick-start documentation. The actual command-line option is --uki, not --uki-mode.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add kernel_module.rs to cryptpilot-core with helper functions for loading kernel modules. The new implementation:
Update all modprobe calls across the codebase to use the new helper:
Add documentation for statically linked kernel configuration in boot.md and boot_zh.md, listing required kernel config options.