All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Andreas Dilger <adilger@dilger.ca>
Cc: Allison Henderson <achender@linux.vnet.ibm.com>,
	linux-ext4@vger.kernel.org
Subject: Re: [Ext4 punch hole 1/5] Ext4 Punch Hole Support: Convert Blocks to Uninit Exts
Date: Thu, 3 Mar 2011 07:23:13 +1100	[thread overview]
Message-ID: <20110302202313.GA15097@dastard> (raw)
In-Reply-To: <02A57041-5FC1-419D-89D2-47D541616DD4@dilger.ca>

On Mon, Feb 28, 2011 at 11:42:48PM -0700, Andreas Dilger wrote:
> On 2011-02-28, at 8:08 PM, Allison Henderson wrote:
> > This first patch adds a function to convert a range of blocks
> > to an uninitialized extent.  This function will
> > be used to first convert the blocks to extents before
> > punching them out.
> 
> This was proposed as a separate function for FALLOCATE by Dave
> Chinner (based on XFS_IOC_ZERO_RANGE), so this is useful as a
> standalone function.

XFS_IOC_ZERO_RANGE converts a range to unwritten extents, not
uninitialised extents. An uninitialised extent is one that is
allocated but had not data written to it (i.e. contains stale data),
while an unwritten/preallocated extent is guaranteed to contain
zeros.  This may be just a terminology issue, but we should try to
use the same jargon across all filesystems...

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

  parent reply	other threads:[~2011-03-02 20:23 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-01  3:08 [Ext4 punch hole 1/5] Ext4 Punch Hole Support: Convert Blocks to Uninit Exts Allison Henderson
2011-03-01  5:42 ` Dave Young
2011-03-01  5:51   ` Allison Henderson
2011-03-01  6:42 ` Andreas Dilger
2011-03-01 20:47   ` Allison Henderson
2011-03-01 23:16   ` Mingming Cao
2011-03-02 20:23   ` Dave Chinner [this message]
2011-03-03  0:56     ` Andreas Dilger
2011-03-03 14:27     ` Theodore Tso
2011-03-01  9:30 ` Amir Goldstein
2011-03-01 20:48   ` Allison Henderson
2011-03-02  1:32   ` Mingming Cao
2011-03-02  7:54     ` Amir Goldstein
2011-03-01 11:09 ` Yongqiang Yang
2011-03-02  3:32   ` Yongqiang Yang
2011-03-01 12:25 ` Yongqiang Yang
2011-03-01 20:52   ` Allison Henderson
2011-03-01 13:22 ` Yongqiang Yang
2011-03-10  5:40 ` Dave Young
  -- strict thread matches above, loose matches on Subject: below --
2011-03-15 21:17 Allison Henderson
2011-03-15 21:41 Allison Henderson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110302202313.GA15097@dastard \
    --to=david@fromorbit.com \
    --cc=achender@linux.vnet.ibm.com \
    --cc=adilger@dilger.ca \
    --cc=linux-ext4@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.