lkml.org 
[lkml]   [2005]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [AIO] aio-2.6.13-rc6-B1
    From
    Date
    On Wed, 2005-08-17 at 14:44 -0400, Benjamin LaHaise wrote:
    > The bugfix followup to the last aio rollup is now available at:
    >
    > http://www.kvack.org/~bcrl/patches/aio-2.6.13-rc6-B1-all.diff
    >
    > with the split up in:
    >
    > http://www.kvack.org/~bcrl/patches/aio-2.6.13-rc6-B1/
    >
    > This fixes the bugs noticed in the -B0 variant. Major changes in this
    > patchset are:
    >
    > - added aio semaphore ops
    > - aio thread based fallbacks
    > - vectored aio file_operations
    > - aio sendmsg/recvmsg via thread fallbacks
    > - retry based aio pipe operations
    >
    > Comments?
    >
    > -ben
    >

    Hi Ben,

    what's the point of calling wake_up_locked(&sem->wait) in
    aio_down_wait? We're already in a wakeup path and end up
    calling __wake_up_common recursively.

    I think it may be one of the cause of my kernel hanging at the
    very beginning.

    When I remove this call things go further but at some point a
    semaphore wait queue gets thrashed and __wake_up_common tries to
    call an invalid callback function.

    Any input appreciated.

    Sébastien.

    --
    ------------------------------------------------------

    Sébastien Dugué BULL/FREC:B1-247
    phone: (+33) 476 29 77 70 Bullcom: 229-7770

    mailto:sebastien.dugue@bull.net

    Linux POSIX AIO: http://www.bullopensource.org/posix

    ------------------------------------------------------

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

    \
     
     \ /
      Last update: 2005-09-20 12:25    [W:0.033 / U:3.060 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site