linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Greg Freemyer <greg.freemyer@gmail.com>
To: Chris Worley <worleys@gmail.com>
Cc: Matthew Wilcox <matthew@wil.cx>, Mark Lord <liml@rtr.ca>,
	Bryan Donlan <bdonlan@gmail.com>,
	david@lang.hm, Markus Trippelsdorf <markus@trippelsdorf.de>,
	Matthew Wilcox <willy@linux.intel.com>,
	Hugh Dickins <hugh.dickins@tiscali.co.uk>,
	Nitin Gupta <ngupta@vflare.org>, Ingo Molnar <mingo@elte.hu>,
	Peter Zijlstra <peterz@infradead.org>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org,
	Linux RAID <linux-raid@vger.kernel.org>
Subject: Re: Discard support (was Re: [PATCH] swap: send callback when swap slot is freed)
Date: Fri, 14 Aug 2009 21:55:36 -0400	[thread overview]
Message-ID: <87f94c370908141855y44fe3c46xbfa155644a283ccb@mail.gmail.com> (raw)
In-Reply-To: <f3177b9e0908141738n5f99b85dx3de0f620180a4b46@mail.gmail.com>

On Fri, Aug 14, 2009 at 8:38 PM, Chris Worley<worleys@gmail.com> wrote:
> On Fri, Aug 14, 2009 at 6:30 PM, Greg Freemyer<greg.freemyer@gmail.com> wrote:
>> On Fri, Aug 14, 2009 at 8:19 PM, Chris Worley<worleys@gmail.com> wrote:
>>> On Fri, Aug 14, 2009 at 5:45 PM, Matthew Wilcox<matthew@wil.cx> wrote:
>>>> On Fri, Aug 14, 2009 at 05:21:32PM -0600, Chris Worley wrote:
>>>>> Sooner is better than waiting to coalesce.  The longer an LBA is
>>>>> inactive, the better for any management scheme.  If you wait until
>>>>> it's reused, you might as well forgo the advantages of TRIM/UNMAP.  If
>>>>> a the controller wants to coalesce, let it coalesce.
>>>>
>>>> I'm sorry, you're wrong.  There is a tradeoff point, and it's different
>>>> for each drive model.  Sending down a steady stream of tiny TRIMs is
>>>> going to give terrible performance.
>>>
>>> Sounds like you might be using junk for a device?
>>>
>>> For junk, a little coalescing may be warranted... like in the I/O
>>> schedular, but no more than 100usecs wait before posting, or then you
>>> effect high performing devices too.
>>>
>>> Chris
>>
>> Why?
>>
>> AIUI, on every write a high performing device allocates a new erase
>> block from its free lists, writes to it, and puts the now unused erase
>> block on the free list.
>
> So erase blocks are 512 bytes (if I write 512 bytes, an erase block is
> now freed)?  Not true.

Seriously, how do you  know?  Are you under NDA?

The write paper I read about typical SSD design described a partial
erase block write as:

Internal logic/micro-controller performs:

Read erase block, modify erase block, allocate new erase block, write
new erase block, free now unused old erase block, old erase block
added to a hardware erase queue the performs the actual erase in the
background at the relatively slow speed of multiple milliseconds..

The purpose of the trim/discard command being to allow the ssd to have
enough free erase blocks ready to go that the writes don't have to
stall while they wait for a erase block to pop out of the erase queue.

