[Deepin-Kernel-SIG] [linux 6.18-y] [revyos] riscv: config: deepin: update config from revyos v6.18 by opsiff · Pull Request #1588 · deepin-community/kernel

opsiff · 2026-03-24T10:20:07Z

Link: https://github.com/revyos/linux/blob/68e2cc426827976f3667c9d3e3e58ee1943e4a18/arch/riscv/configs/debian_defconfig

Summary by Sourcery

Update the deepin RISC-V 64 desktop kernel defconfig to align with the RevyOS/debian configuration for Linux 6.18.

sourcery-ai · 2026-03-24T10:20:15Z

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Updates the deepin RISC-V desktop defconfig to align with the upstream RevyOS/debian_defconfig configuration for Linux 6.18, refreshing kernel feature, driver, and subsystem options accordingly.

File-Level Changes

Change	Details	Files
Refresh deepin RISC-V desktop kernel configuration based on RevyOS debian_defconfig for Linux 6.18.	Synchronize CONFIG_* options with the referenced RevyOS debian_defconfig baseline Enable, disable, or change built-in vs module selection for various subsystems (e.g. filesystems, networking, device drivers, security, debug options) to match the new baseline Drop obsolete or no-longer-present configuration options and introduce any new ones required by Linux 6.18 on RISC-V Adjust RISC-V–specific platform, CPU, and SoC options for better compatibility with the RevyOS environment	`arch/riscv/configs/deepin_riscv64_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

Contact our support team for questions or feedback.
Visit our documentation for detailed guides and information.
Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

deepin-ci-robot · 2026-03-24T10:20:15Z

[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 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:

deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

sourcery-ai

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.}

Copilot

Pull request overview

Updates the Deepin RISC-V 64 desktop kernel defconfig to better align with the RevyOS/Debian configuration for Linux 6.18, expanding enabled subsystems, drivers, and security/features.

Changes:

Adjust scheduler/preemption and core kernel feature selections (e.g., PREEMPT_LAZY, RCU_LAZY, Rust).
Expand hardware/platform enablement (SoC/PCIe/USB/Bluetooth/WiFi, storage, I2C/SPI/pinctrl/GPIO).
Broaden security/integrity and filesystem/network feature coverage (e.g., IPE in LSM list, more crypto/filesystems, NVMe-oF options).

Comments suppressed due to low confidence (1)

arch/riscv/configs/deepin_riscv64_desktop_defconfig:1506

Several regulator drivers are enabled (e.g., CONFIG_REGULATOR_FIXED_VOLTAGE/AXP20X/DA9063/TPS65086), but the regulator framework menuconfig (CONFIG_REGULATOR) is not enabled in this defconfig. Because these options are all under if REGULATOR (drivers/regulator/Kconfig), they will be dropped unless something else selects REGULATOR. Consider explicitly setting CONFIG_REGULATOR=y to ensure these modules are actually built.

CONFIG_MFD_TPS65086=m
CONFIG_REGULATOR_FIXED_VOLTAGE=m
CONFIG_REGULATOR_AXP20X=m
CONFIG_REGULATOR_DA9063=m
CONFIG_REGULATOR_TPS65086=m

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

Copilot · 2026-03-24T10:53:33Z

 # CONFIG_SYSFS_SYSCALL is not set
+CONFIG_KALLSYMS_ALL=y
 CONFIG_PROFILING=y
+CONFIG_RUST=y


CONFIG_RUST is enabled in this defconfig, which makes a Rust toolchain (rustc + bindgen at/above scripts/min-tool-version.sh) a hard build dependency. If the Deepin kernel build/CI environment does not guarantee Rust availability for riscv64, consider disabling CONFIG_RUST here or updating the build docs/CI to install the required toolchain.

Suggested change

CONFIG_RUST=y

# CONFIG_RUST is not set

Copilot · 2026-03-24T10:53:33Z

 CONFIG_MODULE_FORCE_UNLOAD=y
 CONFIG_MODVERSIONS=y
+CONFIG_GENDWARFKSYMS=y
+CONFIG_MODULE_SIG_SHA256=y


CONFIG_MODULE_SIG_SHA256 is set, but CONFIG_MODULE_SIG is not enabled anywhere in this defconfig. As-is, the hash selection (and CONFIG_MODULE_SIG_KEY later) will be ignored by Kconfig; either enable module signing (CONFIG_MODULE_SIG=y, plus any policy options you need) or drop the module signing settings from the defconfig to avoid a misleading no-op.

Suggested change

CONFIG_MODULE_SIG_SHA256=y

# CONFIG_MODULE_SIG_SHA256 is not set

Copilot · 2026-03-24T10:53:33Z

 CONFIG_PACKET_DIAG=m
 CONFIG_UNIX_DIAG=m
-CONFIG_TLS=m
 CONFIG_TLS_DEVICE=y


CONFIG_TLS_DEVICE depends on CONFIG_TLS (net/tls/Kconfig). Since CONFIG_TLS is not set anywhere in this defconfig, TLS_DEVICE will be forced off during olddefconfig. Enable CONFIG_TLS (m/y) if TLS offload is intended, or remove CONFIG_TLS_DEVICE to keep the defconfig consistent.

Suggested change

CONFIG_TLS_DEVICE=y

Copilot · 2026-03-24T10:53:34Z

 CONFIG_NVME_RDMA=m
 CONFIG_NVME_FC=m
 CONFIG_NVME_TCP=m
+CONFIG_NVME_TCP_TLS=y


NVME TCP TLS is enabled (CONFIG_NVME_TCP_TLS=y) but CONFIG_NVME_TCP is set to =m. Since NVME_TCP_TLS is a bool that depends on NVME_TCP (drivers/nvme/host/Kconfig), Kconfig will drop NVME_TCP_TLS unless NVME_TCP is built-in (=y). Align these settings (either make NVME_TCP=y if you want TLS, or disable NVME_TCP_TLS).

Suggested change

CONFIG_NVME_TCP_TLS=y

# CONFIG_NVME_TCP_TLS is not set

Copilot · 2026-03-24T10:53:34Z

 CONFIG_NVME_TARGET_FC=m
+CONFIG_NVME_TARGET_FCLOOP=m
 CONFIG_NVME_TARGET_TCP=m
+CONFIG_NVME_TARGET_TCP_TLS=y


NVME target TCP TLS is enabled (CONFIG_NVME_TARGET_TCP_TLS=y) but CONFIG_NVME_TARGET_TCP is set to =m. Since NVME_TARGET_TCP_TLS is a bool that depends on NVME_TARGET_TCP (drivers/nvme/target/Kconfig), it will be forced off unless NVME_TARGET_TCP is built-in (=y). Align these options (make NVME_TARGET_TCP=y if TLS is desired, or disable NVME_TARGET_TCP_TLS).

Suggested change

CONFIG_NVME_TARGET_TCP_TLS=y

# CONFIG_NVME_TARGET_TCP_TLS is not set

Avenger-285714 · 2026-03-24T10:47:44Z

-CONFIG_DRM_AMDGPU_USERPTR=y
 CONFIG_DRM_AMD_DC_SI=y
+CONFIG_HSA_AMD=y
+# CONFIG_HSA_AMD_SVM is not set


这个有什么特殊的原因吗

这个有什么特殊的原因吗
我怀疑riscv上也有问题(&& !LOONGARCH && !ARM64)
config HSA_AMD_SVM
bool "Enable HMM-based shared virtual memory manager"
depends on HSA_AMD && DEVICE_PRIVATE && !LOONGARCH && !ARM64

Avenger-285714 · 2026-03-24T10:53:07Z

 CONFIG_PAGE_POISONING=y
 CONFIG_SCHED_STACK_END_CHECK=y
 CONFIG_DEBUG_MEMORY_INIT=y
+CONFIG_MEMORY_NOTIFIER_ERROR_INJECT=m


这个有必要嘛

这个有必要嘛

这个只是模块开了也不影响,x86也打开了

Avenger-285714 · 2026-03-24T10:53:52Z

 CONFIG_TRACER_SNAPSHOT=y
 CONFIG_BLK_DEV_IO_TRACE=y
 CONFIG_IO_STRICT_DEVMEM=y
-CONFIG_KUNIT=m


单测是好东西，不建议关

单测是好东西，不建议关

开了

Avenger-285714 · 2026-03-24T10:54:33Z

 CONFIG_CRYPTO_FIPS=y
 CONFIG_CRYPTO_USER=m
-# CONFIG_CRYPTO_MANAGER_DISABLE_TESTS is not set
+CONFIG_CRYPTO_SELFTESTS=y


可能会影响开机时间

可能会影响开机时间

ok

Avenger-285714 · 2026-03-24T10:57:04Z

 CONFIG_PATA_TOSHIBA=m
 CONFIG_ATA_GENERIC=m
 CONFIG_MD=y
-CONFIG_MD_LINEAR=m


大概有用，不建议关

大概有用，不建议关

ok

Avenger-285714 · 2026-03-24T11:00:58Z

 CONFIG_BPF_JIT=y
 CONFIG_BPF_LSM=y
-CONFIG_PREEMPT_VOLUNTARY=y
+CONFIG_PREEMPT_LAZY=y


建议 @opsiff 大神跑分测一下。如果该选项性能比 CONFIG_PREEMPT_VOLUNTARY 更优，可以考虑整一个全架构的替换。甚至考虑 backport 至 linux-6.6.y。没记错的话 linux-6.12.y 及以前没有该选项。

建议 @opsiff 大神跑分测一下。如果该选项性能比 CONFIG_PREEMPT_VOLUNTARY 更优，可以考虑整一个全架构的替换。甚至考虑 backport 至 linux-6.6.y。没记错的话 linux-6.12.y 及以前没有该选项。

暂时不改后面观察一下

Link: https://github.com/revyos/linux/blob/68e2cc426827976f3667c9d3e3e58ee1943e4a18/arch/riscv/configs/debian_defconfig Signed-off-by: Wentao Guan <guanwentao@uniontech.com>

deepin-ci-robot requested review from huangbibo and shy129 March 24, 2026 10:20

sourcery-ai Bot reviewed Mar 24, 2026

View reviewed changes

opsiff force-pushed the linux-6.18.y-2026-03-24-riscv branch from d50c4a5 to 056ac47 Compare March 24, 2026 10:22

Avenger-285714 requested review from Avenger-285714 and Copilot March 24, 2026 10:45

Copilot started reviewing on behalf of Avenger-285714 March 24, 2026 10:47 View session

Copilot AI reviewed Mar 24, 2026

View reviewed changes

Avenger-285714 reviewed Mar 24, 2026

View reviewed changes

riscv: config: deepin: update config from revyos v6.18

045260a

Link: https://github.com/revyos/linux/blob/68e2cc426827976f3667c9d3e3e58ee1943e4a18/arch/riscv/configs/debian_defconfig Signed-off-by: Wentao Guan <guanwentao@uniontech.com>

opsiff force-pushed the linux-6.18.y-2026-03-24-riscv branch from 056ac47 to 045260a Compare March 25, 2026 02:03

opsiff merged commit 4fcaced into deepin-community:linux-6.18.y Mar 25, 2026
10 of 12 checks passed

	CONFIG_MODULE_SIG_SHA256=y
	# CONFIG_MODULE_SIG_SHA256 is not set

	CONFIG_NVME_TARGET_TCP_TLS=y
	# CONFIG_NVME_TARGET_TCP_TLS is not set

Conversation

opsiff commented Mar 24, 2026 • edited by sourcery-ai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by Sourcery

Uh oh!

sourcery-ai Bot commented Mar 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviewer's Guide

File-Level Changes

Interacting with Sourcery

Customizing Your Experience

Getting Help

Uh oh!

deepin-ci-robot commented Mar 24, 2026

Uh oh!

sourcery-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

opsiff commented Mar 24, 2026 •

edited by sourcery-ai Bot

Loading

sourcery-ai Bot commented Mar 24, 2026 •

edited

Loading