On Sat 26-03-11 09:55:58, Dave Chinner wrote: > On Fri, Mar 25, 2011 at 10:22:53PM +0800, Wu Fengguang wrote: > > It just happens to inherit the old *congestion* names, and the upper > > layer now actually hardly care about the congestion state. > > > > > Secondly, the big problem that causes the lumpiness is that we only > > > send commits when we reach at large threshold of unstable pages. > > > Because most servers tend to cache large writes in RAM, > > > the server might have a long commit latency because it has to write > > > hundred of MB of data to disk to complete the commit. > > > > > > IOWs, the client sends the commit only when it really needs the > > > pages the be cleaned, and then we have the latency of the server > > > write before it responds that they are clean. Hence commits can take > > > a long time to complete and mark pages clean on the client side. > > > > That's the point. That's why I add the following patches to limit the > > NFS commit size: > > > > [PATCH 10/27] nfs: limit the commit size to reduce fluctuations > > [PATCH 11/27] nfs: limit the commit range > > They don't solve the exclusion problem that is the root cause of the > burstiness. They do reduce the impact of it, but only in cases where > the server isn't that busy... Well, at least the first patch results in sending commits earlier for smaller amounts of data so that is principially what we want, isn't it? Maybe we could make NFS client trigger the commit on it's own when enough stable pages accumulate (and not depend on flusher thread to call ->write_inode) to make things more fluent. But that's about it and Irix did something like that if I understood your explanation correctly. > > > A solution that IRIX used for this problem was the concept of a > > > background commit. While doing writeback on an inode, if it sent > > > more than than a certain threshold of data (typically in the range > > > of 0.5-2s worth of data) to the server without a commit being > > > issued, it would send an _asynchronous_ commit with the current dirty > > > range to the server. That way the server starts writing the data > > > before it hits dirty thresholds (i.e. prevents GBs of dirty data > > > being cached on the server so commit lantecy is kept low). > > > > > > When the background commit completes the NFS client can then convert > > > pages in the commit range to clean. Hence we keep the number of > > > unstable pages under control without needing to wait for a certain > > > number of unstable pages to build up before commits are triggered. > > > This allows the process of writing dirty pages to clean > > > unstable pages at roughly the same rate as the write rate without > > > needing any magic thresholds to be configured.... > > > > That's a good approach. In linux, by limiting the commit size, the NFS > > flusher should roughly achieve the same effect. > > Not really. It's still threshold triggered, it's still synchronous > and hence will still have problems with commit latency on slow or > very busy servers. That is, it may work ok when you are the only > client writing to the server, but when 1500 other clients are also > writing to the server it won't have the desired effect. It isn't synchronous. We don't wait for commit in WB_SYNC_NONE mode if I'm reading the code right. It's only synchronous in the sense that pages are really clean only after the commit finishes but that's not the problem you are pointing to I believe. > > However there is another problem. Look at the below graph. Even though > > the commits are sent to NFS server in relatively small size and evenly > > distributed in time (the green points), the commit COMPLETION events > > from the server are observed to be pretty bumpy over time (the blue > > points sitting on the red lines). This may not be easily fixable.. So > > we still have to live with bumpy NFS commit completions... > > Right. The load on the server will ultimately determine the commit > latency, and that can _never_ be controlled by the client. We just > have to live with it and design the writeback path to prevent > commits from blocking writes in as many situations as possible. The question is how hard should we try. Here I believe Fengguang's patches can offer more then my approach because he throttles processes based on estimated bandwidth so occasional hiccups of the server are more "smoothed out". If we send commits early enough, hiccups matter less but still it's just a matter of how big they are... Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
