Repository navigation
Extending Arieltune for Alpine Linux LTS - #29
Open
infinitevalence wants to merge 53 commits into
Open
infinitevalence wants to merge 53 commits into
infinitevalence wants to merge 53 commits into
Conversation
Contributor
|
Three files conflict against main (cu.rs, smm.rs, umc.rs) - everything else (all 29 alpine-6.18.53 patches, bc250-enable-40cu-alpine.sh, both install.sh files) merges clean. Root cause: this branch forked before commit 189ede0 landed on main ( Suggested fix, no rebase needed - on the ALPINE branch: git fetch https://github.com/cachenetics/project-ariel.git main
git checkout FETCH_HEAD -- arieltune/crates/apu/src/cu.rs arieltune/crates/bios/src/smm.rs arieltune/crates/mem/src/umc.rs
git commit -m "drop redundant ioctl portability tweaks, already fixed upstream (189ede0)"
git push |
- Switch shebang from #!/usr/bin/env bash to #!/usr/bin/env sh for Alpine compatibility - Remove pipefail from set flags (bashism not supported by busybox ash) - Add privilege elevation detection: prefers sudo (most distros), falls back to doas (Alpine), errors if neither available - Replace all hardcoded sudo calls with $SUDO_CMD
… all combined mode
…hen omitted The detect_kbuild_tree() and resolve_kbuild_tree() functions were already present but the build/all cases required the user to pass <kbuild-tree> as a positional argument. Removed the mandatory-arg guards so both args now auto-detect when omitted, and updated the usage comments to reflect this.
Add /etc/alpine-release check to print mkimage for Alpine instead of mkinitcpio -P. Update CONTEXT.md instructions and ALPINE_WORK_ORDERS to reflect completed fixes and updated line numbers.
…vice suffix rc-update add 'arieltune-gpu.service' looks for /etc/init.d/arieltune-gpu.service which doesn't exist. OpenRC init scripts are named without the .service suffix. Fix: strip '.service' from all OpenRC-specific paths via a shared strip_rc_unit() helper — init_op enable/disable/start/stop/restart, install_enable, enabled, gpu_unit_installed, gpu_unit_active, and remove_legacy_units. This fixes the rc-update add arieltune-gpu.service default exit status: 1 error.
rc-update add fails silently (exit 1) if the init script isn't executable; std::fs::write sets default 0644. Now install_enable sets 0755 for OpenRC and 0644 for systemd unit files.
The shebang #!/usr/bin/env openrc caused OpenRC to report "not a valid runlevel" because it cannot parse unrecognized shebangs for runlevel detection. Use #!/sbin/openrc instead.
…install_headers() function - Alpine branch now detects kver_base from uname -r (strips postdash suffix for detection) and uses it to look up linux-headers- - Alpine header detection now tries the stripped base version first (e.g. linux-headers-7.0.9), falls back to the full version string (e.g. linux-headers-7.0.9-17-pve) before trying linux-lts-headers - Added linux-tools- to the Alpine install list for sensors-cli and hwmon support - Added linux-headers-common as a dependency in the Alpine branch - Alpine headers are now installed in build mode (not install mode) to get KBUILD_TREE - install_headers now passes kver_base instead of raw uname -r output, fixing broken Alpine header installs - Added install mode: both build + install to keep it clean and traceable for the user. build-and-install.sh removed bash from Alpine install list - no bash required for #! /usr/bin/env sh build-and-install.sh removed bash from Alpine install list - no bash required for #! /usr/bin/env sh
…de .ko path, improve usage text
Bug 1: enabled() now checks /etc/runlevel/default/{rc} (matching rc-update default)
Bug 2: generate_openrc_script() uses pidfile= + start-stop-daemon --background --pidfile
… oneshot handling
…gpu apply-boot string
… (alpine/cachyos)
…efore modules_prepare
…e compile Add `make -C MODSRC -j$(nproc)` between `modules_prepare` and amdgpu module build. This generates top-level Module.symvers which fixes modpost 7 undefined symbol errors (kobj_sysfs_ops, drm_writeback_queue_job, memmove, etc.).
…y, update bc250 script and CONTEXT
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
2 participants
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.
Updated install arieltune script to allow building on ALPINE. Added script to patch amdgpu, modified patch 7 & 8 to work on Alpine 6.18.53 LTS kernel.