Skip to content

[Deepin-Kernel-SIG] [linux 6.18-y] [Deepin] deepin_x86_desktop_defconfig: refresh config for v6.18#1524

Open
opsiff wants to merge 2 commits intodeepin-community:linux-6.18.yfrom
opsiff:linux-6.18.y-2026-03-03-config-x86
Open

[Deepin-Kernel-SIG] [linux 6.18-y] [Deepin] deepin_x86_desktop_defconfig: refresh config for v6.18#1524
opsiff wants to merge 2 commits intodeepin-community:linux-6.18.yfrom
opsiff:linux-6.18.y-2026-03-03-config-x86

Conversation

@opsiff
Copy link
Member

@opsiff opsiff commented Mar 3, 2026

deepin inclusion
category: feature

In the config, refresh config for x86 in v6.18.

Log:
Remove CONFIG_ROCKCHIP_PHY.
Enabled CONFIG_STRICT_DEVMEM.
CONFIG_TLS=m enabled by depend.
CONFIG_NETFILTER_XT_TARGET_CT enabled by depend.
CONFIG_NETFILTER_XT_MATCH_SCTP enabled by depend.
CONFIG_EFI_COCO_SECRET=y enabled by depend.
CONFIG_MMC_BLOCK=m enabled by depend.
CONFIG_SERIAL_MULTI_INSTANTIATE=m enabled by depend.
And enabled other configs which has enabled in other dist.

Summary by Sourcery

Build:

  • Update x86 desktop kernel defconfig options, including removal of unused PHY support and enabling additional memory, networking, MMC, serial, and EFI configuration symbols to match current distribution defaults.

@sourcery-ai
Copy link

sourcery-ai bot commented Mar 3, 2026

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Refreshes the deepin x86 desktop kernel defconfig for Linux v6.18, mainly tightening memory access, enabling additional networking, security, and device support options, and dropping an unused Rockchip PHY option.

File-Level Changes

Change Details Files
Drop unused Rockchip PHY option from the x86 desktop defconfig.
  • Remove CONFIG_ROCKCHIP_PHY from the configuration since it is not needed for the x86 desktop profile
arch/x86/configs/deepin_x86_desktop_defconfig
Harden low-level memory access and enable additional security-related options.
  • Enable CONFIG_STRICT_DEVMEM to restrict /dev/mem access
  • Enable CONFIG_EFI_COCO_SECRET for confidential computing secrets handling, enabled via dependency
  • Ensure CONFIG_TLS=m is set via dependencies for kernel TLS support
arch/x86/configs/deepin_x86_desktop_defconfig
Expand networking and connection tracking capabilities via netfilter options.
  • Enable CONFIG_NETFILTER_XT_TARGET_CT via dependency for connection tracking target support
  • Enable CONFIG_NETFILTER_XT_MATCH_SCTP via dependency for SCTP match support
arch/x86/configs/deepin_x86_desktop_defconfig
Enable additional device and driver support commonly enabled in other distributions.
  • Enable CONFIG_MMC_BLOCK=m via dependency for MMC block device support
  • Enable CONFIG_SERIAL_MULTI_INSTANTIATE=m via dependency for multi-instantiate serial devices
  • Align other config options with those already enabled in other distributions (details in diff)
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

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 - I've reviewed your changes and they look great!


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 force-pushed the linux-6.18.y-2026-03-03-config-x86 branch 3 times, most recently from 1200110 to 73409c9 Compare March 3, 2026 10:23
deepin inclusion
category: feature

In the config, refresh config for x86 in v6.18.

Log:
Remove CONFIG_ROCKCHIP_PHY.
Enabled CONFIG_STRICT_DEVMEM.
CONFIG_TLS=m enabled by depend.
CONFIG_NETFILTER_XT_TARGET_CT enabled by depend.
CONFIG_NETFILTER_XT_MATCH_SCTP enabled by depend.
CONFIG_EFI_COCO_SECRET=y enabled by depend.
CONFIG_MMC_BLOCK=m  enabled by depend.
CONFIG_SERIAL_MULTI_INSTANTIATE=m  enabled by depend.
And enabled other configs which has enabled in other dist.

Signed-off-by: Wentao Guan <guanwentao@uniontech.com>
@opsiff opsiff force-pushed the linux-6.18.y-2026-03-03-config-x86 branch from 73409c9 to 81abb99 Compare March 3, 2026 10:29
@opsiff
Copy link
Member Author

opsiff commented Mar 3, 2026

Update to HYPERV driver config and USB3 alt mode driver. @sourcery-ai review

Copy link

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

Refreshes the Deepin x86 desktop kernel defconfig for the Linux 6.18-y series to better align with current distro defaults and newly-available kernel features.

Changes:

  • Adds/enables a broad set of kernel config symbols across cgroups, RAS/ACPI APEI, storage/NVMe, networking, HID/USB, accessibility, virtualization, and security.
  • Removes a small number of previously-set symbols that are no longer desired/needed (e.g., ROCKCHIP PHY, explicit TLS / certain netfilter options).
  • Tightens /dev/mem-related behavior by changing the strict devmem setting behavior in the defconfig.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 6190 to 6192
CONFIG_HIST_TRIGGERS=y
# CONFIG_STRICT_DEVMEM is not set
CONFIG_IO_STRICT_DEVMEM=y
# CONFIG_X86_VERBOSE_BOOTUP is not set
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR description/log says "Enabled CONFIG_STRICT_DEVMEM", but the defconfig change here enables CONFIG_IO_STRICT_DEVMEM instead (and removes the explicit STRICT_DEVMEM=n). If the intent is only to turn on /dev/mem filtering, consider setting CONFIG_STRICT_DEVMEM=y (and leaving IO_STRICT_DEVMEM unset) or update the PR description to reflect that this also enables the stricter I/O-range filtering behavior of IO_STRICT_DEVMEM.

Copilot uses AI. Check for mistakes.
deepin inclusion
category: feature

In the commit, merge more driver configs such as chromebook which use it,
In the commit, merge more inject or test modules for our release.
In the commit, enable nfs_v2.

Signed-off-by: Wentao Guan <guanwentao@uniontech.com>
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found here.

Details 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

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.

4 participants