[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d82e647a0905121736n7a73e166h61a899d8e4a63d0a@mail.gmail.com>
Date: Wed, 13 May 2009 08:36:05 +0800
From: Ming Lei <tom.leiming@...il.com>
To: Américo Wang <xiyou.wangcong@...il.com>
Cc: Frederic Weisbecker <fweisbec@...il.com>, arjan@...radead.org,
linux-kernel@...r.kernel.org, akpm@...ux-foundation.org
Subject: Re: [PATCH] kernel/async.c:introduce async_schedule*_atomic
2009/5/12 Américo Wang <xiyou.wangcong@...il.com>:
> On Tue, May 12, 2009 at 05:44:58PM +0200, Frederic Weisbecker wrote:
>>On Tue, May 12, 2009 at 11:13:42PM +0800, tom.leiming@...il.com wrote:
>>> From: Ming Lei <tom.leiming@...il.com>
>>>
>>> - if (!async_enabled || !entry || atomic_read(&entry_count) > MAX_WORK) {
>>> + sync_run = !async_enabled || !entry || \
>>> + atomic_read(&entry_count) > MAX_WORK;
>
>
> And also this one...
>
> Hey, Ming, none of these are macros... You don't need to add backslash to
> join two lines, C compilers can recognize.
OK, I'll fix it in next version.
Thanks!
>
> --
> Live like a child, think like the god.
>
>
--
Lei Ming
--
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