lkml.org 
[lkml]   [2010]   [Nov]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v6 0/4] ext4: Coordinate data-only flush requests sent by fsync
    Hello,

    On 11/30/2010 01:39 AM, Neil Brown wrote:
    > I haven't seen any of the preceding discussion do I might be missing
    > something important, but this seems needlessly complex and intrusive.
    > In particular, I don't like adding code to md to propagate these timings up
    > to the fs, and I don't the arbitrary '2ms' number.
    >
    > Would it not be sufficient to simply gather flushes while a flush is pending.
    > i.e
    > - if no flush is pending, set the 'flush pending' flag, submit a flush,
    > then clear the flag.
    > - if a flush is pending, then wait for it to complete, and then submit a
    > single flush on behalf of all pending flushes.

    Heh, I was about to suggest exactly the same thing. Unless the delay
    is gonna be multiple times longer than avg flush time, I don't think
    the difference between the above scheme and the one w/ preemptive
    delay would be anything significant especially now that the cost of
    flush is much lower. Also, as Neil pointed out in another message,
    the above scheme will result in lower latency for flushes issued while
    no flush is in progress.

    IMO, this kind of optimization is gonna make noticeable difference
    only when there are a lot of simulatenous fsyncs, in which case the
    above would behave in mostly identical way with the more elaborate
    timer based one anyway.

    Thanks.

    --
    tejun


    \
     
     \ /
      Last update: 2010-11-30 14:49    [W:3.297 / U:0.128 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site