From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
To: Archie Pusaka <apusaka@google.com>
Cc: linux-bluetooth <linux-bluetooth@vger.kernel.org>,
CrosBT Upstreaming <chromeos-bluetooth-upstreaming@chromium.org>,
Archie Pusaka <apusaka@chromium.org>
Subject: Re: [Bluez PATCH v1] adapter: Remove temporary devices before power off
Date: Wed, 6 Jan 2021 11:05:13 -0800 [thread overview]
Message-ID: <CABBYNZKH+iUPyU6QRjkoe6-R7cAFptmooSvOqO1f=jds=1VJSg@mail.gmail.com> (raw)
In-Reply-To: <CABBYNZK6fZ=rFqbhZeKC2ftoBdwrpZAeR_Pp2-4CYLB8hXzLDg@mail.gmail.com>
Hi Archie,
On Wed, Jan 6, 2021 at 10:24 AM Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
>
> Hi Archie,
>
> On Wed, Jan 6, 2021 at 1:27 AM Archie Pusaka <apusaka@google.com> wrote:
> >
> > From: Archie Pusaka <apusaka@chromium.org>
> >
> > If adapter is powered off when a currently connected device is
> > being removed, there is a possibility that we haven't finish waiting
> > for the disconnection but the adapter is already powered down.
> >
> > When this happens, the kernel would fail to clean the device's
> > information, for example the pairing information. This causes
> > disagreement between the user space and the kernel about whether the
> > device is already paired, because the device is successfully removed
> > from the user space's perspective.
> >
> > This patch enforces the removal of such devices before allowing the
> > adapter to power off.
> > ---
> >
> > src/adapter.c | 20 +++++++++++++++++++-
> > 1 file changed, 19 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/adapter.c b/src/adapter.c
> > index ec6a6a64c5..92d1cb2232 100644
> > --- a/src/adapter.c
> > +++ b/src/adapter.c
> > @@ -517,6 +517,7 @@ static void adapter_stop(struct btd_adapter *adapter);
> > static void trigger_passive_scanning(struct btd_adapter *adapter);
> > static bool set_mode(struct btd_adapter *adapter, uint16_t opcode,
> > uint8_t mode);
> > +static void remove_temporary_devices(struct btd_adapter *adapter);
>
> I would have the function above declared just before the first user
> that way we don't have to use a forward declaration like above.
>
> > static void settings_changed(struct btd_adapter *adapter, uint32_t settings)
> > {
> > @@ -622,6 +623,8 @@ static bool set_mode(struct btd_adapter *adapter, uint16_t opcode,
> > switch (mode) {
> > case MGMT_OP_SET_POWERED:
> > setting = MGMT_SETTING_POWERED;
> > + if (!mode)
> > + remove_temporary_devices(adapter);
> > break;
> > case MGMT_OP_SET_CONNECTABLE:
> > setting = MGMT_SETTING_CONNECTABLE;
> > @@ -2888,8 +2891,10 @@ static void property_set_mode(struct btd_adapter *adapter, uint32_t setting,
> > param = &mode;
> > len = sizeof(mode);
> >
> > - if (!mode)
> > + if (!mode) {
> > clear_discoverable(adapter);
> > + remove_temporary_devices(adapter);
> > + }
> >
> > break;
> > case MGMT_SETTING_DISCOVERABLE:
> > @@ -5304,6 +5309,19 @@ static void remove_discovery_list(struct btd_adapter *adapter)
> > adapter->discovery_list = NULL;
> > }
> >
> > +static void remove_temporary_devices(struct btd_adapter *adapter)
> > +{
> > + GSList *l, *next;
> > +
> > + for (l = adapter->devices; l; l = next) {
> > + struct btd_device *dev = l->data;
> > +
> > + next = g_slist_next(l);
> > + if (device_is_temporary(dev))
> > + btd_adapter_remove_device(adapter, dev);
> > + }
> > +}
> > +
> > static void adapter_free(gpointer user_data)
> > {
> > struct btd_adapter *adapter = user_data;
> > --
> > 2.29.2.729.g45daf8777d-goog
> >
>
>
> --
> Luiz Augusto von Dentz
Applied, thanks. I went ahead and did the changes myself.
--
Luiz Augusto von Dentz
prev parent reply other threads:[~2021-01-06 19:06 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-06 9:26 [Bluez PATCH v1] adapter: Remove temporary devices before power off Archie Pusaka
2021-01-06 9:51 ` [Bluez,v1] " bluez.test.bot
2021-01-06 18:24 ` [Bluez PATCH v1] " Luiz Augusto von Dentz
2021-01-06 19:05 ` Luiz Augusto von Dentz [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CABBYNZKH+iUPyU6QRjkoe6-R7cAFptmooSvOqO1f=jds=1VJSg@mail.gmail.com' \
--to=luiz.dentz@gmail.com \
--cc=apusaka@chromium.org \
--cc=apusaka@google.com \
--cc=chromeos-bluetooth-upstreaming@chromium.org \
--cc=linux-bluetooth@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).