bthost: Allow sending ISO packets with sequence number and timestamp
authorPauli Virtanen <pav@iki.fi>
Sat, 25 Feb 2023 21:42:50 +0000 (21:42 +0000)
committerAyush Garg <ayush.garg@samsung.com>
Fri, 5 Jan 2024 10:21:48 +0000 (15:51 +0530)
Change bthost_send_iso to take packet sequence number and timestamp, and
allow it to send timestamped HCI ISO data packets.

Currently, btdev passes through ISO packets, so this can also be used to
test RX timestamping.

emulator/bthost.c
emulator/bthost.h
tools/iso-tester.c

index 28102c7..cd0b33e 100755 (executable)
@@ -37,6 +37,8 @@
 #define acl_flags(h)           (h >> 12)
 
 #define iso_flags_pb(f)                (f & 0x0003)
+#define iso_flags_ts(f)                ((f >> 2) & 0x0001)
+#define iso_flags_pack(pb, ts) (((pb) & 0x03) | (((ts) & 0x01) << 2))
 #define iso_data_len_pack(h, f)        ((uint16_t) ((h) | ((f) << 14)))
 
 #define L2CAP_FEAT_FIXED_CHAN  0x00000080
@@ -727,41 +729,58 @@ void bthost_send_cid_v(struct bthost *bthost, uint16_t handle, uint16_t cid,
        send_iov(bthost, handle, cid, iov, iovcnt);
 }
 
-static void send_iso(struct bthost *bthost, uint16_t handle,
+static void send_iso(struct bthost *bthost, uint16_t handle, bool ts,
+                                       uint16_t sn, uint32_t timestamp,
                                        const struct iovec *iov, int iovcnt)
 {
        struct bt_hci_iso_hdr iso_hdr;
        struct bt_hci_iso_data_start data_hdr;
        uint8_t pkt = BT_H4_ISO_PKT;
-       struct iovec pdu[3 + iovcnt];
+       struct iovec pdu[4 + iovcnt];
+       uint16_t flags, dlen;
        int i, len = 0;
-       static uint16_t sn;
 
        for (i = 0; i < iovcnt; i++) {
-               pdu[3 + i].iov_base = iov[i].iov_base;
-               pdu[3 + i].iov_len = iov[i].iov_len;
+               pdu[4 + i].iov_base = iov[i].iov_base;
+               pdu[4 + i].iov_len = iov[i].iov_len;
                len += iov[i].iov_len;
        }
 
        pdu[0].iov_base = &pkt;
        pdu[0].iov_len = sizeof(pkt);
 
-       iso_hdr.handle = acl_handle_pack(handle, 0x02);
-       iso_hdr.dlen = cpu_to_le16(len + sizeof(data_hdr));
+       flags = iso_flags_pack(0x02, ts);
+       dlen = len + sizeof(data_hdr);
+       if (ts)
+               dlen += sizeof(timestamp);
+
+       iso_hdr.handle = acl_handle_pack(handle, flags);
+       iso_hdr.dlen = cpu_to_le16(dlen);
 
        pdu[1].iov_base = &iso_hdr;
        pdu[1].iov_len = sizeof(iso_hdr);
 
-       data_hdr.sn = cpu_to_le16(sn++);
+       if (ts) {
+               timestamp = cpu_to_le32(timestamp);
+
+               pdu[2].iov_base = &timestamp;
+               pdu[2].iov_len = sizeof(timestamp);
+       } else {
+               pdu[2].iov_base = NULL;
+               pdu[2].iov_len = 0;
+       }
+
+       data_hdr.sn = cpu_to_le16(sn);
        data_hdr.slen = cpu_to_le16(iso_data_len_pack(len, 0));
 
-       pdu[2].iov_base = &data_hdr;
-       pdu[2].iov_len = sizeof(data_hdr);
+       pdu[3].iov_base = &data_hdr;
+       pdu[3].iov_len = sizeof(data_hdr);
 
-       send_packet(bthost, pdu, 3 + iovcnt);
+       send_packet(bthost, pdu, 4 + iovcnt);
 }
 
-void bthost_send_iso(struct bthost *bthost, uint16_t handle,
+void bthost_send_iso(struct bthost *bthost, uint16_t handle, bool ts,
+                                       uint16_t sn, uint32_t timestamp,
                                        const struct iovec *iov, int iovcnt)
 {
        struct btconn *conn;
@@ -770,7 +789,7 @@ void bthost_send_iso(struct bthost *bthost, uint16_t handle,
        if (!conn)
                return;
 
-       send_iso(bthost, handle, iov, iovcnt);
+       send_iso(bthost, handle, ts, sn, timestamp, iov, iovcnt);
 }
 
 bool bthost_l2cap_req(struct bthost *bthost, uint16_t handle, uint8_t code,
index c424444..9218268 100755 (executable)
@@ -79,7 +79,8 @@ void bthost_send_cid(struct bthost *bthost, uint16_t handle, uint16_t cid,
                                        const void *data, uint16_t len);
 void bthost_send_cid_v(struct bthost *bthost, uint16_t handle, uint16_t cid,
                                        const struct iovec *iov, int iovcnt);
-void bthost_send_iso(struct bthost *bthost, uint16_t handle,
+void bthost_send_iso(struct bthost *bthost, uint16_t handle, bool ts,
+                                       uint16_t sn, uint32_t timestamp,
                                        const struct iovec *iov, int iovcnt);
 
 typedef void (*bthost_l2cap_rsp_cb) (uint8_t code, const void *data,
index d790b15..dcfd6a0 100644 (file)
@@ -1146,6 +1146,7 @@ static void iso_recv(struct test_data *data, GIOChannel *io)
 {
        const struct iso_client_data *isodata = data->test_data;
        struct bthost *host;
+       static uint16_t sn;
 
        tester_print("Receive %zu bytes of data", isodata->recv->iov_len);
 
@@ -1156,7 +1157,7 @@ static void iso_recv(struct test_data *data, GIOChannel *io)
        }
 
        host = hciemu_client_get_host(data->hciemu);
-       bthost_send_iso(host, data->handle, isodata->recv, 1);
+       bthost_send_iso(host, data->handle, false, sn++, 0, isodata->recv, 1);
 
        data->io_id[0] = g_io_add_watch(io, G_IO_IN, iso_recv_data, data);
 }