[BUG REPORT] Potential inconsistent state in devfs_setup if devfs_register fails

[DiuDiu777]

描述错误
(1) devfs_setup 函数中的错误处理模式可能会导致设备处于“部分初始化”的状态。

DragonOS/kernel/src/driver/tty/virtual_terminal/mod.rs

Lines 102 to 125 in 9289553

	pub fn devfs_setup(&self) -> Result<(), SystemError> {
	let tty_core = self
	.port
	.port_data()
	.internal_tty()
	.ok_or(SystemError::ENODEV)?;
	let tty_core_data = tty_core.core();
	let devnum = *tty_core_data.device_number();
	let vcname = format!("vc{}", self.index.get());
	// 注册虚拟终端设备并将虚拟终端设备加入到文件系统
	let vcdev = TtyDevice::new(
	vcname.clone(),
	IdTable::new(vcname, Some(devnum)),
	TtyType::Tty,
	);
	device_register(vcdev.clone())?;
	devfs_register(vcdev.name_ref(), vcdev.clone())?;
	tty_core_data.set_vc_index(*self.index.get());
	self.inner.lock().vcdev = Some(vcdev);
	Ok(())
	}

• 119行：device_register 先将设备注册到全局设备层级结构中。

• 120行：随后 devfs_register 尝试在 DevFS 中创建对应的节点。

如果 device_register 执行成功，但随后的 devfs_register 返回错误，函数会提前返回。目前看起来，device_register 成功后产生的副作用似乎并没有被回滚。

请问这是否可能会导致 vcdev 虽然在全局 kset/设备列表中注册了，但却没有对应的 DevFS 入口，实际上变成了一个占用了设备名但无法正常工作的设备？

(2) 此外，这是否还可能存在一个并发风险？

• 120 行：注册到 devfs 后，用户空间可能立即看到并打开设备文件。
• 122 行：self.inner.lock().vcdev = Some(vcdev) 执行前，VirtConsole 内部还没有持有该设备。

也就是如果用户在第 120 行和 122 行之间打开设备并触发回调，驱动程序有可能会因为在 inner 中找不到 vcdev 而出错？

版本信息：

• DragonOS版本hash：92895534f7c053abb5970491fca9af2cd002b5c1

[fslongjin]

是的，你说得对，这是个问题，当初开发的时候没有很认真的处理这个问题。
这个PR #1261 里面bug bot有提到，我会在1261这个PR里面尝试解决部分问题。对于你提到的VC相关的，在#1261合并之后，我去再研究一下这个问题哈～
#1261

[DiuDiu777]

感谢回复！

除此之外，似乎还有几个事务回滚相关的潜在问题，想麻烦你们也看看～

DragonOS/kernel/src/virt/vm/kvm_host/mod.rs

Lines 150 to 184 in fa4d785

	impl Vm {
	#[inline(never)]
	pub fn create_vcpu(&mut self, id: usize) -> Result<usize, SystemError> {
	if id >= self.max_vcpus {
	return Err(SystemError::EINVAL);
	}
	if self.created_vcpus >= self.max_vcpus {
	return Err(SystemError::EINVAL);
	}
	self.created_vcpus += 1;
	let vcpu = self._create_vcpu(id)?;
	if self.dirty_ring_size != 0 {
	todo!()
	}
	vcpu.lock().vcpu_id = self.online_vcpus;
	self.vcpus.insert(self.online_vcpus, vcpu.clone());
	self.online_vcpus += 1;
	let vcpu_inode = KvmVcpuDev::new(vcpu);
	let file = File::new(vcpu_inode, FileFlags::from_bits_truncate(0x777))?;
	let fd = ProcessManager::current_pcb()
	.fd_table()
	.write()
	.alloc_fd(file, None)?;
	Ok(fd as usize)
	}

(1) 161-163行： 先增加 created_vcpus 计数，后续 self._create_vcpu(id)? 若失败没有对这个计数进行回退？

(2) 170-181行：

self.vcpus.insert(self.online_vcpus, vcpu.clone()); // 1. 插入了集合
self.online_vcpus += 1; // 2. 增加了在线计数
// ...
let fd = ProcessManager::current_pcb() 
   .fd_table() 
   .write() 
   .alloc_fd(file, None)?; 
// 3. 如果 alloc_fd 失败（例如文件描述符耗尽），函数返回 Err。
// 但是 vcpu 已经留在了 self.vcpus 中，且计数器已增加。
// 这是否会导致在这个 Vm 生命周期内，始终有僵尸vCPU 占用了位置但无法被用户态访问。

感觉二者对Error返回做个处理会更好一些？