* [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware
@ 2025-08-13 3:35 Shuai Zhang
2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang
` (3 more replies)
0 siblings, 4 replies; 16+ messages in thread
From: Shuai Zhang @ 2025-08-13 3:35 UTC (permalink / raw)
To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang, linux-kernel
This patch series addresses issues encountered during SSR when
the BT_EN pin is pulled up by hardware. The main issues fixed are:
1. Timeout when sending reset command.
2. IBS state of host and controller not being synchronized.
3. Multiple triggers of SSR generating only one coredump file.
4. SSR process failed due to tx_idle_timer timeout
Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com>
---
To: Marcel Holtmann <marcel@holtmann.org>
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: quic_bt@quicinc.com
---
Changes in v2:
- Update commit messages.
- Add version number.
- Add new change 4/4 patch to fix Idle_timer timeout.
- Link to v1/2: https://lore.kernel.org/all/20250715051618.724475-1-quic_shuaz@quicinc.com/
---
Shuai Zhang (4):
driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw
driver: bluetooth: hci_qca: fix host IBS state after SSR
driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one
coredump file
driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due
to tx_idle_timer timeout
drivers/bluetooth/hci_qca.c | 29 +++++++++++++++++++++++++++++
1 file changed, 29 insertions(+)
--
2.34.1
^ permalink raw reply [flat|nested] 16+ messages in thread* [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw 2025-08-13 3:35 [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware Shuai Zhang @ 2025-08-13 3:35 ` Shuai Zhang 2025-08-13 4:18 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot 2025-08-13 11:05 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Konrad Dybcio 2025-08-13 3:35 ` [PATCH v3 2/4] driver: bluetooth: hci_qca: fix host IBS state after SSR Shuai Zhang ` (2 subsequent siblings) 3 siblings, 2 replies; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 3:35 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang When the host actively triggers SSR and collects coredump data, the Bluetooth stack sends a reset command to the controller. However,due to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, the reset command times out. For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to commit: 740011cfe94859df8d05f5400d589a8693b095e7. The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) because this quirk is associated with BT_EN, and can be used to determine whether BT_EN is present in the device tree (DTS). Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 4e56782b0..14b2d1bee 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) skb_queue_purge(&qca->rx_memdump_q); } + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V + * power supply via hardware the driver + * cannot control the bt_en pin of the SoC chip, then during SSR, + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. + * This leads to a reset command timeout failure. + * Also, add msleep delay to wait for controller to complete SSR. + */ + if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { + clear_bit(QCA_SSR_TRIGGERED, &qca->flags); + clear_bit(QCA_IBS_DISABLED, &qca->flags); + msleep(50); + } + clear_bit(QCA_HW_ERROR_EVENT, &qca->flags); } -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* RE: Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware 2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang @ 2025-08-13 4:18 ` bluez.test.bot 2025-08-13 11:05 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Konrad Dybcio 1 sibling, 0 replies; 16+ messages in thread From: bluez.test.bot @ 2025-08-13 4:18 UTC (permalink / raw) To: linux-bluetooth, quic_shuaz [-- Attachment #1: Type: text/plain, Size: 25137 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=990820 ---Test result--- Test Summary: CheckPatch PENDING 0.49 seconds GitLint PENDING 0.28 seconds SubjectPrefix FAIL 0.61 seconds BuildKernel FAIL 20.34 seconds CheckAllWarning FAIL 22.56 seconds CheckSparse FAIL 24.94 seconds BuildKernel32 FAIL 20.11 seconds TestRunnerSetup FAIL 458.78 seconds TestRunner_l2cap-tester FAIL 0.09 seconds TestRunner_iso-tester FAIL 0.09 seconds TestRunner_bnep-tester FAIL 0.09 seconds TestRunner_mgmt-tester FAIL 0.09 seconds TestRunner_rfcomm-tester FAIL 4.59 seconds TestRunner_sco-tester FAIL 0.14 seconds TestRunner_ioctl-tester FAIL 0.14 seconds TestRunner_mesh-tester FAIL 0.09 seconds TestRunner_smp-tester FAIL 0.09 seconds TestRunner_userchan-tester FAIL 0.09 seconds IncrementalBuild PENDING 0.95 seconds Details ############################## Test: CheckPatch - PENDING Desc: Run checkpatch.pl script Output: ############################## Test: GitLint - PENDING Desc: Run gitlint Output: ############################## Test: SubjectPrefix - FAIL Desc: Check subject contains "Bluetooth" prefix Output: "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject ############################## Test: BuildKernel - FAIL Desc: Build Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckAllWarning - FAIL Desc: Run linux kernel with all warning enabled Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckSparse - FAIL Desc: Run sparse tool with linux kernel Output: net/bluetooth/af_bluetooth.c:248:25: warning: context imbalance in 'bt_accept_enqueue' - different lock contexts for basic block net/bluetooth/hci_core.c:85:9: warning: context imbalance in '__hci_dev_get' - different lock contexts for basic block net/bluetooth/hci_core.c: note: in included file (through include/linux/notifier.h, include/linux/memory_hotplug.h, include/linux/mmzone.h, include/linux/gfp.h, include/linux/xarray.h, include/linux/radix-tree.h, ...): ./include/linux/srcu.h:400:9: warning: context imbalance in 'hci_dev_put_srcu' - unexpected unlock net/bluetooth/hci_event.c: note: in included file (through include/net/bluetooth/hci_core.h): ./include/net/bluetooth/hci.h:2657:47: warning: array of flexible structures ./include/net/bluetooth/hci.h:2743:43: warning: array of flexible structures In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... drivers/bluetooth/hci_ag6xx.c:257:24: warning: restricted __le32 degrades to integer make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... net/bluetooth/hci_codec.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures net/bluetooth/sco.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: BuildKernel32 - FAIL Desc: Build 32bit Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunnerSetup - FAIL Desc: Setup kernel and bluez for test-runner Output: Kernel: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[3]: *** Waiting for unfinished jobs.... make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunner_l2cap-tester - FAIL Desc: Run l2cap-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_iso-tester - FAIL Desc: Run iso-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_bnep-tester - FAIL Desc: Run bnep-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mgmt-tester - FAIL Desc: Run mgmt-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_rfcomm-tester - FAIL Desc: Run rfcomm-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_sco-tester - FAIL Desc: Run sco-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_ioctl-tester - FAIL Desc: Run ioctl-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mesh-tester - FAIL Desc: Run mesh-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_smp-tester - FAIL Desc: Run smp-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_userchan-tester - FAIL Desc: Run userchan-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: IncrementalBuild - PENDING Desc: Incremental build with the patches in the series Output: --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw 2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang 2025-08-13 4:18 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot @ 2025-08-13 11:05 ` Konrad Dybcio 2025-08-13 11:28 ` Shuai Zhang 1 sibling, 1 reply; 16+ messages in thread From: Konrad Dybcio @ 2025-08-13 11:05 UTC (permalink / raw) To: Shuai Zhang, linux-bluetooth, linux-arm-msm; +Cc: quic_bt On 8/13/25 5:35 AM, Shuai Zhang wrote: > When the host actively triggers SSR and collects coredump data, > the Bluetooth stack sends a reset command to the controller. However,due > to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, > the reset command times out. > > For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to > commit: 740011cfe94859df8d05f5400d589a8693b095e7. > > The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) > because this quirk is associated with BT_EN, and can be used to > determine whether BT_EN is present in the device tree (DTS). > > Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> > --- > drivers/bluetooth/hci_qca.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c > index 4e56782b0..14b2d1bee 100644 > --- a/drivers/bluetooth/hci_qca.c > +++ b/drivers/bluetooth/hci_qca.c > @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) > skb_queue_purge(&qca->rx_memdump_q); > } > > + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V > + * power supply via hardware the driver weird line break > + * cannot control the bt_en pin of the SoC chip, then during SSR, "System on a Chip chip" > + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. > + * This leads to a reset command timeout failure. > + * Also, add msleep delay to wait for controller to complete SSR. 50 is a random value, I also see 300, 100 across the driver.. where does it come from? Konrad > + */ > + if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { > + clear_bit(QCA_SSR_TRIGGERED, &qca->flags); > + clear_bit(QCA_IBS_DISABLED, &qca->flags); > + msleep(50); > + } > + > clear_bit(QCA_HW_ERROR_EVENT, &qca->flags); > } > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw 2025-08-13 11:05 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Konrad Dybcio @ 2025-08-13 11:28 ` Shuai Zhang 2025-08-13 11:32 ` Konrad Dybcio 0 siblings, 1 reply; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 11:28 UTC (permalink / raw) To: Konrad Dybcio, linux-bluetooth, linux-arm-msm; +Cc: quic_bt Hi, Konrad On 8/13/2025 7:05 PM, Konrad Dybcio wrote: > On 8/13/25 5:35 AM, Shuai Zhang wrote: >> When the host actively triggers SSR and collects coredump data, >> the Bluetooth stack sends a reset command to the controller. However,due >> to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, >> the reset command times out. >> >> For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to >> commit: 740011cfe94859df8d05f5400d589a8693b095e7. >> >> The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) >> because this quirk is associated with BT_EN, and can be used to >> determine whether BT_EN is present in the device tree (DTS). >> >> Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> >> --- >> drivers/bluetooth/hci_qca.c | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> >> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c >> index 4e56782b0..14b2d1bee 100644 >> --- a/drivers/bluetooth/hci_qca.c >> +++ b/drivers/bluetooth/hci_qca.c >> @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) >> skb_queue_purge(&qca->rx_memdump_q); >> } >> >> + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V >> + * power supply via hardware the driver > > weird line break > >> + * cannot control the bt_en pin of the SoC chip, then during SSR, > > "System on a Chip chip" I will modify. > >> + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. >> + * This leads to a reset command timeout failure. >> + * Also, add msleep delay to wait for controller to complete SSR. > > 50 is a random value, I also see 300, 100 across the driver.. where does it > come from? > The controller requires 50ms to stabilize after power-on, so we wait 50ms after the coredump completes > Konrad > >> + */ >> + if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { >> + clear_bit(QCA_SSR_TRIGGERED, &qca->flags); >> + clear_bit(QCA_IBS_DISABLED, &qca->flags); >> + msleep(50); >> + } >> + >> clear_bit(QCA_HW_ERROR_EVENT, &qca->flags); >> } >> BR, Shuai ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw 2025-08-13 11:28 ` Shuai Zhang @ 2025-08-13 11:32 ` Konrad Dybcio 2025-08-14 11:41 ` Shuai Zhang 0 siblings, 1 reply; 16+ messages in thread From: Konrad Dybcio @ 2025-08-13 11:32 UTC (permalink / raw) To: Shuai Zhang, linux-bluetooth, linux-arm-msm; +Cc: quic_bt On 8/13/25 1:28 PM, Shuai Zhang wrote: > Hi, Konrad > > On 8/13/2025 7:05 PM, Konrad Dybcio wrote: >> On 8/13/25 5:35 AM, Shuai Zhang wrote: >>> When the host actively triggers SSR and collects coredump data, >>> the Bluetooth stack sends a reset command to the controller. However,due >>> to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, >>> the reset command times out. >>> >>> For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to >>> commit: 740011cfe94859df8d05f5400d589a8693b095e7. >>> >>> The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) >>> because this quirk is associated with BT_EN, and can be used to >>> determine whether BT_EN is present in the device tree (DTS). >>> >>> Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> >>> --- >>> drivers/bluetooth/hci_qca.c | 13 +++++++++++++ >>> 1 file changed, 13 insertions(+) >>> >>> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c >>> index 4e56782b0..14b2d1bee 100644 >>> --- a/drivers/bluetooth/hci_qca.c >>> +++ b/drivers/bluetooth/hci_qca.c >>> @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) >>> skb_queue_purge(&qca->rx_memdump_q); >>> } >>> >>> + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V >>> + * power supply via hardware the driver >> >> weird line break >> >>> + * cannot control the bt_en pin of the SoC chip, then during SSR, >> >> "System on a Chip chip" > > I will modify. > >> >>> + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. >>> + * This leads to a reset command timeout failure. >>> + * Also, add msleep delay to wait for controller to complete SSR. >> >> 50 is a random value, I also see 300, 100 across the driver.. where does it >> come from? >> > The controller requires 50ms to stabilize after power-on, > so we wait 50ms after the coredump completes Is that common for all controllers supported by this driver? Please #define this value somewhere Konrad ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw 2025-08-13 11:32 ` Konrad Dybcio @ 2025-08-14 11:41 ` Shuai Zhang 0 siblings, 0 replies; 16+ messages in thread From: Shuai Zhang @ 2025-08-14 11:41 UTC (permalink / raw) To: Konrad Dybcio, linux-bluetooth, linux-arm-msm; +Cc: quic_bt On 8/13/2025 7:32 PM, Konrad Dybcio wrote: > On 8/13/25 1:28 PM, Shuai Zhang wrote: >> Hi, Konrad >> >> On 8/13/2025 7:05 PM, Konrad Dybcio wrote: >>> On 8/13/25 5:35 AM, Shuai Zhang wrote: >>>> When the host actively triggers SSR and collects coredump data, >>>> the Bluetooth stack sends a reset command to the controller. However,due >>>> to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, >>>> the reset command times out. >>>> >>>> For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to >>>> commit: 740011cfe94859df8d05f5400d589a8693b095e7. >>>> >>>> The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) >>>> because this quirk is associated with BT_EN, and can be used to >>>> determine whether BT_EN is present in the device tree (DTS). >>>> >>>> Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> >>>> --- >>>> drivers/bluetooth/hci_qca.c | 13 +++++++++++++ >>>> 1 file changed, 13 insertions(+) >>>> >>>> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c >>>> index 4e56782b0..14b2d1bee 100644 >>>> --- a/drivers/bluetooth/hci_qca.c >>>> +++ b/drivers/bluetooth/hci_qca.c >>>> @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) >>>> skb_queue_purge(&qca->rx_memdump_q); >>>> } >>>> >>>> + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V >>>> + * power supply via hardware the driver >>> >>> weird line break >>> >>>> + * cannot control the bt_en pin of the SoC chip, then during SSR, >>> >>> "System on a Chip chip" >> >> I will modify. >> >>> >>>> + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. >>>> + * This leads to a reset command timeout failure. >>>> + * Also, add msleep delay to wait for controller to complete SSR. >>> >>> 50 is a random value, I also see 300, 100 across the driver.. where does it >>> come from? >>> >> The controller requires 50ms to stabilize after power-on, >> so we wait 50ms after the coredump completes > > Is that common for all controllers supported by this driver? > > Please #define this value somewhere Not all controllers require a stabilization period after power-on. However, if it is needed, 50ms is sufficient. > > Konrad ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v3 2/4] driver: bluetooth: hci_qca: fix host IBS state after SSR 2025-08-13 3:35 [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang @ 2025-08-13 3:35 ` Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout Shuai Zhang 3 siblings, 0 replies; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 3:35 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang After SSR, host will not download the firmware, causing controller to remain in the IBS_WAKE state. Host needs to synchronize with the controller to maintain proper operation. Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 14b2d1bee..50ec9b155 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1659,10 +1659,14 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. * This leads to a reset command timeout failure. * Also, add msleep delay to wait for controller to complete SSR. + * + * Host will not download the firmware after SSR, controller to remain + * in the IBS_WAKE state, and the host needs to synchronize with it */ if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { clear_bit(QCA_SSR_TRIGGERED, &qca->flags); clear_bit(QCA_IBS_DISABLED, &qca->flags); + qca->tx_ibs_state = HCI_IBS_TX_AWAKE; msleep(50); } -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file 2025-08-13 3:35 [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 2/4] driver: bluetooth: hci_qca: fix host IBS state after SSR Shuai Zhang @ 2025-08-13 3:35 ` Shuai Zhang 2025-08-13 11:11 ` Konrad Dybcio 2025-08-13 3:35 ` [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout Shuai Zhang 3 siblings, 1 reply; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 3:35 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang Multiple triggers of SSR only first generate coredump file, duo to memcoredump_flag no clear. add clear coredump flag when ssr completed. Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 50ec9b155..33c2dd7b8 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1662,11 +1662,14 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) * * Host will not download the firmware after SSR, controller to remain * in the IBS_WAKE state, and the host needs to synchronize with it + * + * clear memcoredump_flag to ensure next submission of coredump date. */ if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { clear_bit(QCA_SSR_TRIGGERED, &qca->flags); clear_bit(QCA_IBS_DISABLED, &qca->flags); qca->tx_ibs_state = HCI_IBS_TX_AWAKE; + qca->memdump_state = QCA_MEMDUMP_IDLE; msleep(50); } -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file 2025-08-13 3:35 ` [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file Shuai Zhang @ 2025-08-13 11:11 ` Konrad Dybcio 2025-08-13 11:30 ` Shuai Zhang 0 siblings, 1 reply; 16+ messages in thread From: Konrad Dybcio @ 2025-08-13 11:11 UTC (permalink / raw) To: Shuai Zhang, linux-bluetooth, linux-arm-msm; +Cc: quic_bt On 8/13/25 5:35 AM, Shuai Zhang wrote: > Multiple triggers of SSR only first generate coredump file, > duo to memcoredump_flag no clear. > > add clear coredump flag when ssr completed. > > Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> > --- Please read https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes and adjust your commit message accordingly. > drivers/bluetooth/hci_qca.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c > index 50ec9b155..33c2dd7b8 100644 > --- a/drivers/bluetooth/hci_qca.c > +++ b/drivers/bluetooth/hci_qca.c > @@ -1662,11 +1662,14 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) > * > * Host will not download the firmware after SSR, controller to remain > * in the IBS_WAKE state, and the host needs to synchronize with it > + * > + * clear memcoredump_flag to ensure next submission of coredump date. "Since the bluetooth chip has been reset, clear the memdump state" Konrad > */ > if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { > clear_bit(QCA_SSR_TRIGGERED, &qca->flags); > clear_bit(QCA_IBS_DISABLED, &qca->flags); > qca->tx_ibs_state = HCI_IBS_TX_AWAKE; > + qca->memdump_state = QCA_MEMDUMP_IDLE; > msleep(50); > } > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file 2025-08-13 11:11 ` Konrad Dybcio @ 2025-08-13 11:30 ` Shuai Zhang 0 siblings, 0 replies; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 11:30 UTC (permalink / raw) To: Konrad Dybcio, linux-bluetooth, linux-arm-msm; +Cc: quic_bt Hi,Konrad On 8/13/2025 7:11 PM, Konrad Dybcio wrote: > On 8/13/25 5:35 AM, Shuai Zhang wrote: >> Multiple triggers of SSR only first generate coredump file, >> duo to memcoredump_flag no clear. >> >> add clear coredump flag when ssr completed. >> >> Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> >> --- > > Please read > > https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes > > and adjust your commit message accordingly. > >> drivers/bluetooth/hci_qca.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c >> index 50ec9b155..33c2dd7b8 100644 >> --- a/drivers/bluetooth/hci_qca.c >> +++ b/drivers/bluetooth/hci_qca.c >> @@ -1662,11 +1662,14 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) >> * >> * Host will not download the firmware after SSR, controller to remain >> * in the IBS_WAKE state, and the host needs to synchronize with it >> + * >> + * clear memcoredump_flag to ensure next submission of coredump date. > > "Since the bluetooth chip has been reset, clear the memdump state" > > Konrad > I will update. >> */ >> if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { >> clear_bit(QCA_SSR_TRIGGERED, &qca->flags); >> clear_bit(QCA_IBS_DISABLED, &qca->flags); >> qca->tx_ibs_state = HCI_IBS_TX_AWAKE; >> + qca->memdump_state = QCA_MEMDUMP_IDLE; >> msleep(50); >> } >> BR, Shuai ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout 2025-08-13 3:35 [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware Shuai Zhang ` (2 preceding siblings ...) 2025-08-13 3:35 ` [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file Shuai Zhang @ 2025-08-13 3:35 ` Shuai Zhang 2025-08-13 11:13 ` Konrad Dybcio 3 siblings, 1 reply; 16+ messages in thread From: Shuai Zhang @ 2025-08-13 3:35 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang When the SSR (SubSystem Restart) duration exceeds 2 seconds, it triggers host tx_idle_timeout, which sets host TX state to sleep. due to the hardware pulling up bt_en, the firmware is not downloaded after the SSR. As a result, the controller does not enter sleep mode. Consequently, when the host sends a command afterward, it sends 0xFD to the controller, but the controller does not respond, leading to a command timeout. So reset tx_idle_timer after SSR to prevent host enter TX IBS_Sloeep mode. Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 33c2dd7b8..1fb8eea09 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1668,6 +1668,15 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { clear_bit(QCA_SSR_TRIGGERED, &qca->flags); clear_bit(QCA_IBS_DISABLED, &qca->flags); + /* + * When the SSR (Sub-System Restart) duration exceeds 2 seconds, + * it triggers host tx_idle_timeout, which sets host TX state + * to sleep. Reset tx_idle_timer after SSR to prevent + * host enter TX IBS_Sloeep mode. + */ + mod_timer(&qca->tx_idle_timer, jiffies + + msecs_to_jiffies(qca->tx_idle_delay)); + qca->tx_ibs_state = HCI_IBS_TX_AWAKE; qca->memdump_state = QCA_MEMDUMP_IDLE; msleep(50); -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout 2025-08-13 3:35 ` [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout Shuai Zhang @ 2025-08-13 11:13 ` Konrad Dybcio 2025-08-14 11:52 ` Shuai Zhang 0 siblings, 1 reply; 16+ messages in thread From: Konrad Dybcio @ 2025-08-13 11:13 UTC (permalink / raw) To: Shuai Zhang, linux-bluetooth, linux-arm-msm; +Cc: quic_bt On 8/13/25 5:35 AM, Shuai Zhang wrote: > When the SSR (SubSystem Restart) duration exceeds 2 seconds, it triggers > host tx_idle_timeout, which sets host TX state to sleep. due to the Host? Looks like we're sending a command to the BT chip and that triggers the issue > hardware pulling up bt_en, the firmware is not downloaded after the SSR. > As a result, the controller does not enter sleep mode. Consequently, > when the host sends a command afterward, it sends 0xFD to the controller, > but the controller does not respond, leading to a command timeout. > > So reset tx_idle_timer after SSR to prevent host enter TX IBS_Sloeep mode. > > Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> > --- > drivers/bluetooth/hci_qca.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c > index 33c2dd7b8..1fb8eea09 100644 > --- a/drivers/bluetooth/hci_qca.c > +++ b/drivers/bluetooth/hci_qca.c > @@ -1668,6 +1668,15 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) > if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { > clear_bit(QCA_SSR_TRIGGERED, &qca->flags); > clear_bit(QCA_IBS_DISABLED, &qca->flags); > + /* > + * When the SSR (Sub-System Restart) duration exceeds 2 seconds, This comment will become out of date quickly, refer to 'tx_idle_delay' instead Konrad > + * it triggers host tx_idle_timeout, which sets host TX state > + * to sleep. Reset tx_idle_timer after SSR to prevent > + * host enter TX IBS_Sloeep mode. > + */ > + mod_timer(&qca->tx_idle_timer, jiffies + > + msecs_to_jiffies(qca->tx_idle_delay)); > + > qca->tx_ibs_state = HCI_IBS_TX_AWAKE; > qca->memdump_state = QCA_MEMDUMP_IDLE; > msleep(50); ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout 2025-08-13 11:13 ` Konrad Dybcio @ 2025-08-14 11:52 ` Shuai Zhang 0 siblings, 0 replies; 16+ messages in thread From: Shuai Zhang @ 2025-08-14 11:52 UTC (permalink / raw) To: Konrad Dybcio, linux-bluetooth, linux-arm-msm; +Cc: quic_bt Dear,Konrad On 8/13/2025 7:13 PM, Konrad Dybcio wrote: > On 8/13/25 5:35 AM, Shuai Zhang wrote: >> When the SSR (SubSystem Restart) duration exceeds 2 seconds, it triggers >> host tx_idle_timeout, which sets host TX state to sleep. due to the > > Host? Looks like we're sending a command to the BT chip and that > triggers the issue Since the firmware is not downloaded after ssR, the controller does not enter the ibs_sleep state. If the host enters ibs_sleep, it will send a wake-up command before sending any other commands. However, the controller only processes IBS wake-up commands when it is already in ibs_sleep. As a result, it will not respond to the host's wake-up command, which prevents host from send actual command and leads to a send command timeout. Therefore, the host must not enter the ibs_sleep state. > >> hardware pulling up bt_en, the firmware is not downloaded after the SSR. >> As a result, the controller does not enter sleep mode. Consequently, >> when the host sends a command afterward, it sends 0xFD to the controller, >> but the controller does not respond, leading to a command timeout. >> >> So reset tx_idle_timer after SSR to prevent host enter TX IBS_Sloeep mode. >> >> Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> >> --- >> drivers/bluetooth/hci_qca.c | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> >> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c >> index 33c2dd7b8..1fb8eea09 100644 >> --- a/drivers/bluetooth/hci_qca.c >> +++ b/drivers/bluetooth/hci_qca.c >> @@ -1668,6 +1668,15 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) >> if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { >> clear_bit(QCA_SSR_TRIGGERED, &qca->flags); >> clear_bit(QCA_IBS_DISABLED, &qca->flags); >> + /* >> + * When the SSR (Sub-System Restart) duration exceeds 2 seconds, > > This comment will become out of date quickly, refer to 'tx_idle_delay' instead > i will update. > Konrad > >> + * it triggers host tx_idle_timeout, which sets host TX state >> + * to sleep. Reset tx_idle_timer after SSR to prevent >> + * host enter TX IBS_Sloeep mode. >> + */ >> + mod_timer(&qca->tx_idle_timer, jiffies + >> + msecs_to_jiffies(qca->tx_idle_delay)); >> + >> qca->tx_ibs_state = HCI_IBS_TX_AWAKE; >> qca->memdump_state = QCA_MEMDUMP_IDLE; >> msleep(50); BR, Shuai ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v4 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw @ 2025-08-14 12:47 Shuai Zhang 2025-08-14 13:28 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot 0 siblings, 1 reply; 16+ messages in thread From: Shuai Zhang @ 2025-08-14 12:47 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang When the host actively triggers SSR and collects coredump data, the Bluetooth stack sends a reset command to the controller. However,due to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, the reset command times out. For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to commit: 740011cfe94859df8d05f5400d589a8693b095e7. The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) because this quirk is associated with BT_EN, and can be used to determine whether BT_EN is present in the device tree (DTS). Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 4e56782b0..91009c6a7 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1653,6 +1653,20 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) skb_queue_purge(&qca->rx_memdump_q); } + /* + * If the BT chip's bt_en pin is connected to a 3.3V power supply via + * hardware and always stays high, driver cannot control the bt_en pin. + * As a result, during SSR(SubSystem Restart), QCA_SSR_TRIGGERED and + * QCA_IBS_DISABLED flags cannot be cleared, which leads to a reset + * command timeout. + * Add an msleep delay to ensure controller completes the SSR process. + */ + if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { + clear_bit(QCA_SSR_TRIGGERED, &qca->flags); + clear_bit(QCA_IBS_DISABLED, &qca->flags); + msleep(50); + } + clear_bit(QCA_HW_ERROR_EVENT, &qca->flags); } -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* RE: Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware 2025-08-14 12:47 [PATCH v4 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang @ 2025-08-14 13:28 ` bluez.test.bot 0 siblings, 0 replies; 16+ messages in thread From: bluez.test.bot @ 2025-08-14 13:28 UTC (permalink / raw) To: linux-bluetooth, quic_shuaz [-- Attachment #1: Type: text/plain, Size: 25092 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=991506 ---Test result--- Test Summary: CheckPatch PENDING 0.42 seconds GitLint PENDING 0.37 seconds SubjectPrefix FAIL 0.66 seconds BuildKernel FAIL 19.95 seconds CheckAllWarning FAIL 22.17 seconds CheckSparse FAIL 24.96 seconds BuildKernel32 FAIL 20.40 seconds TestRunnerSetup FAIL 463.48 seconds TestRunner_l2cap-tester FAIL 0.08 seconds TestRunner_iso-tester FAIL 0.08 seconds TestRunner_bnep-tester FAIL 0.08 seconds TestRunner_mgmt-tester FAIL 0.08 seconds TestRunner_rfcomm-tester FAIL 0.08 seconds TestRunner_sco-tester FAIL 0.08 seconds TestRunner_ioctl-tester FAIL 0.08 seconds TestRunner_mesh-tester FAIL 0.08 seconds TestRunner_smp-tester FAIL 0.08 seconds TestRunner_userchan-tester FAIL 0.08 seconds IncrementalBuild PENDING 0.80 seconds Details ############################## Test: CheckPatch - PENDING Desc: Run checkpatch.pl script Output: ############################## Test: GitLint - PENDING Desc: Run gitlint Output: ############################## Test: SubjectPrefix - FAIL Desc: Check subject contains "Bluetooth" prefix Output: "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject ############################## Test: BuildKernel - FAIL Desc: Build Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckAllWarning - FAIL Desc: Run linux kernel with all warning enabled Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckSparse - FAIL Desc: Run sparse tool with linux kernel Output: net/bluetooth/af_bluetooth.c:248:25: warning: context imbalance in 'bt_accept_enqueue' - different lock contexts for basic block net/bluetooth/hci_core.c:85:9: warning: context imbalance in '__hci_dev_get' - different lock contexts for basic block net/bluetooth/hci_core.c: note: in included file (through include/linux/notifier.h, include/linux/memory_hotplug.h, include/linux/mmzone.h, include/linux/gfp.h, include/linux/xarray.h, include/linux/radix-tree.h, ...): ./include/linux/srcu.h:400:9: warning: context imbalance in 'hci_dev_put_srcu' - unexpected unlock net/bluetooth/hci_event.c: note: in included file (through include/net/bluetooth/hci_core.h): ./include/net/bluetooth/hci.h:2657:47: warning: array of flexible structures ./include/net/bluetooth/hci.h:2743:43: warning: array of flexible structures In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... drivers/bluetooth/hci_ag6xx.c:257:24: warning: restricted __le32 degrades to integer make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... net/bluetooth/hci_codec.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures net/bluetooth/sco.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: BuildKernel32 - FAIL Desc: Build 32bit Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunnerSetup - FAIL Desc: Setup kernel and bluez for test-runner Output: Kernel: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1669:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1669:7: note: in expansion of macro ‘test_bit’ 1669 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[3]: *** Waiting for unfinished jobs.... make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunner_l2cap-tester - FAIL Desc: Run l2cap-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_iso-tester - FAIL Desc: Run iso-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_bnep-tester - FAIL Desc: Run bnep-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mgmt-tester - FAIL Desc: Run mgmt-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_rfcomm-tester - FAIL Desc: Run rfcomm-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_sco-tester - FAIL Desc: Run sco-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_ioctl-tester - FAIL Desc: Run ioctl-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mesh-tester - FAIL Desc: Run mesh-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_smp-tester - FAIL Desc: Run smp-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_userchan-tester - FAIL Desc: Run userchan-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: IncrementalBuild - PENDING Desc: Incremental build with the patches in the series Output: --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw @ 2025-08-12 12:29 Shuai Zhang 2025-08-12 13:26 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot 0 siblings, 1 reply; 16+ messages in thread From: Shuai Zhang @ 2025-08-12 12:29 UTC (permalink / raw) To: linux-bluetooth, linux-arm-msm; +Cc: quic_bt, Shuai Zhang When the host actively triggers SSR and collects coredump data, the Bluetooth stack sends a reset command to the controller. However,due to the inability to clear the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits, the reset command times out. For the purpose of HCI_QUIRK_NON_PERSISTENT_SETUP, please refer to commit: 740011cfe94859df8d05f5400d589a8693b095e7. The change is placed under if (!HCI_QUIRK_NON_PERSISTENT_SETUP) because this quirk is associated with BT_EN, and can be used to determine whether BT_EN is present in the device tree (DTS). Signed-off-by: Shuai Zhang <quic_shuaz@quicinc.com> --- drivers/bluetooth/hci_qca.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index 4e56782b0..14b2d1bee 100644 --- a/drivers/bluetooth/hci_qca.c +++ b/drivers/bluetooth/hci_qca.c @@ -1653,6 +1653,19 @@ static void qca_hw_error(struct hci_dev *hdev, u8 code) skb_queue_purge(&qca->rx_memdump_q); } + /* If the BT chip's bt_en pin is always pulled high by a dedicated 3.3V + * power supply via hardware the driver + * cannot control the bt_en pin of the SoC chip, then during SSR, + * the QCA_SSR_TRIGGERED and QCA_IBS_DISABLED bits cannot be cleared. + * This leads to a reset command timeout failure. + * Also, add msleep delay to wait for controller to complete SSR. + */ + if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { + clear_bit(QCA_SSR_TRIGGERED, &qca->flags); + clear_bit(QCA_IBS_DISABLED, &qca->flags); + msleep(50); + } + clear_bit(QCA_HW_ERROR_EVENT, &qca->flags); } -- 2.34.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* RE: Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware 2025-08-12 12:29 [PATCH 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang @ 2025-08-12 13:26 ` bluez.test.bot 0 siblings, 0 replies; 16+ messages in thread From: bluez.test.bot @ 2025-08-12 13:26 UTC (permalink / raw) To: linux-bluetooth, quic_shuaz [-- Attachment #1: Type: text/plain, Size: 25047 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=990507 ---Test result--- Test Summary: CheckPatch PENDING 0.41 seconds GitLint PENDING 0.25 seconds SubjectPrefix FAIL 0.69 seconds BuildKernel FAIL 20.27 seconds CheckAllWarning FAIL 23.03 seconds CheckSparse FAIL 25.30 seconds BuildKernel32 FAIL 20.09 seconds TestRunnerSetup FAIL 465.79 seconds TestRunner_l2cap-tester FAIL 0.11 seconds TestRunner_iso-tester FAIL 0.11 seconds TestRunner_bnep-tester FAIL 0.16 seconds TestRunner_mgmt-tester FAIL 0.11 seconds TestRunner_rfcomm-tester FAIL 0.11 seconds TestRunner_sco-tester FAIL 0.11 seconds TestRunner_ioctl-tester FAIL 0.11 seconds TestRunner_mesh-tester FAIL 0.12 seconds TestRunner_smp-tester FAIL 0.11 seconds TestRunner_userchan-tester FAIL 0.40 seconds IncrementalBuild PENDING 0.47 seconds Details ############################## Test: CheckPatch - PENDING Desc: Run checkpatch.pl script Output: ############################## Test: GitLint - PENDING Desc: Run gitlint Output: ############################## Test: SubjectPrefix - FAIL Desc: Check subject contains "Bluetooth" prefix Output: "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject "Bluetooth: " prefix is not specified in the subject ############################## Test: BuildKernel - FAIL Desc: Build Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckAllWarning - FAIL Desc: Run linux kernel with all warning enabled Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: CheckSparse - FAIL Desc: Run sparse tool with linux kernel Output: net/bluetooth/af_bluetooth.c:248:25: warning: context imbalance in 'bt_accept_enqueue' - different lock contexts for basic block net/bluetooth/hci_core.c:85:9: warning: context imbalance in '__hci_dev_get' - different lock contexts for basic block net/bluetooth/hci_core.c: note: in included file (through include/linux/notifier.h, include/linux/memory_hotplug.h, include/linux/mmzone.h, include/linux/gfp.h, include/linux/xarray.h, include/linux/radix-tree.h, ...): ./include/linux/srcu.h:400:9: warning: context imbalance in 'hci_dev_put_srcu' - unexpected unlock net/bluetooth/hci_event.c: note: in included file (through include/net/bluetooth/hci_core.h): ./include/net/bluetooth/hci.h:2657:47: warning: array of flexible structures ./include/net/bluetooth/hci.h:2743:43: warning: array of flexible structures In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... drivers/bluetooth/hci_ag6xx.c:257:24: warning: restricted __le32 degrades to integer make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... net/bluetooth/hci_codec.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures net/bluetooth/sco.c: note: in included file: ./include/net/bluetooth/hci_core.h:151:35: warning: array of flexible structures make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: BuildKernel32 - FAIL Desc: Build 32bit Kernel for Bluetooth Output: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[4]: *** Waiting for unfinished jobs.... make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[2]: *** Waiting for unfinished jobs.... make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunnerSetup - FAIL Desc: Setup kernel and bluez for test-runner Output: Kernel: In file included from ./include/linux/kernel.h:23, from drivers/bluetooth/hci_qca.c:18: drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’: drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:44:37: note: in definition of macro ‘bitop’ 44 | __builtin_constant_p((uintptr_t)(addr) != (uintptr_t)NULL) && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:45:16: note: in definition of macro ‘bitop’ 45 | (uintptr_t)(addr) != (uintptr_t)NULL && \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:46:50: note: in definition of macro ‘bitop’ 46 | __builtin_constant_p(*(const unsigned long *)(addr))) ? \ | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:17: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ drivers/bluetooth/hci_qca.c:1668:53: error: ‘struct hci_dev’ has no member named ‘quirks’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~ ./include/linux/bitops.h:47:32: note: in definition of macro ‘bitop’ 47 | const##op(nr, addr) : op(nr, addr)) | ^~~~ drivers/bluetooth/hci_qca.c:1668:7: note: in expansion of macro ‘test_bit’ 1668 | if (!test_bit(HCI_QUIRK_NON_PERSISTENT_SETUP, &hdev->quirks)) { | ^~~~~~~~ make[4]: *** [scripts/Makefile.build:287: drivers/bluetooth/hci_qca.o] Error 1 make[3]: *** [scripts/Makefile.build:554: drivers/bluetooth] Error 2 make[3]: *** Waiting for unfinished jobs.... make[2]: *** [scripts/Makefile.build:554: drivers] Error 2 make[1]: *** [/github/workspace/src/src/Makefile:2003: .] Error 2 make: *** [Makefile:248: __sub-make] Error 2 ############################## Test: TestRunner_l2cap-tester - FAIL Desc: Run l2cap-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_iso-tester - FAIL Desc: Run iso-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_bnep-tester - FAIL Desc: Run bnep-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mgmt-tester - FAIL Desc: Run mgmt-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_rfcomm-tester - FAIL Desc: Run rfcomm-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_sco-tester - FAIL Desc: Run sco-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_ioctl-tester - FAIL Desc: Run ioctl-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_mesh-tester - FAIL Desc: Run mesh-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_smp-tester - FAIL Desc: Run smp-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: TestRunner_userchan-tester - FAIL Desc: Run userchan-tester with test-runner Output: Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator qemu: could not open kernel file '/github/workspace/src/src/arch/x86/boot/bzImage': No such file or directory ############################## Test: IncrementalBuild - PENDING Desc: Incremental build with the patches in the series Output: --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2025-08-14 13:28 UTC | newest] Thread overview: 16+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-08-13 3:35 [PATCH v3 0/4] Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang 2025-08-13 4:18 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot 2025-08-13 11:05 ` [PATCH v3 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Konrad Dybcio 2025-08-13 11:28 ` Shuai Zhang 2025-08-13 11:32 ` Konrad Dybcio 2025-08-14 11:41 ` Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 2/4] driver: bluetooth: hci_qca: fix host IBS state after SSR Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 3/4] driver: bluetooth: hci_qca: Multiple triggers of SSR only generate one coredump file Shuai Zhang 2025-08-13 11:11 ` Konrad Dybcio 2025-08-13 11:30 ` Shuai Zhang 2025-08-13 3:35 ` [PATCH v3 4/4] driver: bluetooth: hci_qca: SSR(SubSystem Restart)process failed due to tx_idle_timer timeout Shuai Zhang 2025-08-13 11:13 ` Konrad Dybcio 2025-08-14 11:52 ` Shuai Zhang -- strict thread matches above, loose matches on Subject: below -- 2025-08-14 12:47 [PATCH v4 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang 2025-08-14 13:28 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot 2025-08-12 12:29 [PATCH 1/4] driver: bluetooth: hci_qca: fix ssr fail when BT_EN is pulled up by hw Shuai Zhang 2025-08-12 13:26 ` Fix SSR(SubSystem Restart) issues caused by BT_EN being pulled up by hardware bluez.test.bot
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox