[lkml]   [2006]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [rfc][patch] remove racy sync_page?
    On Wed, May 31 2006, Linus Torvalds wrote:
    > On Wed, 31 May 2006, Linus Torvalds wrote:
    > >
    > > The reason it's kicked by wait_on_page() is that is when it's needed.
    > Btw, that's not how it has always been done.
    > For the longest time, it was actually triggered by scheduler activity, in
    > particular, plugging used to be a workqueue event that was triggered by
    > the scheduler (or any explicit points when you wanted it to be triggered
    > earlier).

    Now it's time for me to give Linus a history lesson on plugging,

    Plugging used to be done by the issuer and with immediate unplugging
    when you were done issuing the blocks. Both of these actions happened in
    ll_rw_block() if the caller was submitting more than on buffer_head, and
    it happened without the caller knowing about plugging. He never had to

    1.2 then expanded that to be able to plug more than one device at the
    time. It didn't really do much except allow the array of buffers passed
    in being on separate devices. The plugging was still hidden from the

    1.3 and on introduced a more generalised infrastructure for this, moving
    the plugging to a task queue (tq_disk). This meant that we could finally
    separate the plugging and unplugging from the direct IO issue. So
    whenever someone wanted to the a wait_on_buffer/lock_page() equiv for
    something that might to be issued, it would have to do a
    run_task_queue(&tq_disk) first which would then unplug all the queues
    that were plugged.

    tq_disk was then removed and moved to a block list during the 2.5
    massive io/bio changes. The functionality remained the same, though -
    you had to kick all queues to force the unplug of the page you wanted.
    This infrastructure lasted all up to the point where silly people with
    lots of CPU's started complaining about lock contention for 32-way
    systems with thousands of disks. This is the point where I reevaluated
    the benefits of plugging, found it good, and decided to fix it up.
    Plugging then became a simple state bit in the queue, and you would have
    to pass in eg the page you wanted when asking to unplug. This would kick
    just the specific queue you needed. It also got a timer tied to it, so
    that we could unplug after foo msecs if we wanted. Additionally, it will
    also self-unplug once a certain plug depth has been reached (like 4

    Anyway, the point I wanted to make is that this was never driven by
    scheduler activity. So there!

    Jens Axboe

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

     \ /
      Last update: 2006-05-31 20:14    [W:0.021 / U:42.640 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site