From c60d08b37068289303168cccfe79789348e44bc4 Mon Sep 17 00:00:00 2001 From: Claudio Takahasi Date: Tue, 2 Oct 2012 14:22:56 -0300 Subject: [PATCH] core: Re-connect for ECONNRESET or ECONNABORTED This patch keeps scanning and re-connections active if the disconnection reason is ECONNRESET(Remote Initiated Disconnection). Re-connection is a behaviour determined by Profiles or by the upper layer(user actions). For instance, HoG requires re-connection always active, no matter if the previous disconnection reason was page timeout or remote initiated disconnection (ECONNRESET). Some devices disconnects after some idle time, connectable advertises are sent by the peripheral when commanded by the user(eg: key pressed). Disconnection can be also triggered by the local host (ECONNABORTED) using command line tools or Disconnect method in the Device interface. The peripheral dictates the re-connection controlling the connectable advertises, BlueZ(central) needs to keep the scanning always active to able to detect the advertises and trigger the connection. --- src/device.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/device.c b/src/device.c index d28adeb52..89ab24a61 100644 --- a/src/device.c +++ b/src/device.c @@ -1944,10 +1944,18 @@ static gboolean attrib_disconnected_cb(GIOChannel *io, GIOCondition cond, g_slist_foreach(device->attios, attio_disconnected, NULL); - if (device->auto_connect == FALSE || err != ETIMEDOUT) + if (device->auto_connect == FALSE) { + DBG("Automatic connection disabled"); goto done; + } - adapter_connect_list_add(device_get_adapter(device), device); + /* + * Keep scanning/re-connection active if disconnection reason + * is connection timeout, remote user terminated connection or local + * initiated disconnection. + */ + if (err == ETIMEDOUT || err == ECONNRESET || err == ECONNABORTED) + adapter_connect_list_add(device_get_adapter(device), device); done: attio_cleanup(device); -- 2.47.3