Return-Path: Date: Fri, 17 Apr 2009 16:08:48 +0300 From: Johan Hedberg To: Bastien Nocera Cc: Luiz Augusto von Dentz , connman@moblin.org, "Yin, Kangkai" , "linux-bluetooth@vger.kernel.org" , "Liu, Bing Wei" , "Gao, Bin" Subject: Re: [connman] CreatDevice signal should be emitted at function create_stored_device_from_profiles() in src/adapter.c Message-ID: <20090417130848.GA11008@localhost> References: <49D30FD8.9020506@binarywings.net> <49D45501.8030606@binarywings.net> <2d5a2c100904021028g78376debke885916eba13cf93@mail.gmail.com> <49E5F2BE.1040808@binarywings.net> <9F0C1DB20AFA954FA1DA05309350433D5B2E7CEF@pdsmsx503.ccr.corp.intel.com> <2d5a2c100904160631o30384806td1b5750669466fce@mail.gmail.com> <9F0C1DB20AFA954FA1DA05309350433D5B2E7F5F@pdsmsx503.ccr.corp.intel.com> <9F0C1DB20AFA954FA1DA05309350433D5B2E8031@pdsmsx503.ccr.corp.intel.com> <2d5a2c100904170546l5062da8xb8e17e3ffc55b455@mail.gmail.com> <1239973488.9313.3017.camel@cookie.hadess.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed In-Reply-To: <1239973488.9313.3017.camel@cookie.hadess.net> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Bastien, On Fri, Apr 17, 2009, Bastien Nocera wrote: >On Fri, 2009-04-17 at 09:46 -0300, Luiz Augusto von Dentz wrote: >> Hi, >> >> On Thu, Apr 16, 2009 at 10:47 PM, Xu, Martin 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. > >Does this mean I can remove the work-arounds in gnome-bluetooth, to >check for the "Powered" property change before enumerating the devices? Yes, it should mean just that. Johan