Return-Path: From: =?UTF-8?q?Jo=C3=A3o=20Paulo=20Rechi=20Vita?= To: linux-bluetooth@vger.kernel.org Cc: Claudio Takahasi Subject: [PATCH BlueZ v5 10/14] core: Queue discovery if scanning is active Date: Tue, 4 Sep 2012 16:04:38 -0300 Message-Id: <1346785482-13359-11-git-send-email-jprvita@openbossa.org> In-Reply-To: <1346785482-13359-1-git-send-email-jprvita@openbossa.org> References: <1346785482-13359-1-git-send-email-jprvita@openbossa.org> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: From: Claudio Takahasi This patch manages BR/EDR inquiry and BLE scanning discovery sessions. A scanning session is added in the discovery session list when there is a bonded device which requires re-connection. bluetoothd decides if interleaved or scanning needs to be executed based on the queued discovery sessions. Interleaved discovery has higher priority, scanning only is executed when there is only a scanning session active. --- src/adapter.c | 67 +++++++++++++++++++++++++++++++++++++++++------------------ 1 file changed, 47 insertions(+), 20 deletions(-) diff --git a/src/adapter.c b/src/adapter.c index 70fbc02..8df97eb 100644 --- a/src/adapter.c +++ b/src/adapter.c @@ -133,6 +133,7 @@ struct btd_adapter { GSList *devices; /* Devices structure pointers */ GSList *mode_sessions; /* Request Mode sessions */ GSList *disc_sessions; /* Discovery sessions */ + struct session_req *scanning_session; GSList *connect_list; /* Devices to connect when found */ guint discov_id; /* Discovery timer */ gboolean discovering; /* Discovery active */ @@ -222,18 +223,19 @@ static struct session_req *create_session(struct btd_adapter *adapter, DBusConnection *conn, DBusMessage *msg, uint8_t mode, GDBusWatchFunction cb) { - const char *sender = dbus_message_get_sender(msg); + const char *sender; struct session_req *req; req = g_new0(struct session_req, 1); req->adapter = adapter; - req->conn = dbus_connection_ref(conn); - req->msg = dbus_message_ref(msg); req->mode = mode; - if (cb == NULL) + if (conn == NULL || cb == NULL || msg == NULL) return session_ref(req); + req->conn = dbus_connection_ref(conn); + req->msg = dbus_message_ref(msg); + sender = dbus_message_get_sender(msg); req->owner = g_strdup(sender); req->id = g_dbus_add_disconnect_watch(conn, sender, cb, req, NULL); @@ -445,7 +447,9 @@ static struct session_req *find_session(GSList *list, const char *sender) for (; list; list = list->next) { struct session_req *req = list->data; - if (g_str_equal(req->owner, sender)) + /* req->owner may be NULL if the session has been added by the + * daemon itself, so we use g_strcmp0 instead of g_str_equal */ + if (g_strcmp0(req->owner, sender) == 0) return req; } @@ -520,7 +524,7 @@ static void session_remove(struct session_req *req) struct btd_adapter *adapter = req->adapter; /* Ignore set_mode session */ - if (req->owner == NULL) + if (req->owner == NULL && adapter->pending_mode) return; DBG("%s session %p with %s deactivated", @@ -1009,7 +1013,12 @@ static gboolean discovery_cb(gpointer user_data) struct btd_adapter *adapter = user_data; adapter->discov_id = 0; - mgmt_start_discovery(adapter->dev_id); + + if (adapter->scanning_session && + (g_slist_length(adapter->disc_sessions) == 1)) + mgmt_start_scanning(adapter->dev_id); + else + mgmt_start_discovery(adapter->dev_id); return FALSE; } @@ -2205,6 +2214,8 @@ const char *btd_adapter_get_name(struct btd_adapter *adapter) void adapter_connect_list_add(struct btd_adapter *adapter, struct btd_device *device) { + struct session_req *req; + if (g_slist_find(adapter->connect_list, device)) { DBG("ignoring already added device %s", device_get_path(device)); @@ -2216,10 +2227,21 @@ void adapter_connect_list_add(struct btd_adapter *adapter, DBG("%s added to %s's connect_list", device_get_path(device), adapter->name); - if (adapter->disc_sessions) + if (!adapter->up) + return; + + if (adapter->off_requested) + return; + + if (adapter->scanning_session) return; - mgmt_start_scanning(adapter->dev_id); + if (adapter->disc_sessions == NULL) + adapter->discov_id = g_idle_add(discovery_cb, adapter); + + req = create_session(adapter, NULL, NULL, 0, NULL); + adapter->disc_sessions = g_slist_append(adapter->disc_sessions, req); + adapter->scanning_session = req; } void adapter_connect_list_remove(struct btd_adapter *adapter, @@ -2239,6 +2261,7 @@ void adapter_connect_list_remove(struct btd_adapter *adapter, void btd_adapter_start(struct btd_adapter *adapter) { + struct session_req *req; char address[18]; gboolean powered; @@ -2265,8 +2288,15 @@ void btd_adapter_start(struct btd_adapter *adapter) info("Adapter %s has been enabled", adapter->path); - if (g_slist_length(adapter->connect_list)) - mgmt_start_scanning(adapter->dev_id); + if (g_slist_length(adapter->connect_list) == 0 || + adapter->disc_sessions) + return; + + req = create_session(adapter, NULL, NULL, 0, NULL); + adapter->disc_sessions = g_slist_append(adapter->disc_sessions, req); + adapter->scanning_session = req; + + adapter->discov_id = g_idle_add(discovery_cb, adapter); } static void reply_pending_requests(struct btd_adapter *adapter) @@ -2610,6 +2640,11 @@ void adapter_set_discovering(struct btd_adapter *adapter, connect_list_size = g_slist_length(adapter->connect_list); + if (connect_list_size == 0 && adapter->scanning_session) { + session_unref(adapter->scanning_session); + adapter->scanning_session = NULL; + } + if (adapter_has_discov_sessions(adapter)) { adapter->discov_id = g_idle_add(discovery_cb, adapter); @@ -2618,14 +2653,6 @@ void adapter_set_discovering(struct btd_adapter *adapter, g_slist_length(adapter->disc_sessions)); return; } - - if (connect_list_size) { - mgmt_start_scanning(adapter->dev_id); - - DBG("hci%u restarting scanning connect_list size %u", - adapter->dev_id, connect_list_size); - return; - } } static void suspend_discovery(struct btd_adapter *adapter) @@ -2940,7 +2967,7 @@ static gboolean clean_connecting_state(GIOChannel *io, GIOCondition cond, gpoint if (adapter->waiting_to_connect == 0 && g_slist_length(adapter->connect_list)) - mgmt_start_scanning(adapter->dev_id); + adapter->discov_id = g_idle_add(discovery_cb, adapter); btd_device_unref(device); return FALSE; -- 1.7.11.4