Re: [PATCH] Fix a dead loop in async_synchronize_full()

From: Li Zhong
Date: Sun Jul 08 2012 - 21:33:33 EST


On Fri, 2012-07-06 at 17:12 +0800, Cong Wang wrote:
> On Mon, Jul 2, 2012 at 2:04 PM, Li Zhong <zhong@xxxxxxxxxxxxxxxxxx> wrote:
> > void async_synchronize_full(void)
> > {
> > - do {
> > - async_synchronize_cookie(next_cookie);
> > - } while (!list_empty(&async_running) || !list_empty(&async_pending));
> > + async_synchronize_cookie_domain(next_cookie, NULL);
> > }
>
> A nitpick:
>
> async_synchronize_cookie_domain(next_cookie, NULL);
>
> equals to
>
> async_synchronize_full_domain(NULL)
>

Why do we need one more level in the function calling stack?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/