From: "José Bollo" <jose.bollo@iot.bzh>
To: Petri Gynther <pgynther@google.com>
Cc: Luiz Augusto von Dentz <luiz.dentz@gmail.com>,
linux-bluetooth <linux-bluetooth@vger.kernel.org>
Subject: Re: BlueZ 5.43: HoG peripheral services re-discovered again and again on every reconnect
Date: Fri, 9 Dec 2016 09:43:20 +0100 [thread overview]
Message-ID: <CA+LDafDDb9J--mg7EQ9bOY0DWHeDsZRrgOXCTEkBn4TESQK2Tw@mail.gmail.com> (raw)
In-Reply-To: <CAGXr9JGdZ_RezX45dqUNmeow--r+QnCQjc+4aggWu7m2uifKyw@mail.gmail.com>
Hi Petri,
This is perhaps related to the question I asked yesterday.
Where can I find your 4 patches?
Best regards
Jos=C3=A9 Bollo
Jos=C3=A9 Bollo - Senior Software Engineer
www.iot.bzh
2016-12-08 22:52 GMT+01:00 Petri Gynther <pgynther@google.com>:
> Hi Luiz,
>
> On Fri, Dec 2, 2016 at 3:56 AM, Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
>> Hi Petri,
>>
>> On Thu, Dec 1, 2016 at 11:12 PM, Petri Gynther <pgynther@google.com> wro=
te:
>>> However, in this case, I think this handling is correct.
>>> load_gatt_db() showed "No cache for LEFQAU", so BlueZ has to
>>> re-discover everything from the device over BLE.
>>> But, since the device disconnected in the middle of discovery, BlueZ
>>> couldn't fully populate the GATT DB and has to clean it.
>>> If the device didn't disconnect so quickly, this would have most likely=
worked.
>>>
>>>> But I do agree that we should not reset the db if that was populated
>>>> already, so I will try to fix that.
>>>
>>> Yes, once BlueZ does not reset the GATT DB on failed reconnects, this
>>> problem will be solved.
>>
>> Please check if everything is alright when using upstream, it should
>> clear the database anymore if we get an early disconnect.
>
> I applied my 2 patches and your 2 patches on top of 5.43.
>
> GATT DB is no longer destroyed on reconnection errors, so that's excellen=
t.
>
> However, we are still seeing reconnect problems, but not GATT DB related:
> bluez: bluetoothd[17780]:
> src/gatt-client.c:btd_gatt_client_connected() Device connected.
> bluez: bluetoothd[17780]: profiles/gap/gas.c:read_device_name_cb()
> Reading device name failed with ATT error: 0
> bluez: bluetoothd[17780]: profiles/gap/gas.c:read_appearance_cb()
> Reading appearance failed with ATT error: 0
> bluez: bluetoothd[17780]: Error reading PNP_ID value: Unexpected error co=
de
> bluez: bluetoothd[17780]: BATT level_read_char_cb failed: Request
> attribute has encountered an unlikely error
>
> We are still investigating this. Likely a firmware issue on the remote.
>
>>
>>
>> --
>> Luiz Augusto von Dentz
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth=
" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-12-09 8:43 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-09 19:39 BlueZ 5.43: HoG peripheral services re-discovered again and again on every reconnect Petri Gynther
2016-11-10 14:17 ` Luiz Augusto von Dentz
2016-11-11 2:42 ` Petri Gynther
2016-11-11 9:07 ` Luiz Augusto von Dentz
[not found] ` <CAGXr9JET_-mB1qtOryXpUMhQPTe_ecw84f4kFH+Wv9_WYp3iog@mail.gmail.com>
2016-11-29 23:15 ` Petri Gynther
2016-11-30 8:36 ` Luiz Augusto von Dentz
2016-11-30 9:37 ` Luiz Augusto von Dentz
2016-11-30 22:15 ` Petri Gynther
2016-11-30 22:55 ` Petri Gynther
2016-12-01 7:54 ` Luiz Augusto von Dentz
2016-12-01 8:21 ` Luiz Augusto von Dentz
2016-12-01 21:12 ` Petri Gynther
2016-12-02 11:56 ` Luiz Augusto von Dentz
2016-12-08 21:52 ` Petri Gynther
2016-12-09 8:43 ` José Bollo [this message]
2016-12-09 17:38 ` Petri Gynther
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=CA+LDafDDb9J--mg7EQ9bOY0DWHeDsZRrgOXCTEkBn4TESQK2Tw@mail.gmail.com \
--to=jose.bollo@iot.bzh \
--cc=linux-bluetooth@vger.kernel.org \
--cc=luiz.dentz@gmail.com \
--cc=pgynther@google.com \
/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).