[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20080819161512.GA5401@joi>
Date: Tue, 19 Aug 2008 18:15:19 +0200
From: Marcin Slusarz <marcin.slusarz@...il.com>
To: Kevin Diggs <kevdig@...ersurf.com>
Cc: linux-kernel@...r.kernel.org
Subject: Re: [Fwd: Corrections please ...]
On Tue, Aug 19, 2008 at 01:07:35AM -0700, Kevin Diggs wrote:
> Dave Chinner wrote:
>> On Mon, Aug 18, 2008 at 06:53:51PM -0700, Kevin Diggs wrote:
>>> Could someone ... anyone take a look at these kernel doc additions?
>> .....
>>> +/**
>>> + * complete: - signals a single thread waiting on this completion
>>> + * @x: holds the state of this particular completion
>>> + *
>>> + * This will wake up a single thread waiting on this completion. If
>>> multiple
>> Your mailer appears to be wrapping lines.
>>> + * threads are waiting ???
>>> + */
>>> void complete(struct completion *x)
>>> {
>>> unsigned long flags;
>> complete() will only wake one waiting thread. If there are multiple
>> waiters, then further calls to complete() are required to wake them,
>> or a single call to complete_all() could be used.
>>> EXPORT_SYMBOL(wait_for_completion);
>>>
>>> +/**
>
> Take 2 ...
You might want to read Documentation/SubmittingPatches.
(Especially points 2, 7, 11 and 12)
Marcin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists