* RE: [v1] Bluetooth: hci_event: Add HCI_Write_Link_Supervision_Timeout command/event structures
[not found] <20260723095616.1543159-1-zhangchen200426@163.com>
@ 2026-07-23 15:40 ` bluez.test.bot
2026-07-23 16:56 ` [PATCH v1] " Luiz Augusto von Dentz
1 sibling, 0 replies; 2+ messages in thread
From: bluez.test.bot @ 2026-07-23 15:40 UTC (permalink / raw)
To: linux-bluetooth, zhangchen200426
[-- Attachment #1: Type: text/plain, Size: 2692 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=1133206
---Test result---
Test Summary:
CheckPatch PASS 0.84 seconds
VerifyFixes PASS 0.11 seconds
VerifySignedoff PASS 0.11 seconds
GitLint FAIL 1.20 seconds
SubjectPrefix PASS 0.10 seconds
BuildKernel PASS 19.80 seconds
CheckAllWarning PASS 22.19 seconds
CheckSparse PASS 21.50 seconds
BuildKernel32 PASS 19.58 seconds
CheckKernelLLVM SKIP 0.00 seconds
TestRunnerSetup PASS 326.53 seconds
TestRunner_l2cap-tester PASS 50.67 seconds
TestRunner_iso-tester PASS 72.05 seconds
TestRunner_bnep-tester PASS 14.13 seconds
TestRunner_mgmt-tester FAIL 191.83 seconds
TestRunner_rfcomm-tester PASS 19.47 seconds
TestRunner_sco-tester PASS 25.38 seconds
TestRunner_ioctl-tester PASS 20.91 seconds
TestRunner_mesh-tester FAIL 20.66 seconds
TestRunner_smp-tester PASS 17.85 seconds
TestRunner_userchan-tester PASS 15.27 seconds
TestRunner_6lowpan-tester PASS 17.90 seconds
IncrementalBuild PASS 19.83 seconds
Details
##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
[v1] Bluetooth: hci_event: Add HCI_Write_Link_Supervision_Timeout command/event structures
1: T1 Title exceeds max length (90>80): "[v1] Bluetooth: hci_event: Add HCI_Write_Link_Supervision_Timeout command/event structures"
##############################
Test: CheckKernelLLVM - SKIP
Desc: Build kernel with LLVM + context analysis
Output:
Clang not found
##############################
Test: TestRunner_mgmt-tester - FAIL
Desc: Run mgmt-tester with test-runner
Output:
Total: 494, Passed: 489 (99.0%), Failed: 1, Not Run: 4
Failed Test Cases
Read Exp Feature - Success Failed 0.211 seconds
##############################
Test: TestRunner_mesh-tester - FAIL
Desc: Run mesh-tester with test-runner
Output:
Total: 10, Passed: 8 (80.0%), Failed: 2, Not Run: 0
Failed Test Cases
Mesh - Send cancel - 1 Timed out 2.141 seconds
Mesh - Send cancel - 2 Timed out 1.988 seconds
https://github.com/bluez/bluetooth-next/pull/481
---
Regards,
Linux Bluetooth
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH v1] Bluetooth: hci_event: Add HCI_Write_Link_Supervision_Timeout command/event structures
[not found] <20260723095616.1543159-1-zhangchen200426@163.com>
2026-07-23 15:40 ` [v1] Bluetooth: hci_event: Add HCI_Write_Link_Supervision_Timeout command/event structures bluez.test.bot
@ 2026-07-23 16:56 ` Luiz Augusto von Dentz
1 sibling, 0 replies; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2026-07-23 16:56 UTC (permalink / raw)
To: zhangchen200426
Cc: luiz.von.dentz, yang.li, tim.bird, naga.akella, gustavoars, pav,
zhangchen01, linux-bluetooth
Hi,
On Thu, Jul 23, 2026 at 6:05 AM <zhangchen200426@163.com> wrote:
>
> From: Chen Zhang <zhangchen01@kylinos.cn>
>
> Define HCI_Write_Link_Supervision_Timeout command, event structure, and
> corresponding event handler function.
>
> Signed-off-by: Chen Zhang <zhangchen01@kylinos.cn>
> ---
> include/net/bluetooth/hci.h | 10 ++++++++
> include/net/bluetooth/hci_core.h | 1 +
> net/bluetooth/hci_event.c | 40 ++++++++++++++++++++++++++++++++
> 3 files changed, 51 insertions(+)
>
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 50f0eef71fb1..18a5a17eb218 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -1239,6 +1239,16 @@ struct hci_cp_host_buffer_size {
> __le16 sco_max_pkt;
> } __packed;
>
> +#define HCI_OP_WRITE_LINK_SUPERVISION_TIMEOUT 0x0c37
> +struct hci_cp_write_link_supervision_timeout {
> + __le16 handle;
> + __le16 timeout;
> +} __packed;
> +struct hci_rp_write_link_supervision_timeout {
> + __u8 status;
> + __le16 handle;
> +} __packed;
> +
> #define HCI_OP_READ_NUM_SUPPORTED_IAC 0x0c38
> struct hci_rp_read_num_supported_iac {
> __u8 status;
> diff --git a/include/net/bluetooth/hci_core.h b/include/net/bluetooth/hci_core.h
> index e7133ff87fbf..866903a9b5c8 100644
> --- a/include/net/bluetooth/hci_core.h
> +++ b/include/net/bluetooth/hci_core.h
> @@ -702,6 +702,7 @@ struct hci_conn {
> __u8 le_features[248];
> __u16 pkt_type;
> __u16 link_policy;
> + __u16 link_supervision_timeout;
> __u8 key_type;
> __u8 auth_type;
> __u8 sec_level;
> diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
> index 741d658e9630..3b1f4cf922c3 100644
> --- a/net/bluetooth/hci_event.c
> +++ b/net/bluetooth/hci_event.c
> @@ -606,6 +606,44 @@ static u8 hci_cc_write_voice_setting(struct hci_dev *hdev, void *data,
> return rp->status;
> }
>
> +static u8 hci_cc_write_link_supervision_timeout(struct hci_dev *hdev, void *data,
> + struct sk_buff *skb)
> +{
> + struct hci_ev_status *rp = data;
> + struct hci_cp_write_link_supervision_timeout *sent;
> + struct hci_conn *conn;
> + u16 handle;
> + u8 status = rp->status;
> +
> + bt_dev_dbg(hdev, "status 0x%2.2x", status);
> +
> + sent = hci_sent_cmd_data(hdev, HCI_OP_WRITE_LINK_SUPERVISION_TIMEOUT);
> + if (!sent)
> + return status;
> +
> + handle = le16_to_cpu(sent->handle);
> +
> + hci_dev_lock(hdev);
> +
> + conn = hci_conn_hash_lookup_handle(hdev, handle);
> + if (!conn) {
> + status = 0xFF;
> + goto done;
> + }
> +
> + if (!status) {
> + conn->link_supervision_timeout = __le16_to_cpu(sent->timeout);
> + bt_dev_dbg(hdev, "handle 0x%4.4x timeout set to 0x%4.4x (%u ms)",
> + handle, conn->link_supervision_timeout,
> + conn->link_supervision_timeout * 5 / 8);
> + }
> +
> +done:
> + hci_dev_unlock(hdev);
> +
> + return status;
> +}
> +
> static u8 hci_cc_read_num_supported_iac(struct hci_dev *hdev, void *data,
> struct sk_buff *skb)
> {
> @@ -4097,6 +4135,8 @@ static const struct hci_cc {
> HCI_CC_STATUS(HCI_OP_WRITE_CLASS_OF_DEV, hci_cc_write_class_of_dev),
> HCI_CC(HCI_OP_READ_VOICE_SETTING, hci_cc_read_voice_setting,
> sizeof(struct hci_rp_read_voice_setting)),
> + HCI_CC(HCI_OP_WRITE_LINK_SUPERVISION_TIMEOUT, hci_cc_write_link_supervision_timeout,
> + sizeof(struct hci_rp_write_link_supervision_timeout)),
Ok, but I don't see use sending the
HCI_OP_WRITE_LINK_SUPERVISION_TIMEOUT command, so what is this for,
handling some custom entity that does perform this sort of operation
behind bluetoothd back is not an acceptable excuse.
> HCI_CC_STATUS(HCI_OP_WRITE_VOICE_SETTING, hci_cc_write_voice_setting),
> HCI_CC(HCI_OP_READ_NUM_SUPPORTED_IAC, hci_cc_read_num_supported_iac,
> sizeof(struct hci_rp_read_num_supported_iac)),
> --
> 2.25.1
>
>
--
Luiz Augusto von Dentz
^ permalink raw reply [flat|nested] 2+ messages in thread