Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp1861448pxu; Sun, 13 Dec 2020 05:53:21 -0800 (PST) X-Google-Smtp-Source: ABdhPJynV0zyTcQTRh90KA2+QFcIZq19JkzFXdAhQt/2mZHij4u6wFOhZEw8/YW2bUvBCHXBEl1z X-Received: by 2002:a17:906:6546:: with SMTP id u6mr19274938ejn.36.1607867601282; Sun, 13 Dec 2020 05:53:21 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607867601; cv=none; d=google.com; s=arc-20160816; b=JZrz/KN2E6DzoE9tq8i/b7vFbMohusTAtJHZGBO8rJ16fKS/opR+nJ1QLxsTotUKHT S8/filFumkWFe5pf3zgSZo2p9X98YLPkBqPA9jktgDNXbRmDtdvlzRkM8R7KInpmXvv/ qDK+tHspzIdGXui1MmNELVZa3ghUf9zK5N8Cr4o4o9k63mFLaPUKn9nRvDIkMALShusl d9Tec+oV1MYRWa/78vE1zoEvvHo4wA3qGGQKzPgioXeHOJVHZ1e163/O9RZAfuV29E1h fYl2+uSxJiXCakffxGVf/vsXr8vMaF9KTh3gQndLyMqGRFM6GGSTbpXV0Is0KjoHtoi3 aXYg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=NJdNhGuQiOBVJEvxZvsdZw1J4E7rGVqBUpbQLbbMPtQ=; b=JXsdBqpNJKo5kF7Htvef99UVq+4yPcD7/gS7jomGiZaYsbrp1l31m9N4vE4ArGZvj6 /EznwnPkzkmUh3J68GsQdEffiTeQ4rH+aYacv0EkJVlStp40g0vG2UdoPx/Wh0VPUD/y k+fu/jpQaSm5/pTWsC4cXPIG3S1f8v/tTKThYNwo9NSEdBmiSs4hB5ukqc0lNYrDep3w LLdhX+b/yNxb2avqv1xSFTsF8DsC3GtX+4AUKi7g2fuIlvr66n+GT5UrXuWSbSHa6Yzw cTzTPezRLge3cBY2QZoSGZk3gqwZXYQ+HMEW59ir+7n3NrFDEpOGKBSi0XJctxXfB0Q+ c7zQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=QwSq5Gji; spf=pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-bluetooth-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z16si840582ejw.548.2020.12.13.05.52.57; Sun, 13 Dec 2020 05:53:21 -0800 (PST) Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=QwSq5Gji; spf=pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-bluetooth-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2394791AbgLLAGT (ORCPT + 99 others); Fri, 11 Dec 2020 19:06:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48860 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2407282AbgLLAFu (ORCPT ); Fri, 11 Dec 2020 19:05:50 -0500 Received: from mail-vs1-xe43.google.com (mail-vs1-xe43.google.com [IPv6:2607:f8b0:4864:20::e43]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BA6ECC0613D3 for ; Fri, 11 Dec 2020 16:05:09 -0800 (PST) Received: by mail-vs1-xe43.google.com with SMTP id h6so5742926vsr.6 for ; Fri, 11 Dec 2020 16:05:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=NJdNhGuQiOBVJEvxZvsdZw1J4E7rGVqBUpbQLbbMPtQ=; b=QwSq5GjimKvOP9C2vNkNarip+PEOYI6USDRs/MdpLy0xXfQd5rq5Y3ftCXhtmT/54F /j6WimQ/fiNWhEFe5tT6UYtdYvd+3Kn9au+4xIbVfWCAjDsp1SmZhXXNIBQ7ZJ0OapY6 no+zQkGhDro8Vc8sx3ApsZuK0usijyYgreMYY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=NJdNhGuQiOBVJEvxZvsdZw1J4E7rGVqBUpbQLbbMPtQ=; b=OF78W8qj3/QB+ToZVB2n+f9puWS4CL8xIq2kMtaY2vwMEtryGQ4sxWLZIM9/6J4VAa hyNt6/B10ejLNJKpEt6CK/2DmWYw7DornTW9hJAez/jxrokmlc2zCMsiTJM8LuYqrHw/ LuC+oFshgovnNnWkx7QVUcORgoCnZwyHc9H5zHG3r91moDDiJj8zKmS0Wuple0KjtY/O oHrCNujbri63/1CwsPQTn0UNjNRIR05YgHxQTkSRjq+PysqsSAJ3dtkvFQOz+7feWADj IJvOVUz5uJfYTHoPPYYSAgCcRXaBCUGjKCeaUR4UmcCYFVtb/LdPuwXgdpZcv8whBCio FClg== X-Gm-Message-State: AOAM531qXk/ONKYirjTqOTZGI5muCfXiATzOB94v+a6Rn03BUTfwf4ZD OXWduAD+ZDRtvPAkePMXixldZvppnLmMWklWPR1YEs62yg8= X-Received: by 2002:a67:fe92:: with SMTP id b18mr16618450vsr.30.1607731508798; Fri, 11 Dec 2020 16:05:08 -0800 (PST) MIME-Version: 1.0 References: <20201207234924.140606-1-abhishekpandit@chromium.org> <20201207154903.bluez.1.I3e043a481273442748bcff0728b2f0e208017cd2@changeid> In-Reply-To: From: Abhishek Pandit-Subedi Date: Fri, 11 Dec 2020 16:04:56 -0800 Message-ID: Subject: Re: [bluez PATCH 1/1] policy: Refactor reconnect policy for resume To: Luiz Augusto von Dentz Cc: "linux-bluetooth@vger.kernel.org" , ChromeOS Bluetooth Upstreaming , Alain Michaud Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org Hi Luiz, Alain is investigating a generic quirks implementation since we have more than one use-case for it. For now, I've merged this into ChromeOS to resolve the immediate problems with Airpods. Once we have a generic quirks library, we can apply it here so that Link Loss for this specific generation of Airpods is treated specially (not triggering reconnect). Abhishek On Mon, Dec 7, 2020 at 7:08 PM Luiz Augusto von Dentz wrote: > > Hi Abhishek, > > On Mon, Dec 7, 2020 at 5:19 PM Abhishek Pandit-Subedi > wrote: > > > > Hi Luiz, > > > > On Mon, Dec 7, 2020 at 4:36 PM Luiz Augusto von Dentz > > wrote: > > > > > > Hi Abhishek, > > > > > > On Mon, Dec 7, 2020 at 3:49 PM Abhishek Pandit-Subedi > > > wrote: > > > > > > > > Airpods incorrectly emit a connection timeout as the disconnect reason > > > > when a different device connects to it and triggers the automatic > > > > reconnection by mistake. See hci trace below: > > > > > > > > > HCI Event: Disconnect Complete (0x05) plen 4 > > > > Status: Success (0x00) > > > > Handle: 256 > > > > Reason: Connection Timeout (0x08) > > > > @ MGMT Event: Device Disconnected (0x000c) plen 8 > > > > BR/EDR Address: E4:90:FD:7D:E7:5F (OUI E4-90-FD) > > > > Reason: Connection timeout (0x01) > > > > @ MGMT Event: Device Disconnected (0x000c) plen 8 > > > > BR/EDR Address: E4:90:FD:7D:E7:5F (OUI E4-90-FD) > > > > Reason: Connection timeout (0x01) > > > > @ MGMT Event: Device Disconnected (0x000c) plen 8 > > > > BR/EDR Address: E4:90:FD:7D:E7:5F (OUI E4-90-FD) > > > > Reason: Connection timeout (0x01) > > > > @ MGMT Event: Device Disconnected (0x000c) plen 8 > > > > BR/EDR Address: E4:90:FD:7D:E7:5F (OUI E4-90-FD) > > > > Reason: Connection timeout (0x01) > > > > < HCI Command: Write Scan Enable (0x03|0x001a) plen 1 > > > > Scan enable: Page Scan (0x02) > > > > > HCI Event: Command Complete (0x0e) plen 4 > > > > Write Scan Enable (0x03|0x001a) ncmd 2 > > > > Status: Success (0x00) > > > > < HCI Command: Create Connection (0x01|0x0005) plen 13 > > > > Address: E4:90:FD:7D:E7:5F (OUI E4-90-FD) > > > > Packet type: 0xcc18 > > > > DM1 may be used > > > > DH1 may be used > > > > DM3 may be used > > > > DH3 may be used > > > > DM5 may be used > > > > DH5 may be used > > > > Page scan repetition mode: R2 (0x02) > > > > Page scan mode: Mandatory (0x00) > > > > Clock offset: 0x0000 > > > > Role switch: Allow slave (0x01) > > > > > HCI Event: Command Status (0x0f) plen 4 > > > > Create Connection (0x01|0x0005) ncmd 1 > > > > > > > > To fix this behavior, separate the ReconnectUUIDs into services that > > > > reconnect on timeout and on resume. This ensures that the a2dp-sink > > > > profile reconnects are correctly applied only on resume from suspend and > > > > adds the option to reconnect other profiles automatically on resume. > > > > > > > > Reviewed-by: Alain Michaud > > > > --- > > > > > > > > plugins/policy.c | 72 ++++++++++++++++++++++++++++++++++++------------ > > > > src/main.conf | 9 +++++- > > > > 2 files changed, 62 insertions(+), 19 deletions(-) > > > > > > > > diff --git a/plugins/policy.c b/plugins/policy.c > > > > index ba9e1be02..a3cc3445e 100644 > > > > --- a/plugins/policy.c > > > > +++ b/plugins/policy.c > > > > @@ -44,19 +44,22 @@ > > > > > > > > struct reconnect_data { > > > > struct btd_device *dev; > > > > - bool reconnect; > > > > + bool on_timeout; > > > > + bool on_resume; > > > > GSList *services; > > > > guint timer; > > > > bool active; > > > > unsigned int attempt; > > > > - bool on_resume; > > > > + bool mark_on_suspend; > > > > }; > > > > > > > > static const char *default_reconnect[] = { > > > > - HSP_AG_UUID, HFP_AG_UUID, A2DP_SOURCE_UUID, > > > > - A2DP_SINK_UUID, NULL }; > > > > + HSP_AG_UUID, HFP_AG_UUID, A2DP_SOURCE_UUID, NULL }; > > > > static char **reconnect_uuids = NULL; > > > > > > > > +static const char *default_reconnect_resume[] = { A2DP_SINK_UUID, NULL }; > > > > +static char **reconnect_resume_uuids = NULL; > > > > + > > > > static const size_t default_attempts = 7; > > > > static size_t reconnect_attempts = 0; > > > > > > > > @@ -551,14 +554,14 @@ static void reconnect_reset(struct reconnect_data *reconnect) > > > > } > > > > } > > > > > > > > -static bool reconnect_match(const char *uuid) > > > > +static bool reconnect_match_internal(const char *uuid, char **matchlist) > > > > { > > > > char **str; > > > > > > > > - if (!reconnect_uuids) > > > > + if (!matchlist) > > > > return false; > > > > > > > > - for (str = reconnect_uuids; *str; str++) { > > > > + for (str = matchlist; *str; str++) { > > > > if (!bt_uuid_strcmp(uuid, *str)) > > > > return true; > > > > } > > > > @@ -566,6 +569,16 @@ static bool reconnect_match(const char *uuid) > > > > return false; > > > > } > > > > > > > > +static bool reconnect_match(const char *uuid) > > > > +{ > > > > + return reconnect_match_internal(uuid, reconnect_uuids); > > > > +} > > > > + > > > > +static bool reconnect_resume_match(const char *uuid) > > > > +{ > > > > + return reconnect_match_internal(uuid, reconnect_resume_uuids); > > > > +} > > > > + > > > > static struct reconnect_data *reconnect_add(struct btd_service *service) > > > > { > > > > struct btd_device *dev = btd_service_get_device(service); > > > > @@ -687,10 +700,15 @@ static void service_cb(struct btd_service *service, > > > > * be the first profile that's connected so we might have an > > > > * entry but with the reconnect flag set to false. > > > > */ > > > > - if (!reconnect->reconnect) > > > > - reconnect->reconnect = reconnect_match(profile->remote_uuid); > > > > + if (!reconnect->on_timeout) > > > > + reconnect->on_timeout = reconnect_match(profile->remote_uuid); > > > > + > > > > + if (!reconnect->on_resume) > > > > + reconnect->on_resume = > > > > + reconnect_resume_match(profile->remote_uuid); > > > > > > > > - DBG("Added %s reconnect %u", profile->name, reconnect->reconnect); > > > > + DBG("Added %s reconnect on timeout(%u), resume(%u)", > > > > + profile->name, reconnect->on_timeout, reconnect->on_resume); > > > > } > > > > > > > > static gboolean reconnect_timeout(gpointer data) > > > > @@ -704,7 +722,7 @@ static gboolean reconnect_timeout(gpointer data) > > > > reconnect->timer = 0; > > > > > > > > /* Mark any reconnect on resume as handled */ > > > > - reconnect->on_resume = false; > > > > + reconnect->mark_on_suspend = false; > > > > > > > > err = btd_device_connect_services(reconnect->dev, reconnect->services); > > > > if (err < 0) { > > > > @@ -750,7 +768,7 @@ static void disconnect_cb(struct btd_device *dev, uint8_t reason) > > > > return; > > > > > > > > reconnect = reconnect_find(dev); > > > > - if (!reconnect || !reconnect->reconnect) > > > > + if (!reconnect || !(reconnect->on_timeout || reconnect->on_resume)) > > > > return; > > > > > > > > reconnect_reset(reconnect); > > > > @@ -759,11 +777,11 @@ static void disconnect_cb(struct btd_device *dev, uint8_t reason) > > > > > > > > switch (reason) { > > > > case MGMT_DEV_DISCONN_LOCAL_HOST_SUSPEND: > > > > - if (btd_device_get_service(dev, A2DP_SINK_UUID)) { > > > > + if (reconnect->on_resume) { > > > > Note: Previously, only A2DP-SINK would reconnect if disconnected during suspend. > > > > > > DBG("%s configured to reconnect on resume", > > > > device_get_path(dev)); > > > > > > > > - reconnect->on_resume = true; > > > > + reconnect->mark_on_suspend = true; > > > > > > > > /* If the kernel supports resume events, it is > > > > * preferable to set the reconnect timer there as it is > > > > @@ -774,7 +792,8 @@ static void disconnect_cb(struct btd_device *dev, uint8_t reason) > > > > } > > > > break; > > > > case MGMT_DEV_DISCONN_TIMEOUT: > > > > - reconnect_set_timer(reconnect, -1); > > > > + if (reconnect->on_timeout) > > > > + reconnect_set_timer(reconnect, -1); > > > > break; > > > > default: > > > > DBG("Developer error. Reason = %d", reason); > > > > @@ -790,7 +809,7 @@ static void policy_adapter_resume(struct btd_adapter *adapter) > > > > for (l = reconnects; l; l = g_slist_next(l)) { > > > > struct reconnect_data *reconnect = l->data; > > > > > > > > - if (reconnect->on_resume && > > > > + if (reconnect->mark_on_suspend && > > > > device_get_adapter(reconnect->dev) == adapter) { > > > > reconnect_set_timer(reconnect, resume_delay); > > > > } > > > > @@ -804,7 +823,7 @@ static void conn_fail_cb(struct btd_device *dev, uint8_t status) > > > > DBG("status %u", status); > > > > > > > > reconnect = reconnect_find(dev); > > > > - if (!reconnect || !reconnect->reconnect) > > > > + if (!reconnect || !(reconnect->on_timeout || reconnect->on_resume)) > > > > return; > > > > > > > > if (!reconnect->active) > > > > @@ -845,6 +864,7 @@ static int policy_init(void) > > > > { > > > > GError *gerr = NULL; > > > > GKeyFile *conf; > > > > + bool has_reconnect_uuids; > > > > > > > > service_id = btd_service_add_state_cb(service_cb, NULL); > > > > > > > > @@ -892,6 +912,15 @@ static int policy_init(void) > > > > auto_enable = g_key_file_get_boolean(conf, "Policy", "AutoEnable", > > > > NULL); > > > > > > > > + reconnect_resume_uuids = g_key_file_get_string_list( > > > > + conf, "Policy", "ReconnectResumeUUIDs", NULL, &gerr); > > > > + > > > > + if (gerr) { > > > > + g_clear_error(&gerr); > > > > + reconnect_resume_uuids = > > > > + g_strdupv((char **)default_reconnect_resume); > > > > + } > > > > + > > > > resume_delay = g_key_file_get_integer( > > > > conf, "Policy", "ResumeDelay", &gerr); > > > > > > > > @@ -900,7 +929,11 @@ static int policy_init(void) > > > > resume_delay = default_resume_delay; > > > > } > > > > done: > > > > - if (reconnect_uuids && reconnect_uuids[0] && reconnect_attempts) { > > > > + has_reconnect_uuids = > > > > + (reconnect_uuids && reconnect_uuids[0]) || > > > > + (reconnect_resume_uuids && reconnect_resume_uuids[0]); > > > > + > > > > + if (reconnect_attempts && has_reconnect_uuids) { > > > > btd_add_disconnect_cb(disconnect_cb); > > > > btd_add_conn_fail_cb(conn_fail_cb); > > > > } > > > > @@ -918,6 +951,9 @@ static void policy_exit(void) > > > > if (reconnect_uuids) > > > > g_strfreev(reconnect_uuids); > > > > > > > > + if (reconnect_resume_uuids) > > > > + g_strfreev(reconnect_resume_uuids); > > > > + > > > > g_free(reconnect_intervals); > > > > > > > > g_slist_free_full(reconnects, reconnect_destroy); > > > > diff --git a/src/main.conf b/src/main.conf > > > > index ad36638b7..6f66acb14 100644 > > > > --- a/src/main.conf > > > > +++ b/src/main.conf > > > > @@ -220,7 +220,7 @@ > > > > # timeout). The policy plugin should contain a sane set of values by > > > > # default, but this list can be overridden here. By setting the list to > > > > # empty the reconnection feature gets disabled. > > > > -#ReconnectUUIDs=00001112-0000-1000-8000-00805f9b34fb,0000111f-0000-1000-8000-00805f9b34fb,0000110a-0000-1000-8000-00805f9b34fb,0000110b-0000-1000-8000-00805f9b34fb > > > > +#ReconnectUUIDs=00001112-0000-1000-8000-00805f9b34fb,0000111f-0000-1000-8000-00805f9b34fb,0000110a-0000-1000-8000-00805f9b34fb > > > > > > > > # ReconnectAttempts define the number of attempts to reconnect after a link > > > > # lost. Setting the value to 0 disables reconnecting feature. > > > > @@ -237,6 +237,13 @@ > > > > # in later on. Defaults to 'false'. > > > > #AutoEnable=false > > > > > > > > +# The ReconnectResumeUUIDs defines the set of remote services that should try > > > > +# to be reconnected after resume from suspend if they were connected before > > > > +# suspend. If a device is disconnected due to suspend, it will be queued for > > > > +# reconnect on resume first using the ResumeDelay value and then the remaining > > > > +# values in ReconnectIntervals. > > > > +#ReconnectResumeUUIDs=0000110b-0000-1000-8000-00805f9b34fb > > > > > > I don't follow the reasoning here really, we need to be consistent > > > with regards to policy so how about the other profiles listed to be > > > reconnected does it means they no longer are connected on resume? From > > > the looks of you it we cannot really tell if the device was > > > disconnected due to link supervision timeout or not so we can't really > > > assume the Connection timeout (0x01) is in fact a second device > > > connecting, IMO if it is a mistake on the manufacturer then this > > > should be made device specific even though I think you will have to > > > disable the reconnection logic altogether if we cannot trust the > > > reason for the disconnection or does it cleanup the A2DP stream before > > > disconnecting, in that case we should have consider a clean > > > disconnection. > > > > ReconnectUUIDs was overloaded to mean both "reconnect on link loss" > > and "reconnect only a2dp-sink if disconnected due to suspend". This > > patch removes that overloading and ReconnectUUIDs simply always means, > > "Reconnect if link loss". ReconnectResumeUUIDs means simply, "If this > > device was disconnected during suspend, attempt to reconnect it on > > resume." It's adding a Reconnect policy for disconnect reason = > > "Disconnect due to Suspend". I think this is more generic and enables > > adding other uuids to this policy. > > To my understanding disconnected due to suspend was actually > considered a link loss due to suspend, if we were to carried out > different policies for these conditions the whole thing becomes a lot > more complicated, and to make matters worse this so far has been a > problem to a single manufacturer which in the future might actually > change the disconnect reason. How about the other profiles, if it were > connected to HFP shouldn't it be reconnected as well? Also would the > problem of stealing the connection still manifest when the laptop > resumes when one attempts to connect to the phone? > > > If we really want link loss reconnects for a2dp-sink, I agree that we > > should be treating the Airpods differently because it is specific to > > that device. > > > > This is what I'm seeing on the Airpods when I connect from an Android > > phone (while already connected to Chromebook). There is no profile > > disconnect (no l2cap disconnection requests either). > > > > > HCI Event: Number of Completed Packets (0x13) plen 5 #1674 [hci0] 38.810693 > > Num handles: 1 > > Handle: 256 > > Count: 1 > > < ACL Data TX: Handle 256 flags 0x02 dlen 969 > > #1675 [hci0] > > 38.820328 > > Channel: 11011 len 965 [PSM 25 mode Basic (0x00)] {chan 2} > > < ACL Data TX: Handle 256 flags 0x00 dlen 7 > > #1676 [hci0] > > 38.821054 > > Channel: 10756 len 3 [PSM 25 mode Basic (0x00)] {chan 3} > > AVDTP: Suspend (0x09) Command (0x00) type 0x00 label 4 nosp 0 > > ACP SEID: 1 > > > HCI Event: Number of Completed Packets (0x13) plen 5 #1677 [hci0] 38.835673 > > Num handles: 1 > > Handle: 256 > > Count: 1 > > > HCI Event: Number of Completed Packets (0x13) plen 5 #1678 [hci0] 38.836347 > > Num handles: 1 > > Handle: 256 > > Count: 1 > > > ACL Data RX: Handle 256 flags 0x02 dlen 6 #1679 [hci0] 38.861708 > > Channel: 67 len 2 [PSM 25 mode Basic (0x00)] {chan 3} > > AVDTP: Suspend (0x09) Response Accept (0x02) type 0x00 label 4 > > nosp 0 > > > HCI Event: Disconnect Complete (0x05) plen 4 #1680 [hci0] 62.656436 > > Status: Success (0x00) > > Handle: 256 > > Reason: Connection Timeout (0x08) > > This is a link loss in my books, so if any profiles on ReconnectUUIDs > were connected this still would trigger a reconnection and we can't > really remove everything from there, perhaps for Chrome OS only A2DP > is connected but in other cases HFP might be connected as well. Imo a > better way to handle this is with a quirk for broken reason, or > perhaps expose an interface where the upper layer can disable policies > on per-device, in which case the user can disable the UUIDs to be > reconnected. > > > > > > > > > > # Audio devices that were disconnected due to suspend will be reconnected on > > > > # resume. ResumeDelay determines the delay between when the controller > > > > # resumes from suspend and a connection attempt is made. A longer delay is > > > > -- > > > > 2.29.2.576.ga3fc446d84-goog > > > > > > > > > > > > > -- > > > Luiz Augusto von Dentz > > > > -- > Luiz Augusto von Dentz