[lkml]   [2000]   [Dec]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: Linux 2.2.19pre2
    On Sat, Dec 23, 2000 at 05:56:42PM +1100, Andrew Morton wrote:
    > If we elect to not address this problem in 2.2 and to rely upon the network

    I see. There are two races:

    1) race inside __wake_up when it's run on the same waitqueue: 2.2.19pre3
    is affected as well as 2.2.18aa2, and 2.4.x is affected
    as well when #defining USE_RW_WAIT_QUEUE_SPINLOCK to 1.
    2) race between two parallel __wake_up running on different waitqueues
    (both 2.2.x and 2.4.x are affected)

    1) could be fixed trivially by making the waitqueue_lock a spinlock, but
    this way doesn't solve 2). And if we solve 2) properly than 1) gets fixed as

    I agree the right fix for 2) (and in turn for 1) ) is to count the number of
    exclusive wake_up_process that moves the task in the runqueue, if the task was
    just in the runqueue we must not consider it as an exclusive wakeup (so in turn
    we'll try again to wakeup the next exclusive-wakeup waiter). This will
    fix both races. Since the fix is self contained in __wake_up it's fine
    for 2.2.19pre3 as well and we can keep using a read_write lock then.

    Those races of course are orthogonal with the issue we discussed previously
    in this thread: a task registered in two waitqueues and wanting an exclusive
    wakeup from one waitqueue and a wake-all from the other waitqueue (for
    addressing that we need to move the wake-one information from the task struct
    to the waitqueue_head and I still think that shoudln't be addressed in 2.2.x,
    2.2.x is fine with a per-task-struct wake-one information)

    Should I take care of the 2.2.x fix, or will you take care of it? I'm not using
    the wake-one patch in 2.2.19pre3 because I don't like it (starting from the
    useless wmb() in accept) so if you want to take care of 2.2.19pre3 yourself I'd
    suggest to apply the wake-one patch against 2.2.19pre3 in my ftp-patch area
    first. Otherwise give me an ack and I'll extend myself my wake-one patch to
    ignore the wake_up_process()es that doesn't move the task in the runqueue.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 12:52    [W:25.549 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site