1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
|
From 466fce0209a3878512672159168943047a9e2323 Mon Sep 17 00:00:00 2001
From: Pauli Virtanen <pav@iki.fi>
Date: Sun, 2 Jul 2023 22:34:18 +0300
Subject: transport: handle BAP Enabling state correctly when resuming
If BAP stream is in Enabling state when transport acquire is attempted,
we should wait for bap_state_changed to emit the completion event.
transport->resume() is only called with new owner with no
owner->pending, and shall return nonzero completion id on success.
Currently if BAP stream is Enabling, it returns zero which fails the
acquire operation.
To fix this, return valid completion id in this case instead. Also keep
track of the g_idle_add resume id, so that we don't try to give it to
bt_bap_stream_cancel.
Fixes sound server getting spurious Not Authorized errors when trying to
acquire a pending transport. This can happen on BAP server: linked
transports become pending when the first of the two enters Streaming. If
sound server tries to acquire the other linked transport whose stream is
still Enabling, the acquire fails (media_owner_free +
btd_error_not_authorized).
Log:
===============================================================
profiles/audio/transport.c:bap_state_changed() stream 0x25c2880: qos(2) -> enabling(3)
...
profiles/audio/transport.c:bap_state_changed() stream 0x25cc590: qos(2) -> enabling(3)
...
src/shared/bap.c:bap_stream_state_changed() stream 0x25cc590 dir 0x01: enabling -> streaming
profiles/audio/bap.c:bap_state() stream 0x25cc590: enabling(3) -> streaming(4)
profiles/audio/transport.c:bap_state_changed() stream 0x25cc590: enabling(3) -> streaming(4)
/org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/fd1: fd(36) ready
profiles/audio/transport.c:transport_update_playing() /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/fd1 State=TRANSPORT_STATE_IDLE Playing=1
profiles/audio/transport.c:transport_set_state() State changed /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/fd1: TRANSPORT_STATE_IDLE -> TRANSPORT_STATE_PENDING
profiles/audio/transport.c:transport_set_state() State changed /org/bluez/hci0/dev_XX_XX_XX_XX_XX_XX/fd0: TRANSPORT_STATE_IDLE -> TRANSPORT_STATE_PENDING
profiles/audio/transport.c:media_owner_create() Owner created: sender=:1.1242
profiles/audio/transport.c:media_owner_free() Owner :1.1242
===============================================================
---
profiles/audio/transport.c | 67 +++++++++++++++++++++++++++++++++++++---------
1 file changed, 55 insertions(+), 12 deletions(-)
diff --git a/profiles/audio/transport.c b/profiles/audio/transport.c
index 77216e10b3..aa3a718b0f 100644
--- a/profiles/audio/transport.c
+++ b/profiles/audio/transport.c
@@ -86,6 +86,7 @@ struct bap_transport {
unsigned int state_id;
bool linked;
struct bt_bap_qos qos;
+ guint resume_id;
};
struct media_transport {
@@ -1191,17 +1192,27 @@ static void bap_enable_complete(struct bt_bap_stream *stream,
media_transport_remove_owner(owner->transport);
}
-static gboolean resume_complete(void *data)
+static void bap_resume_complete(struct media_transport *transport)
{
- struct media_transport *transport = data;
+ struct bap_transport *bap = transport->data;
struct media_owner *owner = transport->owner;
+ DBG("stream %p owner %p resume complete", bap->stream, owner);
+
+ if (bap->resume_id) {
+ g_source_remove(bap->resume_id);
+ bap->resume_id = 0;
+ }
+
if (!owner)
- return FALSE;
+ return;
+
+ if (owner->pending)
+ owner->pending->id = 0;
if (transport->fd < 0) {
media_transport_remove_owner(transport);
- return FALSE;
+ return;
}
if (owner->pending) {
@@ -1215,15 +1226,13 @@ static gboolean resume_complete(void *data)
DBUS_TYPE_INVALID);
if (!ret) {
media_transport_remove_owner(transport);
- return FALSE;
+ return;
}
}
media_owner_remove(owner);
transport_set_state(transport, TRANSPORT_STATE_ACTIVE);
-
- return FALSE;
}
static void bap_update_links(const struct media_transport *transport);
@@ -1306,6 +1315,32 @@ static void bap_update_qos(const struct media_transport *transport)
"Delay");
}
+static gboolean bap_resume_complete_cb(void *data)
+{
+ struct media_transport *transport = data;
+ struct bap_transport *bap = transport->data;
+
+ bap->resume_id = 0;
+ bap_resume_complete(transport);
+ return FALSE;
+}
+
+static gboolean bap_resume_wait_cb(void *data)
+{
+ struct media_transport *transport = data;
+ struct bap_transport *bap = transport->data;
+ struct media_owner *owner = transport->owner;
+
+ /* bap_state_changed will call completion callback when ready */
+ DBG("stream %p owner %p resume wait", bap->stream, owner);
+
+ bap->resume_id = 0;
+ if (owner && owner->pending)
+ owner->pending->id = 0;
+
+ return FALSE;
+}
+
static guint resume_bap(struct media_transport *transport,
struct media_owner *owner)
{
@@ -1315,17 +1350,19 @@ static guint resume_bap(struct media_transport *transport,
if (!bap->stream)
return 0;
+ if (bap->resume_id)
+ return 0;
bap_update_links(transport);
switch (bt_bap_stream_get_state(bap->stream)) {
case BT_BAP_STREAM_STATE_ENABLING:
bap_enable_complete(bap->stream, 0x00, 0x00, owner);
- if (owner->pending)
- return owner->pending->id;
- return 0;
+ bap->resume_id = g_idle_add(bap_resume_wait_cb, transport);
+ return bap->resume_id;
case BT_BAP_STREAM_STATE_STREAMING:
- return g_idle_add(resume_complete, transport);
+ bap->resume_id = g_idle_add(bap_resume_complete_cb, transport);
+ return bap->resume_id;
}
meta = bt_bap_stream_get_metadata(bap->stream);
@@ -1389,6 +1426,12 @@ static void cancel_bap(struct media_transport *transport, guint id)
{
struct bap_transport *bap = transport->data;
+ if (id == bap->resume_id && bap->resume_id) {
+ g_source_remove(bap->resume_id);
+ bap->resume_id = 0;
+ return;
+ }
+
if (!bap->stream)
return;
@@ -1491,7 +1534,7 @@ static void bap_state_changed(struct bt_bap_stream *stream, uint8_t old_state,
transport_update_playing(transport, TRUE);
done:
- resume_complete(transport);
+ bap_resume_complete(transport);
}
static void bap_connecting(struct bt_bap_stream *stream, bool state, int fd,
--
cgit
|
