Skip to content

[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

Merged
merged 3 commits into from
Jun 9, 2025

Conversation

Avenger-285714
Copy link
Collaborator

@Avenger-285714 Avenger-285714 commented Jun 9, 2025

Summary by Sourcery

Refresh Deepin default kernel configurations across ARM64, LoongArch, and x86 architectures to synchronize with linux 6.6-y updates.

Enhancements:

  • Update Deepin desktop defconfig for ARM64 to align with linux 6.6-y
  • Update Deepin desktop defconfig for LoongArch to align with linux 6.6-y
  • Update Deepin desktop defconfig for x86 to align with linux 6.6-y

…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]>
@Avenger-285714 Avenger-285714 requested a review from Copilot June 9, 2025 08:13
Copy link

sourcery-ai bot commented Jun 9, 2025

Reviewer's Guide

This 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

Change Details Files
Refresh arm64 defconfig to new Linux 6.6 defaults
  • Applied upstream kernel defconfig updates
  • Removed deprecated Deepin-specific config options
  • Added new GPU and power management settings
arch/arm64/configs/deepin_arm64_desktop_defconfig
Update loongarch defconfig for Linux 6.6
  • Synced with upstream loongarch default configs
  • Enabled performance and virtualization features
  • Cleaned up obsolete config flags
arch/loongarch/configs/deepin_loongarch_desktop_defconfig
Synchronize x86 defconfig with latest kernel release
  • Added new CPU microarchitecture optimizations
  • Updated default filesystem and networking configs
  • Removed removed or deprecated options
arch/x86/configs/deepin_x86_desktop_defconfig

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from avenger-285714. For more information see the Code Review Process.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Avenger-285714 Avenger-285714 requested a review from opsiff June 9, 2025 08:14
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见如下:

  1. arch/arm64/configs/deepin_arm64_desktop_defconfig文件中,移除了CONFIG_PCI_IOV配置项,但未提供移除的原因。如果该配置项不再需要,请确认是否有替代方案或相关文档说明。如果移除是必要的,请确保没有其他模块依赖于该配置项。

  2. arch/arm64/configs/deepin_arm64_desktop_defconfig文件中,添加了CONFIG_BRCMSTB_GISB_ARB=m等新的配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

  3. arch/arm64/configs/deepin_arm64_desktop_defconfig文件中,移除了CONFIG_CRYPTO_DES=mCONFIG_CRYPTO_CHACHA20POLY1305=m配置项。如果这些配置项不再需要,请确认是否有替代方案或相关文档说明。如果移除是必要的,请确保没有其他模块依赖于这些配置项。

  4. arch/loongarch/configs/deepin_loongarch_desktop_defconfig文件中,添加了CONFIG_CMDLINE="vfio_iommu_type1.allow_unsafe_interrupts=1 nokaslr"CONFIG_CMDLINE_EXTEND=y配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

  5. arch/loongarch/configs/deepin_loongarch_desktop_defconfig文件中,移除了CONFIG_KPROBES配置项。如果该配置项不再需要,请确认是否有替代方案或相关文档说明。如果移除是必要的,请确保没有其他模块依赖于该配置项。

  6. arch/loongarch/configs/deepin_loongarch_desktop_defconfig文件中,添加了CONFIG_VFIO_PCI_INTX=yCONFIG_QAT_VFIO_PCI=m配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

  7. arch/x86/configs/deepin_x86_desktop_defconfig文件中,添加了CONFIG_VGA_SWITCHEROO=yCONFIG_DRM_DEBUG_DP_MST_TOPOLOGY_REFS=y配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

  8. arch/x86/configs/deepin_x86_desktop_defconfig文件中,移除了CONFIG_RUNTIME_TESTING_MENU配置项。如果该配置项不再需要,请确认是否有替代方案或相关文档说明。如果移除是必要的,请确保没有其他模块依赖于该配置项。

  9. arch/x86/configs/deepin_x86_desktop_defconfig文件中,添加了CONFIG_DRM_I915_WERROR=yCONFIG_DRM_PHYTIUM=m配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

  10. arch/x86/configs/deepin_x86_desktop_defconfig文件中,添加了CONFIG_INTEL_IFS=mCONFIG_INTEL_BYTCRC_PWRSRC=m配置项。请确保这些配置项是必要的,并且已经过充分测试。如果这些配置项是新增功能的一部分,请提供相应的文档或测试结果。

总体来说,这些更改可能涉及内核配置的修改,需要确保这些更改不会影响系统的稳定性和安全性。建议在合并这些更改之前进行充分的测试和验证。

Copy link

@Copilot Copilot AI left a 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"

Copy link

@sourcery-ai sourcery-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.

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

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@opsiff opsiff merged commit ad689d8 into deepin-community:linux-6.6.y Jun 9, 2025
6 of 8 checks passed
@opsiff
Copy link
Member

opsiff commented Jun 9, 2025

Wangyuli said that all arch can boot, so merged.

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.

3 participants