RE: [PATCH] kernel: async: Fixed the wrong waitting condition of wait_event

["Chunyan Zhang (张春艳)" <Chunyan.Zhang@spreadtrum.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb2312", Size: 1021 bytes --]

Hi, Andrew

Could you give us the test log including the runtime problems you've got?

Thank you so much.


Best regards
Chunyan

On Mon, 5 Jan 2015 11:35:35 +0800 Chunyan Zhang <chunyan.zhang@spreadtrum.com> wrote:

> This patch changes the waitting condition of wait_event in the
> function 'async_synchronize_cookie_domain'.
>
> This function waits until all asynchronous function calls for the certain
> domain have been done.
>
> But, the function 'lowest_in_progress' returns the lowest *pending*
> entry's cookie.
>
> So we should make sure that the return value of lowest_in_progress(domain)
> must be larger than @cookie.

I'm having runtime problems with this.  Twice the kernel silently hangs
late in initcalls and the third time I got a great stream of oopses.

I'll drop it.  Hopefully you'll be able to work through the bug with
Arjan and Tejun.

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