Date: Mon, 15 Mar 2010 12:08:01 -0400
From: Christoph Hellwig <hch@infradead.org>
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com
Subject: Re: [RFC] Delayed logging
Message-ID: <20100315160801.GB10406@infradead.org>
References: <20100315043000.GK4732@dastard>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20100315043000.GK4732@dastard>
User-Agent: Mutt/1.5.19 (2009-01-05)

Some comments on the patches:

 - "xfs: cluster fsync transaction" seems like something that can stand
   on it's own and go into the tree now.  Despite the comments it
   currently only clusters in fsync and not write_inode which might
   lead to higher benefits, btw.

 - the busy extent tracking might be worth to be reordered before the
   delayed logging series.  In itself it might also want some reordering
   as there's a lot churn in the patches there.  Making
   "XFS: Simplify transaction busy extent tracking" first in that
   subseries might help quite a bit to reduce that churn

 - The actual CIL implementation seems to be split into too small
   patches IMHO.  E.g. "xfs: extend the log item to support delayed
   logging" and "xfs: Introduce the Committed Item List" are two
   sides of the same coin and splitting it might not make too much
   sense.