linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Con Kolivas <kernel@kolivas.org>
To: Rik van Riel <riel@redhat.com>, ck list <ck@vds.kolivas.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	linux-mm <linux-mm@kvack.org>
Subject: Re: [PATCH] free swap space of (re)activated pages
Date: Sat, 3 Mar 2007 14:04:20 +1100	[thread overview]
Message-ID: <200703031404.21064.kernel@kolivas.org> (raw)
In-Reply-To: <45E8CFD8.7050808@redhat.com>

On Saturday 03 March 2007 12:31, Rik van Riel wrote:
> Andrew Morton wrote:
> > On Fri, 02 Mar 2007 15:31:19 -0500
> >
> > Rik van Riel <riel@redhat.com> wrote:
> >> the attached patch frees the swap space of already resident pages
> >> when swap space starts getting tight, instead of only freeing up
> >> the swap space taken up by newly swapped in pages.
> >>
> >> This should result in the swap space of pages that remain resident
> >> in memory being freed, allowing kswapd more chances to actually swap
> >> a page out (instead of rotating it back onto the active list).
> >
> > Fair enough.   How do we work out if this helps things?
>
> I suspect it should mostly help on desktop systems that slowly
> fill up (and run out of) swap.  I'm not sure how to create that
> synthetically.

Ooh you have a vm patch that helps swap on the desktop! I can help you here 
with my experience from swap prefetch.

1. Get it reviewed and have noone show any evidence it harms
2. Find hundreds of users who can testify it helps
3. Find a way of quantifying it.
4. ...
5. Merge into mainline.


There, that should get you as far as 4. 

I haven't figured out what 4 is yet. I believe it may be goto 1;

-- 
-ck

--
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:[~2007-03-03  3:04 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-02 20:31 Rik van Riel
2007-03-03  1:18 ` Andrew Morton
2007-03-03  1:31   ` Rik van Riel
2007-03-03  3:04     ` Con Kolivas [this message]

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=200703031404.21064.kernel@kolivas.org \
    --to=kernel@kolivas.org \
    --cc=akpm@linux-foundation.org \
    --cc=ck@vds.kolivas.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=riel@redhat.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