RE: [connman] CreatDevice signal should be emitted at function create_stored_device_from_profiles() in src/adapter.c

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

 



Thanks for your feedback.
I will try release 4.36 whence you release it.
If it does work, then we need not change anything at common side. 

-----Original Message-----
From: connman-bounces@xxxxxxxxxx [mailto:connman-bounces@xxxxxxxxxx] On Behalf Of Luiz Augusto von Dentz
Sent: 2009年4月17日 20:47
To: connman@xxxxxxxxxx
Cc: Yin, Kangkai; linux-bluetooth@xxxxxxxxxxxxxxx; Liu, Bing Wei; Gao, Bin
Subject: Re: [connman] CreatDevice signal should be emitted at function create_stored_device_from_profiles() in src/adapter.c

Hi,

On Thu, Apr 16, 2009 at 10:47 PM, Xu, Martin <martin.xu@xxxxxxxxx> wrote:
>>I know the way you mentioned, but this way has some issue. You know there is >a time interval between AdapterAdded and DeviceCreated.
> I found that the time interval on my machine is about 230 milliseconds. The time is enough to schedule out to other process to handle the AdapterAdded signal.

There is a fix for this race condition already on git, release 4.36
will be shipped with it. AdapterAdded is now emitted when adapter
properties are "ready", the problem was that we need the adapter
address for accessing the storage but that is only available when
receiving DEVUP event but we were actually emitting AdapterAdded
before when handling DEVREG event. The interval you mentioned is
probably the interval between DEVREG and DEVUP, with the fix it
shouldn't happen anymore since AdapterAdded will only be triggered by
the very first DEVUP to that device.

-- 
Luiz Augusto von Dentz
Engenheiro de Computação
_______________________________________________
connman mailing list
connman@xxxxxxxxxx
https://lists.moblin.org/mailman/listinfo/connman
��.n��������+%������w��{.n�����{����^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�m


[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