linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Hugh Dickins <hugh.dickins@tiscali.co.uk>
To: Nitin Gupta <ngupta@vflare.org>
Cc: Matthew Wilcox <willy@linux.intel.com>,
	Ingo Molnar <mingo@elte.hu>,
	Peter Zijlstra <peterz@infradead.org>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [PATCH] swap: send callback when swap slot is freed
Date: Thu, 13 Aug 2009 18:45:18 +0100 (BST)	[thread overview]
Message-ID: <Pine.LNX.4.64.0908131836270.14631@sister.anvils> (raw)
In-Reply-To: <4A837AAF.4050103@vflare.org>

On Thu, 13 Aug 2009, Nitin Gupta wrote:
> On 08/13/2009 04:18 AM, Hugh Dickins wrote:
> 
> > But fundamentally, though I can see how this cutdown communication
> > path is useful to compcache, I'd much rather deal with it by the more
> > general discard route if we can.
> 
> I tried this too -- make discard bio request as soon as a swap slot becomes
> free (I can send details if you want). However, I could not get it to work.

I'll send you an updated version of what I experimented with eight
months ago: but changes in the swap_map count handling since then
mean that it might need some subtle adjustments - I'll need to go
over it carefully and retest before sending you.

(But that won't be a waste of my time: I shall soon need to try
that experiment again myself, and I do need to examine those
intervening swap_map count changes more closely.)

> Also, allocating bio to issue discard I/O request looks like a complete
> artifact in compcache case.

Yes, I do understand that feeling.

Hugh

--
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>

  parent reply	other threads:[~2009-08-13 17:45 UTC|newest]

Thread overview: 91+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12 14:37 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 [this message]
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
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
  -- strict thread matches above, loose matches on Subject: below --
2009-08-12 14:37 [PATCH] swap: send callback when swap slot is freed Nitin Gupta

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=Pine.LNX.4.64.0908131836270.14631@sister.anvils \
    --to=hugh.dickins@tiscali.co.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --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