Greg

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2009-08-15  1:55 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12 14:37 [PATCH] swap: send callback when swap slot is freed Nitin Gupta
2009-08-12 22:48 ` Hugh Dickins
2009-08-13  2:30   ` Nitin Gupta
2009-08-13  6:53     ` Peter Zijlstra
2009-08-13 14:44       ` Nitin Gupta
2009-08-13 17:45     ` Hugh Dickins
2009-08-13  2:41   ` Nitin Gupta
2009-08-13  5:05     ` compcache as a pre-swap area (was: [PATCH] swap: send callback when swap slot is freed) Al Boldi
2009-08-13 17:31       ` Nitin Gupta
2009-08-14  4:02         ` Al Boldi
2009-08-14  4:53           ` compcache as a pre-swap area Nitin Gupta
2009-08-14 15:49             ` Al Boldi
2009-08-15 11:00               ` Al Boldi
2009-08-13 15:13   ` Discard support (was Re: [PATCH] swap: send callback when swap slot is freed) Matthew Wilcox
2009-08-13 15:17     ` david
2009-08-13 15:26       ` Matthew Wilcox
2009-08-13 15:43     ` James Bottomley
2009-08-13 18:22       ` Ric Wheeler
2009-08-13 16:13     ` Nitin Gupta
2009-08-13 16:26     ` Markus Trippelsdorf
2009-08-13 16:33       ` david
2009-08-13 18:15         ` Greg Freemyer
2009-08-13 19:18           ` James Bottomley
2009-08-13 20:31             ` Richard Sharpe
2009-08-14 22:03             ` Mark Lord
2009-08-14 22:54               ` Greg Freemyer
2009-08-15 13:12                 ` Mark Lord
2009-08-13 20:44           ` david
2009-08-13 20:54             ` Bryan Donlan
2009-08-14 22:10               ` Mark Lord
2009-08-14 23:21                 ` Chris Worley
2009-08-14 23:45                   ` Matthew Wilcox
2009-08-15  0:19                     ` Chris Worley
2009-08-15  0:30                       ` Greg Freemyer
2009-08-15  0:38                         ` Chris Worley
2009-08-15  1:55                           ` Greg Freemyer [this message]
2009-08-15 13:20                           ` Mark Lord
2009-08-16 22:52                             ` Chris Worley
2009-08-17  2:03                               ` Mark Lord
2009-08-15 12:59                       ` James Bottomley
2009-08-15 13:22                         ` Mark Lord
2009-08-15 13:55                           ` James Bottomley
2009-08-15 17:39                             ` jim owens
2009-08-16 17:08                               ` Robert Hancock
2009-08-16 14:05                             ` Alan Cox
2009-08-16 14:16                               ` Mark Lord
2009-08-16 15:34                               ` Arjan van de Ven
2009-08-16 15:44                                 ` Theodore Tso
2009-08-16 17:28                                   ` Mark Lord
2009-08-16 17:37                                     ` Mark Lord
2009-08-17 16:30                                       ` Bill Davidsen
2009-08-17 16:56                                         ` jim owens
2009-08-17 17:14                                           ` Bill Davidsen
2009-08-17 17:37                                             ` jim owens
2009-08-16 17:37                                     ` Mark Lord
2009-08-16 15:52                                 ` James Bottomley
2009-08-16 16:32                                   ` Mark Lord
2009-08-16 18:07                                     ` James Bottomley
2009-08-16 18:19                                       ` Mark Lord
2009-08-16 18:24                                         ` James Bottomley
2009-08-17 16:37                                           ` Bill Davidsen
2009-08-17 17:08                                             ` Greg Freemyer
2009-08-17 17:19                                               ` James Bottomley
2009-08-17 18:16                                                 ` Ric Wheeler
2009-08-17 18:21                                                 ` Greg Freemyer
2009-08-17 19:18                                                   ` James Bottomley
2009-08-17 20:19                                                     ` Mark Lord
2009-08-17 20:28                                                       ` James Bottomley
2009-08-17 20:28                                               ` Mark Lord
2009-08-16 16:59                                   ` Christoph Hellwig
2009-08-17  4:24                                     ` Douglas Gilbert
2009-08-17 13:56                                     ` James Bottomley
2009-08-17 14:10                                       ` Matthew Wilcox
2009-08-17 19:12                                         ` Christoph Hellwig
2009-08-17 19:24                                           ` James Bottomley
2009-08-16 21:50                                   ` Discard support Roland Dreier
2009-08-16 22:06                                     ` Jeff Garzik
2009-08-16 22:13                                     ` Theodore Tso
2009-08-16 22:51                                       ` Mark Lord
2009-08-16 19:29                                 ` Discard support (was Re: [PATCH] swap: send callback when swap slot is freed) Alan Cox
2009-08-16 23:05                                   ` John Robinson
2009-08-17  2:05                                     ` Mark Lord
2009-08-13 21:28             ` Greg Freemyer
2009-08-13 22:20               ` Richard Sharpe
2009-08-14  0:19                 ` Greg Freemyer
     [not found]                   ` <46b8a8850908131758s781b07f6v2729483c0e50ae7a@mail.gmail.com>
2009-08-14 21:33                     ` Greg Freemyer
2009-08-14 21:56                       ` Discard support Roland Dreier
2009-08-14 22:10                         ` Greg Freemyer
2009-08-13 17:19     ` Discard support (was Re: [PATCH] swap: send callback when swap slot is freed) Hugh Dickins
2009-08-13 18:08     ` Douglas Gilbert

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=87f94c370908141855y44fe3c46xbfa155644a283ccb@mail.gmail.com \
    --to=greg.freemyer@gmail.com \
    --cc=bdonlan@gmail.com \
    --cc=david@lang.hm \
    --cc=hugh.dickins@tiscali.co.uk \
    --cc=liml@rtr.ca \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=markus@trippelsdorf.de \
    --cc=matthew@wil.cx \
    --cc=mingo@elte.hu \
    --cc=ngupta@vflare.org \
    --cc=peterz@infradead.org \
    --cc=willy@linux.intel.com \
    --cc=worleys@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox