[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241125175111.8598-1-quic_jiaymao@quicinc.com>
Date: Tue, 26 Nov 2024 01:51:11 +0800
From: Jiayang Mao <quic_jiaymao@...cinc.com>
To: Marcel Holtmann <marcel@...tmann.org>,
Johan Hedberg
<johan.hedberg@...il.com>,
Luiz Augusto von Dentz <luiz.dentz@...il.com>
CC: <linux-bluetooth@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<quic_chejiang@...cinc.com>, <quic_jiaymao@...cinc.com>
Subject: [PATCH v1] Bluetooth: hci_sync: clear cmd_sync_work_list when power off
Clear the remaining command in cmd_sync_work_list when BT is
performing power off. In some cases, this list is not empty after
power off. BT host will try to send more HCI commands.
This can cause unexpected results.
Signed-off-by: Jiayang Mao <quic_jiaymao@...cinc.com>
---
net/bluetooth/hci_sync.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/net/bluetooth/hci_sync.c b/net/bluetooth/hci_sync.c
index c86f4e42e..bc622d074 100644
--- a/net/bluetooth/hci_sync.c
+++ b/net/bluetooth/hci_sync.c
@@ -5139,6 +5139,7 @@ int hci_dev_close_sync(struct hci_dev *hdev)
{
bool auto_off;
int err = 0;
+ struct hci_cmd_sync_work_entry *entry, *tmp;
bt_dev_dbg(hdev, "");
@@ -5258,6 +5259,11 @@ int hci_dev_close_sync(struct hci_dev *hdev)
clear_bit(HCI_RUNNING, &hdev->flags);
hci_sock_dev_event(hdev, HCI_DEV_CLOSE);
+ mutex_lock(&hdev->cmd_sync_work_lock);
+ list_for_each_entry_safe(entry, tmp, &hdev->cmd_sync_work_list, list)
+ _hci_cmd_sync_cancel_entry(hdev, entry, -ECANCELED);
+ mutex_unlock(&hdev->cmd_sync_work_lock);
+
/* After this point our queues are empty and no tasks are scheduled. */
hdev->close(hdev);
--
2.25.1
Powered by blists - more mailing lists