Re: [PATCHv2] adapter: Handle MGMT_SETTING_LE change

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Petri,

On Tue, Feb 18, 2014, Petri Gynther wrote:
> On dual-mode Bluetooth adapters, BLE is not enabled by default. When bluetoothd
> starts, BLE is enabled in read_info_complete(), but this management action does
> not always complete before adapter_start() is called. Therefore, bluetoothd
> needs to handle the MGMT_SETTING_LE change correctly in settings_changed().
> Specifically, trigger_passive_scanning() needs to be called so that paired BLE
> devices (e.g. HoG devices) are able to reconnect back to host.
> 
> The trace below shows this case:
> bluetoothd[1087]: src/adapter.c:adapter_start() adapter /org/bluez/hci0 has been enabled
>  => adapter_start() is called, but BLE is not yet enabled
> bluetoothd[1087]: src/adapter.c:load_link_keys_complete() link keys loaded for hci0
> bluetoothd[1087]: src/adapter.c:load_ltks_complete() LTKs loaded for hci0
> bluetoothd[1087]: src/adapter.c:get_connections_complete() Connection count: 0
> bluetoothd[1087]: src/adapter.c:new_settings_callback() Settings: 0x000000c1
> bluetoothd[1087]: src/adapter.c:settings_changed() Changed settings: 0x00000040
> bluetoothd[1087]: src/adapter.c:new_settings_callback() Settings: 0x000002c1
> bluetoothd[1087]: src/adapter.c:settings_changed() Changed settings: 0x00000200
>  => BLE got enabled, so need to call trigger_passive_scanning()
> bluetoothd[1087]: src/adapter.c:new_settings_callback() Settings: 0x000002d1
> bluetoothd[1087]: src/adapter.c:settings_changed() Changed settings: 0x00000010
> bluetoothd[1087]: src/adapter.c:new_settings_callback() Settings: 0x000002d3
> bluetoothd[1087]: src/adapter.c:settings_changed() Changed settings: 0x00000002
> 
> The issue is easily reproducible with this init sequence:
> hciconfig hci0 up
> bluetoothd -n -d
> ---
>  src/adapter.c | 7 +++++++
>  1 file changed, 7 insertions(+)

Applied (after reformatting the commit message a bit to follow the "less
than 76 chars wide" rule). Thanks.

Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux