Conversation
099357e to
832d9bc
Compare
|
这个 PR 改动也太大了，我感觉很难 review。有没有更好的分解方式，按照子模块，或者 feature 改进分步进行 PR？ 如果正如 PR 上的如下描述改动了 10 个点，这些点看上去是独立的，那也应该要有至少 10 个 PR。
另外为啥要改动，原因请详细列出来，也为我们 review 提供帮助，对于重大修改，PR 描述中至少应该包括：
最后，只验证一个 virt64 恐怕是不够的，对其他产品，这么大的改动一次合入主线能保证其他 RISC-V 产品（特别是 T-head）的 BSP 工作不受影响吗？有没有更好的规划？ 总之，我建议小步快跑方式，越是大的改动，越需要谨慎。 |
github-actions
Bot
added
RT-Smart
|
一步步来解释。 首先这个 PR 肯定不会是直接以这样的方式申请合并，上面 更多是希望和相关人员探讨这么改（仅从技术角度讨论）是否可以接受，PR 会一直根据大家的意见进行改进，等到相关依赖都合并，相关验证、重新适配完成，以及大家意见统一为止。 WHY本次提交目的很简单：
根据对已有的操作系统和 RISC-V 目前标准来看，RT-Thread 在 RISC-V 支持上存在技术支持落后，代码维护性差，以及新平台适配没有统一标准等问题，因此根据这些问题提出的解决方案只有重写。 很遗憾，本次 PR 无法再进行拆分，否则将会陷入长期的 Kernel 跟进 BSP -> BSP 跟进 Kernel -> Kernel 跟进 BSP -> 新 BSP 插入 -> 新 BSP 重新支持 等无穷无尽的拉锯战；一次性把问题解决掉、将不积极跟进主线的 BSP 踢掉或者放弃，才是目前让 Kernel 得以快速发展的唯一解。 同时上述提到的各种新的点，仅是对改进的一个总结，并不是说代码真的可以分成这么多个模块，有的模块之间存在依赖，在已有代码基础上拆分提交，（从 AArch64 的统一经验来看）这种工作量就干脆不要搞了。 HOWISA manager interface.增加对目前 RISC-V ISA 位说明，使用位图进行管理，便于代码动态监测 ISA 状况，平台可以自行设置 ISA 支持情况（可参考 Merge 32E/32I/64I.32 位和 64 位只有访存指令，部分浮点数据位长度不一样，其他都是兼容的，因此相关差异用宏进行封装编程就可以，相关文件位于 至于 32E，仅是支持的寄存器更少，在上下文管理中使用 Cache by CBOM ISA.该 ISA 就是标准的 D-Cache 操作，只有 T-Head 平台没支持该 ISA，而且还提供了 DM 版本的 HWCache 支持，Cache 操作完全可以统一，至于 T-Head 平台，Common 中的 Cache 实现均为 weak 声明，T-Head 重新实现相关接口即可。 设备树平台将自动扫描 CPU 节点中 MMU support Sv32/Sv39/Sv48/Sv57/Sv64, SVPBMT ISA.各种页表翻译模式本身就是统一的，只有相关 PTE 位允许厂商自己修改实现，以支持 PMA，NOCACHE，IO 属性，ISA SVPBMT 中已经对前面提到的属性进行了统一，只有 T-Head 平台没支持该 ISA，直接使用 Vendor 声明的 同时对于 MMU 配置，Vendor 可以使用 为了考虑不同平台支持的 Sv 模式不同，提供了 Merge M-mode, S-mode CSR ops.重新提供统一的 CSR 寄存器描述表，其中 M-mode，S-mode 大部分寄存器仅是前缀不一样，使用 Support SMP.这块支持最大的问题就是，CPU ID 的获取。 在其他操作系统上，多任务和多核会非常早启动，其使用了进程中的 CPU 成员来获取 CPU ID，进程的信息存放在 RT-Thread 上的 M-mode 下无 MMU 不能支持 LWP，因此在该模式下， 支持 CPU 身份识别后，对于 SMP 上下文调度支持，参考架构 SMP 移植要求即可。 Support APLIC & IMSIC (AIA), CLIC, ACLINT/CLINT, INTC, PLIC.这个是刚需，这个外设 IP 就是统一的，代码也应该只有一份，对于一些平台有 quirk 的情况，代码里面已经使用 quirk bitmap 管理以支持不同特性的平台。 对于 AIA 架构，要求强制使用 DM 模式，而 CLIC, ACLINT/CLINT, INTC, PLIC 则使用与 DM 版本代码对具体硬件操作进行 API 封装，在 Support DM (only by OFW) or not DM.RISC-V 在其他较先进的 OS 中本身就支持设备树 DM 模式，只要平台支持好 DM 要求的资源管理接口即可。 由于 RISC-V 存在资源紧张的平台，代码重构早期就考虑到了部分代码与 DM 版本共享，以及相关属性使用静态配置的方式实现支持，前面的描述已经讲清楚支持方式。 Merge T-HEAD vendor.T-HEAD 本身大部分硬件特性就是固定的，没必要拆成 C906 和 C908，而且该 Vendor 遵守大部分 ISA，只要保留部分细微的差异即可，前面的描述已经讲清楚支持方式。 Kernel support boot in not CPU#0
RT-Thread 对 CPU#0 依赖的代码很明显，主要是系统 tick 管理，以及 idle 线程的主核心任务，修改相关对 CPU#0 的显示指定即可。 以上的描述可能不够清晰，如果有疑问可以进一步讨论。 |
|
我有如下想法，供大家参考：
所以如果从 kernel 的角度确定无法循序渐进地 pr（即只能一次性 merge），我觉得一种可行的办法就是需要确定与 riscv 相关的需要重点看护的 bsp，至少在 review 过程中做好充分的测试，并保证这些重点看护 bsp 可以基本正常工作后该 pr 才可以合入。 我这里目前维护的 riscv board 有两个：
其他关心这个 PR 的 bsp 有哪些，欢迎补充。 另外，我也看到 PR 说：
所以我理解这个 PR 并没有急着要合入，所以不妨在上面我提的第一个思路，即拉入更多关心此 PR 的 reviewer 基础上，我们在 review 这个 PR 的同时也考虑拆分的问题。我相信经过大家的努力，或许也是有可能拆分的（其实我大概看了一下 作者 上面的详细的描述，感觉有些功能点还是比较独立的，有很大可能性可以拆分出来先提交进 master， 我也有意愿参与这份工作）。 最后顺便提一下，我建议再找一些原先做 libcpu 下的人来一起 review，我参与 rtt 工作以来主要也是在 bsp 上多一些，有些 kenrel 公共部分还希望有更多有经验的人来一起 review。 |
|
重点看护的 BSP 肯定是要保证正常运行的，这个 PR 显然提交之前也清楚难以一两个人就能 review 好，但是目前据我认识的相关 RISC-V 前维护者已经不在 RT-Thread 社区活跃，如果是这样的话也还有个比较好的办法，那就是先提交到开发分支，在该分支上进行长期（显然时间肯定不会太长，因为近期可能也会有新的 Vendor 和 BSP 引入）的验证和重新适配，最后再变基并一次性合并到主线也可以。 |
也可以，如果能确保你这个 pr 的关联开发分支 https://github.com/GuEe-GUI/rt-thread/tree/next_riscv 不定期 rebase 到最新的 master 就行。 Review 过程中如果发现一些可以单独摘出来的 feature 也可以随时拿出来单独提 pr 先合入 master，然后 next_riscv 继续 rebase 到最新的 master。如此循环几次就可以实现化整为零，分批次提交了。 |
|
pr里提到的功能，比如cbo指令支持、smp功能支持、M mode支持、DM支持这些确实是目前riscv 平台需要实现的，这个pr相当于是对目前riscv平台相关代码的重构。个人同意@unicornx的观点，尽可能将pr内容拆开，cbo指令支持等这些相对独立的功能可以单独merge。我在rtt 5.2.0版本上基于我们的dp1000 平台目前也实现了pr里的部分功能，后续我可以参与该pr的review。 |
|
确定需要把rv32、rv64融合起来？确定以这种方式，后续各个riscv芯片厂商间可以融合在一起，可以统一在一起？ |
|
我的小pr 啥时融入 |
理论上可以，涉及到用户态和虚拟内存可能要注意下 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-10-02 15:08 CST)
📝 Review Instructions
|
|
|
@GuEe-GUI 在 环境：Apple Silicon，工具链 xPack riscv-none-elf-gcc 14.3.0，模拟器是 XUANTIE-RV/qemu 的 这块原来走 common64 和 BSP 自带的 libcpu。改成 编的时候碰到两个问题，不修过不去：
|
|
@GuEe-GUI 补一下 环境：Apple Silicon，xPack riscv-none-elf-gcc 14.3.0，上游 QEMU 10.1.5，启动进来的 OpenSBI 是 v1.5.1。 RV64 是 4 核，用的是 RV32 把 XLEN 改成 根分区 elm 没挂上（ 另外试了 smartl e907，M-Mode，没开 MMU，也没开设备树。玄铁 QEMU 8.2.94， 改动还在本地，没推到分支上。 |
|
@GuEe-GUI 补一笔 RV64 默认 AIA 的结果，跟上一条不是同一套模拟器。 上一条是 QEMU 10.1.5、OpenSBI v1.5.1，默认开 AIA 会卡在 PCI，所以那次用了 这组日志是 OpenSBI v1.4， 后面在 QEMU 10.1.5 上再跑默认 AIA，还是会卡。所以这条只能说明：OpenSBI v1.4 这套能进 msh，10.1.5 那套默认配置没跑通。 |
|
好的，稍等一下 |
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-10-02 23:17 CST)
📝 Review Instructions
|
|
AIA 问题已修复 |
|
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-10-02 23:43 CST)
📝 Review Instructions
|
|
|
新头 连续 3 次都进了 msh。IPI 是 上次卡在 PCI、NVMe 超时是旧头 |
|
头 不设环境变量重新编译，命令行里是 xlen = '64' if config_enabled('ARCH_RISCV64') else '32'
- abi_base = 'lp64' if xlen == '64' else 'ilp32'
extensions = 'imac'
+ abi_base = 'lp64' if xlen == '64' else 'ilp32'
if config_enabled('ARCH_RISCV_FPU_D'):
extensions = 'imafdc'
+ abi_base = 'lp64d' if xlen == '64' else 'ilp32d'
elif config_enabled('ARCH_RISCV_FPU_S'):
extensions = 'imafc'
+ abi_base = 'lp64f' if xlen == '64' else 'ilp32f'
abi = RISCV_ABI
- if not abi and (xlen == '32' or config_enabled('RT_USING_SMART')):
+ if not abi:
abi = abi_base改动还在本地，没推到分支上。 |
|
virt64 已经 checkout 回去，smartl c907 可以试试这个补丁： |
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-10-03 02:45 CST)
📝 Review Instructions
|
|
Introduce an opt-in unified RISC-V common layer for RV32 and RV64 while keeping existing BSPs on the legacy implementation during migration. - use ARCH_RISCV and ARCH_CPU_64BIT for architecture and XLEN selection - unify context, trap, MMU, cache, timer, SBI, SMP and ISA handling - add PLIC, CLIC, ACLINT, APLIC and IMSIC support - map physical hart IDs to contiguous logical CPU IDs - add T-Head vendor overrides while preserving legacy ports - migrate qemu-virt64-riscv with RV32 and RV64 support - update clock_time, MM, LWP, OFW, PCI and Rust integration Existing BSPs continue using legacy_common or common64 unless they select RT_USING_RISCV_NEW_COMMON. Signed-off-by: GuEe-GUI <2991707448@qq.com>
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-10-03 09:35 CST)
📝 Review Instructions
|
|
This branch has not been deployed
12 participants
拉取/合并请求描述：(PR description)
[
qemu-virt64-riscv64Plan:
Depend on:
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality：
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up