linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Richard Sharpe <realrichardsharpe@gmail.com>
To: James Bottomley <James.Bottomley@hansenpartnership.com>
Cc: Greg Freemyer <greg.freemyer@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: Thu, 13 Aug 2009 13:31:15 -0700	[thread overview]
Message-ID: <46b8a8850908131331g5b40a9c8j17bd80c8ba13a55f@mail.gmail.com> (raw)
In-Reply-To: <1250191095.3901.116.camel@mulgrave.site>

On Thu, Aug 13, 2009 at 12:18 PM, James Bottomley
<James.Bottomley@hansenpartnership.com> wrote:
> Actually, I think, if we go in-kernel, the discard might be better tied
> into the block plugging mechanism.  The real test might be no
> outstanding commands and queue plugged, keep plugged and begin
> discarding.

I am very interested in this topic, as I have implemented UNMAP
support in SCST and scst_local.c and ib_srp.c for one SSD vendor, as
well as the block layer changes to have it work correctly (they were
minor changes and based on Matthew's original TRIM or UNMAP patch from
long ago). I believe that the performance was acceptable for them (I
will have to check).

I am also working on other, non-SSD, devices that are in a lower price
range than the large storage arrays where both DISCARD/UNMAP (and
WRITE SAME) would be useful in Linux. It also seems that Microsoft
supports TRIM in Windows 7 if you switch it on, although that really
only implies we should implement UNMAP support in our firmware and
hook it up to existing mechanisms.

I have logged internal enhancement bugs in bugzilla asking for both
TRIM and UNMAP/WRITE SAME support, and although one environment is
iSCSI in userland, and thus can be dealt with without support in the
Linux kernel, there are use cases where DISCARD/UNMAP support in the
Linux kernel would be useful.

I would be very willing to make the firmware changes needed in our
device to support UNMAP/WRITE SAME and to test changes to the Linux
kernel to support same.

I will go through this thread in more detail when I get back from my
trip to Australia, but if there are any GIT trees around with nascent
support in them I would love to know about them, as it will help my
internal efforts to get UNMAP/WRITE SAME support implemented as well.

--
Regards,
Richard Sharpe

--
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-13 20:31 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 [this message]
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
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-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 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=46b8a8850908131331g5b40a9c8j17bd80c8ba13a55f@mail.gmail.com \
    --to=realrichardsharpe@gmail.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=david@lang.hm \
    --cc=greg.freemyer@gmail.com \
    --cc=hugh.dickins@tiscali.co.uk \
    --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=mingo@elte.hu \
    --cc=ngupta@vflare.org \
    --cc=peterz@infradead.org \
    --cc=willy@linux.intel.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