| To: | Sunil Mushran <sunil.mushran@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] xfstests 255: add a seek_data/seek_hole tester |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Wed, 29 Jun 2011 13:36:48 -0400 |
| Cc: | P?draig Brady <P@xxxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, Dave Chinner <david@xxxxxxxxxxxxx>, Josef Bacik <josef@xxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, viro@xxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx |
| In-reply-to: | <4E0B60DE.50908@xxxxxxxxxx> |
| References: | <1309275199-10801-1-git-send-email-josef@xxxxxxxxxx> <1309275199-10801-5-git-send-email-josef@xxxxxxxxxx> <20110629065306.GC1026@dastard> <20110629074021.GA26086@xxxxxxxxxxxxx> <4E0B019E.8080800@xxxxxxxxxxxxxx> <4E0B60DE.50908@xxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, Jun 29, 2011 at 10:29:02AM -0700, Sunil Mushran wrote: > I'm not too sure about that. Atleast not enabled by default. Most users > use cp to backup data. Not empty space. In this case, this empty extent > may not even be de-dupable. > > Frankly I'd be happier of cp started to exploited fallocate() to create larger > extents before copying data into them. Atleast for the large files. That's what delayed allocation is for. |
| Previous by Date: | Re: [PATCH] xfstests 255: add a seek_data/seek_hole tester, Sunil Mushran |
|---|---|
| Next by Date: | Re: [PATCH] xfstests 255: add a seek_data/seek_hole tester, Sunil Mushran |
| Previous by Thread: | Re: [PATCH] xfstests 255: add a seek_data/seek_hole tester, Sunil Mushran |
| Next by Thread: | Re: [PATCH] xfstests 255: add a seek_data/seek_hole tester, Sunil Mushran |
| Indexes: | [Date] [Thread] [Top] [All Lists] |