framework - enable NTSYNC as a module on most kernels

[tabrisnet]

Description

requested feature #9021
Explicitly exclude filogic, not so much because it's necessary but because it also serves as an example for armbian/documentation#842

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

• time ./compile.sh rewrite-kernel-config BOARD=pocketbeagle2 BRANCH=edge KERNEL_GIT=full MANAGE_ACNG='http://squid.tabris.net:3142/'
• time ./compile.sh rewrite-kernel-config BOARD=bananapir4 BRANCH=current KERNEL_GIT=full MANAGE_ACNG='http://squid.tabris.net:3142/'
• time ./compile.sh rewrite-kernel-config BOARD=bananapir4 BRANCH=edge KERNEL_GIT=full MANAGE_ACNG='http://squid.tabris.net:3142/'

Checklist:

Please delete options that are not relevant.

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings

[coderabbitai]

Walkthrough

Adds a new kernel configuration function to enable NTSYNC (Windows NT synchronization primitives) for kernel versions 6.14 and above, while disabling this feature for the filogic platform family through configuration override.

Changes

Cohort / File(s)	Summary
Kernel NTSYNC Configuration lib/functions/compilation/armbian-kernel.sh	New function armbian_kernel_config__enable_ntsync() that enables NTSYNC support for kernel >= 6.14; skips activation on vendor branches and appends to opts_m list when enabled
Filogic Family Configuration config/sources/families/filogic.conf	Disables the ntsync kernel config function for filogic family, noting that Proton/WINE is unlikely on this platform

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

• Focus on verifying the NTSYNC function's version check logic (>= 6.14) and opts_m list handling
• Confirm that unsetting the function in filogic.conf appropriately disables the feature for that platform

Suggested reviewers

• igorpecovnik
• chraac
• AGM1968
• teknoid
• EvilOlaf

Poem

🐰 A kernel tweak for Windows sync so fine,
Where NTSYNC dances in code's design,
But filogic says "nay, we've no need for wine,"
So the function's unset in its config line—
Smart choices keep systems running divine! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'framework - enable NTSYNC as a module on most kernels' accurately reflects the main objective of enabling NTSYNC across most kernels while excluding specific cases like filogic.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

✅ This PR has been reviewed and approved — all set for merge!