BTstack项目中AVRCP连接问题的分析与修复

问题背景

在蓝牙协议栈BTstack的开发过程中，开发者发现了一个关于AVRCP(音频/视频远程控制协议)连接的问题。当设备在已经建立A2DP(高级音频分发协议)连接后尝试建立AVRCP连接时，连接过程会卡住，无法正常完成。这个问题特别出现在同时连接多个蓝牙协议(如HFP、PBAP等)的场景下。

问题现象

具体表现为：

1. 设备成功建立A2DP Sink连接后
2. 尝试建立AVRCP连接时，L2CAP创建连接通道的步骤没有被触发
3. SDP(服务发现协议)客户端的回调函数没有被正常执行
4. sdp_client_query_requests列表中仍然保留着avrcp_handle_start_sdp_client_query，但没有被执行

技术分析

经过深入分析，发现问题根源在于BTstack的SDP客户端处理机制存在缺陷。具体表现为：

1. SDP查询请求队列处理不完整：原代码中sdp_client_notify_callbacks函数只处理队列中的第一个请求，然后就返回。当有多个SDP查询请求排队时，后续请求会被遗漏。

2. AVDTP连接状态管理问题：在AVDTP(音频/视频分发传输协议)连接过程中，SDP查询请求的注册时机不当。即使连接状态已经改变，不再需要SDP查询，请求仍然被保留在队列中。

3. 多协议并发问题：当同时使用多个蓝牙协议(特别是PBAP协议)时，这个问题更容易显现，因为多个协议会并发发起SDP查询请求。

解决方案

开发团队提出了两个关键修复：

1. 改进SDP客户端回调机制：将sdp_client_notify_callbacks函数改为循环处理所有就绪的SDP查询请求，而不是只处理第一个。这确保了所有注册的回调都能得到执行机会。

static void sdp_client_notify_callbacks(void){
    while (sdp_client_ready() == true) {
        btstack_context_callback_registration_t * callback = 
            (btstack_context_callback_registration_t*) btstack_linked_list_pop(&sdp_client_query_requests);
        if (callback == NULL) {
            return;
        }
        (*callback->callback)(callback->context);
    }
}

2. 优化AVDTP的SDP查询注册逻辑：修改AVDTP代码，仅在确实需要SDP查询时才注册回调，避免不必要的查询请求排队。

技术启示

这个问题的解决过程给我们几点重要启示：

1. 蓝牙协议栈的复杂性：蓝牙协议栈中各个协议层相互影响，一个协议的行为可能影响其他协议的正常工作。

2. 状态管理的重要性：在协议实现中，必须严格管理各种连接状态，确保在状态变更时正确处理相关的资源和服务请求。

3. 队列处理的完整性：对于请求队列的处理必须完整，不能假设每次只处理一个请求就能满足所有场景。

4. 多协议并发测试的必要性：蓝牙设备通常需要支持多个协议同时工作，测试时必须考虑多协议并发的场景。

总结

通过对BTstack中AVRCP连接问题的分析和修复，我们不仅解决了特定场景下的连接问题，还改进了协议栈的核心处理机制。这个案例展示了蓝牙协议栈开发中的典型挑战，以及如何通过深入分析协议交互和状态管理来解决问题。这些改进已被合并到BTstack的主干代码中，为后续开发提供了更健壮的基础。