All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Sandeen <sandeen@sandeen.net>
To: Dave Chinner <david@fromorbit.com>
Cc: Peter Niemayer <niemayer@isg.de>, xfs@oss.sgi.com
Subject: Re: XFS support for TRIM / blkdev_issue_discard?
Date: Sun, 19 Apr 2009 10:51:24 -0500	[thread overview]
Message-ID: <49EB487C.1020205@sandeen.net> (raw)
In-Reply-To: <20090419081927.GE16929@discord.disaster>

Dave Chinner wrote:
> On Tue, Apr 14, 2009 at 01:32:35PM -0500, Felix Blyakher wrote:
>> On Apr 14, 2009, at 9:44 AM, Peter Niemayer wrote:
>>
>>> Dear Felix Blyakher,
>> Hi Peter,
>>
>>> do you know whether there are plans to support the ATA TRIM command /
>>> blkdev_issue_discard() call in XFS for Linux anytime soon?
>> This topic was indeed brought up in xfs discussions.
>> Though, we don't have any definite plans on supporting it yet.
> 
> I did some initial work on tracking extents being freed, but then
> the TRIM request morphed into a "tell the largest free space around
> the area just freed" and that is much harder to do than simply to
> issue a "we just freed this bit" command.

FWIW, ext[34] is still doing it block by block.  At least for SSDs, I
think that's fine, isn't it?  For think provisioning it might be
different.  I  think (?) the simply block-by-block free is still
relevant, no?

-Eric

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  reply	other threads:[~2009-04-19 15:51 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-31  5:30 [GIT PULL] XFS update for 2.6.30 Felix Blyakher
2009-03-31  5:30 ` Felix Blyakher
2009-04-02 16:26 ` Felix Blyakher
2009-04-02 16:26   ` Felix Blyakher
2009-04-03 16:52   ` Linus Torvalds
2009-04-03 16:52     ` Linus Torvalds
2009-04-03 17:02     ` Linus Torvalds
2009-04-03 17:02       ` Linus Torvalds
2009-04-03 17:41       ` Felix Blyakher
2009-04-03 17:41         ` Felix Blyakher
2009-04-04  1:19         ` david
2009-04-04  1:19           ` david
2009-04-04  1:54           ` Felix Blyakher
2009-04-04  1:54             ` Felix Blyakher
2009-04-14 14:44 ` XFS support for TRIM / blkdev_issue_discard? Peter Niemayer
2009-04-14 14:44   ` Peter Niemayer
2009-04-14 18:32   ` Felix Blyakher
2009-04-15  9:43     ` Peter Niemayer
2009-04-15 11:58     ` Peter Niemayer
2009-04-17 16:09     ` Peter Niemayer
2009-04-17 16:50       ` Eric Sandeen
2009-04-19  8:19     ` Dave Chinner
2009-04-19 15:51       ` Eric Sandeen [this message]
2009-04-20 18:45         ` Peter Niemayer
2009-04-20 19:11           ` Martin K. Petersen

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=49EB487C.1020205@sandeen.net \
    --to=sandeen@sandeen.net \
    --cc=david@fromorbit.com \
    --cc=niemayer@isg.de \
    --cc=xfs@oss.sgi.com \
    /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.