MGBA模拟器中InputController线程安全问题分析与修复

问题背景

在MGBA游戏模拟器的Qt平台实现中，InputController类负责处理用户输入事件。开发者发现了一个随机崩溃问题，该问题与输入事件处理的多线程安全性有关。

崩溃原因分析

从崩溃堆栈可以清晰地看到问题发生在InputController::hasPendingEvent()方法中，该方法被InputController::pollEvents()调用。崩溃的根本原因是多线程环境下对QSet类型成员变量m_pendingEvents的非同步访问。

具体来说：

1. 主线程和模拟器核心线程可能同时访问m_pendingEvents
2. QSet/QHash内部数据结构在没有同步保护的情况下被并发访问
3. 当哈希表正在rehash时，另一个线程尝试访问，导致内存访问冲突

历史背景

这个问题实际上在几年前就已经被发现并修复过(提交ca67e63)，但在后续重构时(提交0c77227)意外地移除了保护锁，导致问题重现。这提醒我们在重构代码时需要特别注意线程安全机制的保护。

技术细节

InputController类中的关键数据结构：

• m_pendingEvents：存储待处理输入事件的集合
• m_mutex：原本用于保护集合访问的互斥锁

问题代码路径：

1. 模拟器核心线程通过CoreController::updateKeys()调用InputController::pollEvents()
2. pollEvents()需要检查m_pendingEvents集合
3. 同时GUI线程可能正在修改这个集合

解决方案

正确的修复方法是恢复对m_pendingEvents的所有访问的互斥锁保护。具体包括：

1. 在hasPendingEvent()方法中添加QMutexLocker
2. 确保所有修改m_pendingEvents的地方都有锁保护
3. 保持锁的粒度合理，避免长时间持有锁

经验教训

这个案例提供了几个重要的开发经验：

1. 线程安全代码需要特别关注，重构时容易破坏原有保护机制
2. 所有共享数据的访问点都需要保护，遗漏任何一个都可能导致问题
3. 随机崩溃往往是线程安全问题的典型表现
4. 代码历史记录可以帮助诊断重现的问题

结论

多线程编程中的资源共享问题需要开发者保持高度警惕。在MGBA这样的模拟器中，输入处理作为连接用户界面和模拟核心的关键路径，其线程安全性尤为重要。通过恢复适当的互斥锁保护，确保了输入事件处理的可靠性和稳定性。