lkml.org 
[lkml]   [2005]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Subject[PATCH linux-2.6-block:post-2.6.15 00/11] blk: reimplementation of I/O barrier
    Date
    Hello, Jens, James, Jeff and Bartlomiej.

    This is the sixth posting of blk ordered reimplementation. The last
    posting was on 17th, Nov.

    http://marc.theaimsgroup.com/?l=linux-kernel&m=113224210310915&w=2

    Changes from the previous post are...

    1. Proxy requests (pre_flush_rq, bar_rq and post_flush_rq) are inlined
    into request_queue such that blk_queue_ordered doesn't have to
    allocate when changing ordered mode. This change removes @gfp_mask
    argument from blk_queue_ordered. Also, there's no need to reset
    ordered to NONE when a device goes offline anymore. So, changes to
    to ->remove or ->release callbacks are gone.

    2. blk_queue_ordered() does not synchronize using queuelock.
    Synchronization is now the caller's responsibility. It's still
    perfectly safe to call blk_queue_ordered() without any
    synchronization. The worst that can happen is wrong ordered
    sequence issued around the call.

    3. Note that queuelock is not enough for proper ordered-mode change
    synchronization. Currently, there's no block driver which requires
    this. Drivers either don't change ordered mode during operation or
    doesn't properly synchronize configuration changes in the first
    place (IDE).

    This change removes for blk_queue_ordered_locked() function.

    4. IDE ordered/fua support has been updated. As IDE supports dynamic
    configuration change from EH and user input, ordered/fua settings
    should also be changed accordingly. New IDE HL driver callback
    ->protocol_changed is implmemented and used to properly configure
    ordered mode on configuration changes.

    Jens, #1 and #2 are changes to core ordered code but they are quite
    simple changes. There shouldn't be any behavior change.

    Bartlomiej, IDE ordered/fua support is now splitted over three patches
    08-ide-update-ordered, 09-ide-add-protocol_changed and
    10-ide-add-fua-support. I think #08 should be okay, but if you don't
    like #09 and #10, we can skip them for now. So....

    If #09 and #10 are ack'd, we can push all patches into -mm. If #09
    and #10 are nack'd, we can push all patches except the two into -mm.

    Thanks.

    [ Start of patch descriptions ]

    01_blk_add-uptodate-to-end_that_request_last.patch
    : add @uptodate to end_that_request_last() and @error to rq_end_io_fn()

    Add @uptodate argument to end_that_request_last() and @error
    to rq_end_io_fn(). There's no generic way to pass error code
    to request completion function, making generic error handling
    of non-fs request difficult (rq->errors is driver-specific and
    each driver uses it differently). This patch adds @uptodate
    to end_that_request_last() and @error to rq_end_io_fn().

    For fs requests, this doesn't really matter, so just using the
    same uptodate argument used in the last call to
    end_that_request_first() should suffice. IMHO, this can also
    help the generic command-carrying request Jens is working on.

    02_blk_implement-init_request_from_bio.patch
    : separate out bio init part from __make_request

    Separate out bio initialization part from __make_request. It
    will be used by the following blk_ordered_reimpl.

    03_blk_reimplement-ordered.patch
    : reimplement handling of barrier request

    Reimplement handling of barrier requests.

    * Flexible handling to deal with various capabilities of
    target devices.
    * Retry support for falling back.
    * Tagged queues which don't support ordered tag can do ordered.

    04_blk_scsi-update-ordered.patch
    : update SCSI to use new blk_ordered

    All ordered request related stuff delegated to HLD. Midlayer
    now doens't deal with ordered setting or prepare_flush
    callback. sd.c updated to deal with blk_queue_ordered
    setting. Currently, ordered tag isn't used as SCSI midlayer
    cannot guarantee request ordering.

    05_blk_scsi-add-fua-support.patch
    : add FUA support to SCSI disk

    Add FUA support to SCSI disk.

    06_blk_libata-update-ordered.patch
    : update libata to use new blk_ordered

    Reflect changes in SCSI midlayer and updated to use new
    ordered request implementation

    07_blk_libata-add-fua-support.patch
    : add FUA support to libata

    Add FUA support to libata.

    08_blk_ide-update-ordered.patch
    : update IDE to use new blk_ordered

    Update IDE to use new blk_ordered. This change makes the
    following behavior changes.

    * Partial completion of the barrier request is handled as
    failure of the whole ordered sequence. No more partial
    completion for barrier requests.

    * Any failure of pre or post flush request results in failure
    of the whole ordered sequence.

    So, successfully completed ordered sequence guarantees that
    all requests prior to the barrier made to physical medium and,
    then, the while barrier request made to the physical medium.

    09_blk_ide-add-protocol_changed.patch
    : implement ide_driver_t->protocol_changed callback

    This patch implements protocol_changed callback for IDE HL
    drivers. The callback is called whenever transfer protocol
    changes (DMA / multisector PIO / PIO). The callback is
    sometimes with context and sometimes without, sometimes with
    queuelock, sometimes not. So, actual callbacks should be
    written carefully.

    To hook dma setting changes, this function implements
    ide_dma_on() and ide_dma_off_quietly() which notifies protocl
    change and calls low level driver callback. __ide_dma_off()
    is renamed to ide_dma_off() for consistency. All dma on/off
    operations must be done by using these wrapper functions.

    10_blk_ide-add-fua-support.patch
    : add FUA support to IDE

    Add FUA support to IDE. IDE FUA support makes use of
    ->protocol_changed callback to correctly adjust FUA setting
    according to transfer protocol change.

    11_blk_add-barrier-doc.patch
    : I/O barrier documentation

    I/O barrier documentation

    [ End of patch descriptions ]

    --
    tejun

    -
    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-11-24 17:28    [W:0.028 / U:0.084 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site