fix: fix chroot and tmpfs

[fslongjin]

No description provided.

[fslongjin]

@claude review this PR in Chinese

[dragonos-community-ai-pr-reviewer]

Claude finished @fslongjin's task in 3m 46s —— View job

---

Claude正在审阅PR #1480

待办清单：

• 分析PR上下文和变更文件
• 查看主要变更文件内容
• 分析chroot相关修复
• 分析tmpfs相关修复
• 检查代码质量和潜在问题
• 提供中文审阅反馈

详细审阅报告

已全面分析PR #1480 "fix: fix chroot and tmpfs"。以下是详细的代码审阅反馈：

✅ 主要改进

1. chroot系统调用实现完整正确

   • 新增 sys_chroot.rs 实现了完整的 chroot(2) 系统调用
   • 严格遵循Linux语义：需要 CAP_SYS_CHROOT 权限，路径必须是目录且有执行权限
   • 正确处理路径解析：绝对路径基于进程的 fs root（受chroot影响），相对路径基于cwd
   • 成功后只更新进程的 fs root，不改变 cwd（符合Linux行为）

2. tmpfs架构升级

   • 从简单的 Vec<u8> 升级为基于 PageCache 的现代文件系统架构
   • 新增 fault() 和 map_pages() 方法，支持内存映射和页错误处理
   • 统一 truncate() 到 resize()，保证扩展/收缩逻辑一致性

3. 重命名逻辑修复

   • 新增 tmpfs_move_entry_between_dirs() 处理跨目录重命名
   • 正确处理 RENAME_NOREPLACE、目录非空检查等边界情况
   • 修复了inode父子关系更新和硬链接计数维护

4. VFS基础设施增强

   • user_path_at() 注释明确绝对路径从进程 fs root 开始解析
   • 进程 fs_struct 支持设置root，为chroot提供基础设施
   • 页缓存管理更加完善

⚠️ 潜在问题与建议

1. 锁顺序潜在风险 (tmpfs/mod.rs:tmpfs_move_entry_between_dirs)

   let inode_to_move = src_dir.children.get(old_key).cloned()...;
   let old_type = inode_to_move.0.lock().metadata.file_type;  // 获取子inode锁

   • 函数接收 &mut TmpfsInode（调用者已持有锁），内部又获取子inode锁
   • 建议：明确锁顺序约定（如按inode_id排序），避免未来死锁

2. 错误码一致性

   • tmpfs_move_entry_between_dirs 中 Arc::ptr_eq(existing, &dst_self) 返回 EBUSY
   • Linux在重命名到自身时可能返回 EINVAL 或 EBUSY，需要确认是否符合预期

3. 页缓存生命周期管理

   • TmpfsInode 中的 page_cache: Option<Arc<PageCache>> 需要确保正确释放
   • 检查 resize() 实现是否正确处理页缓存的截断和扩展

4. 安全检查

   • sys_chroot.rs 使用 check_and_clone_cstr（应已重命名为 vfs_check_and_clone_cstr）
   • 确认用户空间字符串复制长度限制 MAX_PATHLEN 合理

📝 代码质量评价

优点：

• 代码结构清晰，符合Rust安全规范
• 错误处理完整，返回适当的系统错误码
• 注释详细，特别是Linux语义说明很有价值
• 模块化设计，新增函数职责单一

待改进：

• 部分函数较长（如 tmpfs_move_entry_between_dirs），可考虑进一步拆分
• 缺少一些边界情况的单元测试
• 锁的使用文档可以更详细

🔧 测试建议

1. chroot测试用例

   • 权限测试：无CAP_SYS_CHROOT应返回EPERM
   • 路径测试：非目录、不存在、无执行权限等情况
   • 嵌套chroot和文件访问测试

2. tmpfs重命名测试

   • 跨目录重名替换
   • RENAME_NOREPLACE 标志测试
   • 目录非空时的重命名
   • 重命名到自身的边界情况

3. 页缓存一致性测试

   • 文件截断后的内存映射访问
   • 并发读写下的数据一致性
   • 内存压力下的页回收

🎯 总体评价

这是一个高质量的PR，主要解决了两个重要问题：

1. 实现了缺失的chroot系统调用，完善了进程隔离功能
2. 重构了tmpfs文件系统，从简单内存存储升级为基于页缓存的现代架构

建议合并，但推荐在合并前：

1. 添加锁顺序约定的注释
2. 补充关键边界情况的测试用例
3. 确认所有错误码与Linux行为一致

代码变更量大但结构清晰，体现了对文件系统语义的深入理解。chroot实现符合POSIX/Linux标准，tmpfs重构为后续性能优化奠定了基础。

---

审阅完成 - 所有待办事项已完成