Return-Path: MIME-Version: 1.0 In-Reply-To: References: <88feb93f-928f-9783-7d16-b4b87b402a1a@gmail.com> <73597a16-0a96-4d6f-64f5-46a7bb3f983a@gmail.com> From: Luiz Augusto von Dentz Date: Tue, 6 Feb 2018 11:30:40 -0200 Message-ID: Subject: Re: Apparent bluez 5.48 regression: Headphones fail to reconnect after suspend/resume To: Vincent Petry Cc: Nathaniel McCallum , "linux-bluetooth@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Vincent, On Tue, Feb 6, 2018 at 11:17 AM, Vincent Petry wrote: > Hi Luiz, > >> Hi Vincent, >> >> On Tue, Feb 6, 2018 at 4:53 AM, Vincent Petry wrote: >>> Hello, >>> >>> I have bisected bluez between 5.47 (good) and 5.48 (bad) to find the >>> breaking commit for the headset reconnect issue. >>> >>> Here are the results: >>> >>> git bisect start >>> # bad: [0d1e3b9c5754022c779da129025d493a198d49cf] Release 5.48 >>> git bisect bad 0d1e3b9c5754022c779da129025d493a198d49cf >>> # good: [d139fd866241fe0d99b5e430f937c8d6160cc7dd] Release 5.47 >>> git bisect good d139fd866241fe0d99b5e430f937c8d6160cc7dd >>> # bad: [65aaf36f2a36895e4a351ac3fa1cb8da87d4589c] mesh: Correct length >>> test in agent.c:request_ascii >>> git bisect bad 65aaf36f2a36895e4a351ac3fa1cb8da87d4589c >>> # good: [c50a8a397d4abd994a9115230279d5fe922b4aa5] adapter: Add >>> btd_request_authorization_cable_configured() >>> git bisect good c50a8a397d4abd994a9115230279d5fe922b4aa5 >>> # bad: [c133489d54cb6d28c3fd308557937acbc5245f5e] battery: Add BT SIG >>> reserved number used by Battery Service >>> git bisect bad c133489d54cb6d28c3fd308557937acbc5245f5e >>> # good: [e577e478e9cb1d1a22e63fd7d8fff07c471590de] gatt: Add >>> implementation of link option >>> git bisect good e577e478e9cb1d1a22e63fd7d8fff07c471590de >>> # bad: [5b9596dac4d0e25c5179be8643726a02c058b00a] advertising: Add >>> implementation of Duration and Timeout >>> git bisect bad 5b9596dac4d0e25c5179be8643726a02c058b00a >>> # bad: [f9a2b1f515c7f5dced80397f4ea891d6c372175d] client: Fix not >>> detecting advertising instance is no longer valid >>> git bisect bad f9a2b1f515c7f5dced80397f4ea891d6c372175d >>> # bad: [d6e9539e31c6bb5afd39ec6f09c518d232e6345d] doc/advertising-api: >>> Mark LEAdvertisingManager1 stable >>> git bisect bad d6e9539e31c6bb5afd39ec6f09c518d232e6345d >>> # good: [10760c91c234fe2bfedf924c9e61f31861c2dc72] gatt: Fix crash while >>> disconnecting ATT >>> git bisect good 10760c91c234fe2bfedf924c9e61f31861c2dc72 >>> # first bad commit: [d6e9539e31c6bb5afd39ec6f09c518d232e6345d] >>> doc/advertising-api: Mark LEAdvertisingManager1 stable >> Im not able to decipher this, what is the patch where the problem >> first appeared? Also could you collect the HCI trace and syslog when >> it happens? > > Please try this: > > 1. git clone git://git.kernel.org/pub/scm/bluetooth/bluez.git > 2. cd bluez > 3. git show d6e9539e31c6bb5afd39ec6f09c518d232e6345d > > This will show your the first bad commit / the patch that introduces the > issue. > > For the syslog, see attachments to my original report here > https://bugzilla.suse.com/show_bug.cgi?id=1076898 > > Do you have more information about how I could get the HCI trace ? > Should I run bluetoothd through gdb and query the backtrace from the > point where the error message happens, if that's what you are asking for ? Jan 20 15:50:47 vvortex.ttv pulseaudio[1964]: [pulseaudio] bluez5-util.c: Information about device /org/bluez/hci0/dev_E3_28_E9_20_58_08 is invalid Jan 20 15:50:47 vvortex.ttv bluetoothd[3136]: Endpoint replied with an error: org.bluez.Error.InvalidArguments So this is really PulseAudio rejecting the connection, it probably loose track of the objects when resuming or we did not send the objects but in that case bluetoothctl would not be able to list them as well.