From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nishanth Aravamudan Date: Thu, 03 Feb 2005 00:25:02 +0000 Subject: [KJ] Re: [PATCH 19/20] bluetooth/bluecard_cs: remove Message-Id: <20050203002502.GB2546@us.ibm.com> MIME-Version: 1 Content-Type: multipart/mixed; boundary="===============12275851463776521==" List-Id: References: <20050202225855.GY2546@us.ibm.com> <1107386189.11944.21.camel@pegasus> In-Reply-To: <1107386189.11944.21.camel@pegasus> To: Marcel Holtmann Cc: Max Krasnyansky , bluez-devel@lists.sourceforge.net, kernel-janitors@lists.osdl.org --===============12275851463776521== Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Feb 03, 2005 at 12:16:29AM +0100, Marcel Holtmann wrote: > Hi Nish, > > > Please consider applying. > > > > Description: Remove deprecated interruptible_sleep_on_timeout() function calls > > and replace with direct wait-queue usage. Some naming conflicts occurred, which > > I tried to resolve appropriately. Patch is compile-tested. > > you introduce more code than you remove. Is our code a special case or > why don't exists a simple API function as replacement? Yes, I do. *sleep_on*() was the API, which has been replaced with wait_event*(). However, wait_event*() explicitly needs a condition (the "event"), which in thise case does not exist. If I had more background, I would have gone the wait_event*() route. I am looking into a better API to mask away all of the DEFINE_WAIT(), prepare_to_wait(), schedule()/schedule_timeout(), finish_wait() additions (the KJ list can attest to how many of these replacements I'm doing), but there simply does not exist such an API right now. Any input you could provide, though, would be greatly appreciated. -Nish --===============12275851463776521== Content-Type: text/plain; charset="iso-8859-1" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline _______________________________________________ Kernel-janitors mailing list Kernel-janitors@lists.osdl.org http://lists.osdl.org/mailman/listinfo/kernel-janitors --===============12275851463776521==-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 2 Feb 2005 16:25:02 -0800 From: Nishanth Aravamudan To: Marcel Holtmann Cc: Max Krasnyansky , bluez-devel@lists.sourceforge.net, kernel-janitors@lists.osdl.org Subject: Re: [PATCH 19/20] bluetooth/bluecard_cs: remove interruptible_sleep_on_timeout() usage Message-ID: <20050203002502.GB2546@us.ibm.com> References: <20050202225855.GY2546@us.ibm.com> <1107386189.11944.21.camel@pegasus> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1107386189.11944.21.camel@pegasus> List-ID: On Thu, Feb 03, 2005 at 12:16:29AM +0100, Marcel Holtmann wrote: > Hi Nish, > > > Please consider applying. > > > > Description: Remove deprecated interruptible_sleep_on_timeout() function calls > > and replace with direct wait-queue usage. Some naming conflicts occurred, which > > I tried to resolve appropriately. Patch is compile-tested. > > you introduce more code than you remove. Is our code a special case or > why don't exists a simple API function as replacement? Yes, I do. *sleep_on*() was the API, which has been replaced with wait_event*(). However, wait_event*() explicitly needs a condition (the "event"), which in thise case does not exist. If I had more background, I would have gone the wait_event*() route. I am looking into a better API to mask away all of the DEFINE_WAIT(), prepare_to_wait(), schedule()/schedule_timeout(), finish_wait() additions (the KJ list can attest to how many of these replacements I'm doing), but there simply does not exist such an API right now. Any input you could provide, though, would be greatly appreciated. -Nish