-
Notifications
You must be signed in to change notification settings - Fork 92
[Deepin-Kernel-SIG] [linux 6.6-y] Update defconfigs #861
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Deepin-Kernel-SIG] [linux 6.6-y] Update defconfigs #861
Conversation
…r feats Follow changes: *1. Enable ntsync. 2. Enable more Intel device drivers. 3. Enable CONFIG_DRM_I915_WERROR. 4. Ennable CONFIG_DRM_PHYTIUM for e2000q BMC. 5. Enable ONFIG_DRM_INSPUR for Inspur BMC. 6. Change CONFIG_VFIO_PCI from 'y' to 'm'. Signed-off-by: WangYuli <[email protected]>
…her feats Follow changes: *1. Enable ntsync. 2. Enable more DRM drivers. 3. Enable more CRYPTO drivers. 4. Enable more other drivers. Signed-off-by: WangYuli <[email protected]>
…d other feats Follow changes: *1. Enable ntsync. 2. Adjust orders by savedefconfig. 3. Enable CONFIG_DRM_INSPUR for Inspur BMC. Signed-off-by: WangYuli <[email protected]>
Reviewer's GuideThis PR refreshes the Deepin desktop defconfigs for arm64, loongarch, and x86 to align with Linux 6.6-y upstream updates, cleaning up deprecated entries and enabling new features. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
deepin pr auto review代码审查意见如下:
总体来说,这些更改可能涉及内核配置的修改,需要确保这些更改不会影响系统的稳定性和安全性。建议在合并这些更改之前进行充分的测试和验证。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Updates Deepin kernel defconfigs across x86, LoongArch, and ARM64 for Linux 6.6-y by enabling new modules, adjusting VFIO/IOMMU options, and reworking default platform driver configurations.
- Add NTSYNC, DRM enhancements, and Intel platform support on x86.
- Introduce custom cmdline flags, reorganize IOMMU/VFIO, and include NTSYNC/DRM_INSPUR on LoongArch.
- Adjust PCI and crypto defaults, add GPIO/LCD/PMU/crypto drivers, and refine VFIO on ARM64.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
File | Description |
---|---|
arch/x86/configs/deepin_x86_desktop_defconfig | Enabled new modules (NTSYNC, VGA_SWITCHEROO, DRM flags), adjusted VFIO PCI, added Intel IFS and power drivers |
arch/loongarch/configs/deepin_loongarch_desktop_defconfig | Added boot cmdline and EXTEND, reordered IOMMU/VFIO settings, added NTSYNC and DRM_INSPUR |
arch/arm64/configs/deepin_arm64_desktop_defconfig | Removed PCI_IOV and select crypto ciphers, added numerous bus/GPIO/LCD/PMU/crypto drivers, updated VFIO |
Comments suppressed due to low confidence (5)
arch/arm64/configs/deepin_arm64_desktop_defconfig:741
- Removing PCI_IOV support disables SR-IOV functionality, which may be required for virtualization workloads; verify if this was intentional or revert it.
CONFIG_PCI_IOV=y
arch/arm64/configs/deepin_arm64_desktop_defconfig:4484
- Removing the CHACHA20-POLY1305 cipher may break kernel modules or userland features that depend on it; ensure there’s an alternative or re-enable support.
CONFIG_CRYPTO_CHACHA20POLY1305=m
arch/x86/configs/deepin_x86_desktop_defconfig:3361
- [nitpick] Enabling a debug flag in the default config may degrade runtime performance; consider disabling this in production defconfigs or gating it behind a debug build option.
CONFIG_DRM_DEBUG_DP_MST_TOPOLOGY_REFS=y
arch/x86/configs/deepin_x86_desktop_defconfig:3382
- [nitpick] Treating warnings as errors for the i915 driver can break builds if kernel warnings appear; consider removing or making this optional to avoid unexpected CI failures.
CONFIG_DRM_I915_WERROR=y
arch/loongarch/configs/deepin_loongarch_desktop_defconfig:60
- Disabling KASLR and allowing unsafe interrupts weakens kernel security; recommend removing or documenting the risk for production builds.
CONFIG_CMDLINE="vfio_iommu_type1.allow_unsafe_interrupts=1 nokaslr"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Avenger-285714 - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Wangyuli said that all arch can boot, so merged. |
Summary by Sourcery
Refresh Deepin default kernel configurations across ARM64, LoongArch, and x86 architectures to synchronize with linux 6.6-y updates.
Enhancements: