[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120717071927.GA28058@gmail.com>
Date: Tue, 17 Jul 2012 15:19:27 +0800
From: Zheng Liu <gnehzuil.liu@...il.com>
To: Eric Sandeen <sandeen@...hat.com>
Cc: linux-ext4@...r.kernel.org, Zach Brown <zab@...bo.net>,
Andreas Dilger <adilger@...ger.ca>,
Zheng Liu <wenqing.lz@...bao.com>
Subject: Re: [PATCH v2] ext4: dynamical adjust the length of zero-out chunk
On Thu, Jul 12, 2012 at 09:49:38AM -0500, Eric Sandeen wrote:
> On 7/12/12 1:48 AM, Zheng Liu wrote:
> > From: Zheng Liu <wenqing.lz@...bao.com>
> >
> > Currently in ext4 the length of zero-out chunk is set to 7. But it is
> > too short so that it will cause a lot of fragmentation of extent when
> > we use fallocate to preallocate some uninitialized extents and the
> > workload frequently does some uninitialized extent conversions. Thus,
> > now we set it to 256 (1MB chunk), and put it into super block in order
> > to adjust it dynamically in sysfs.
>
> Does this in fact help the workload for which you wanted the non-flagged
> fallocate interface?
No, it almost doesn't provide any helps for that workload, but it quite
can reduce the fragmentation of extents in my test. So IMO it can bring
some benifits for ext4. :-)
>
> I'm a little wary of adding another user tunable; how will the user have
> any idea what value to use here?
>
> At any rate, something should also go into Documentation/filesystems/ext4.txt
> to explain the new tunable.
Agree. I will add this tunable parameter in ext4 doc if this patch can
be applied.
Regards,
Zheng
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists