From: Rajagopal Ananthanarayanan <ananth@sgi.com>
To: "Quintela Carreira Juan J." <quintela@vexeta.dc.fi.udc.es>
Cc: Linus Torvalds <torvalds@transmeta.com>,
Andrea Arcangeli <andrea@suse.de>,
Benjamin Redelings I <bredelin@ucla.edu>,
linux-mm@kvack.org
Subject: Re: [DATAPOINT] pre7-6 will not swap
Date: Mon, 08 May 2000 19:28:36 -0700 [thread overview]
Message-ID: <391777D4.86460218@sgi.com> (raw)
In-Reply-To: ytt66sov6a9.fsf@vexeta.dc.fi.udc.es
"Quintela Carreira Juan J." wrote:
>
> >>>>> "linus" == Linus Torvalds <torvalds@transmeta.com> writes:
>
> linus> in vmscan.c, and that seems to be quite well-behaved too (but if somebody
> linus> has the energy to test the two different versions, I'd absolutely love to
> linus> hear results..)
>
> Hi Linus,
> I have tested two versions of the patch (against vanilla
> pre7-6), the first was to remove the test altogether (I think this is
> from Rajagopal):
>
> --- pre7-6/mm/vmscan.c Fri May 5 23:58:56 2000
> +++ testing/mm/vmscan.c Mon May 8 23:30:52 2000
> @@ -114,8 +114,9 @@
> * Don't do any of the expensive stuff if
> * we're not really interested in this zone.
> */
> - if (!page->zone->zone_wake_kswapd)
> +/* if (!page->zone->zone_wake_kswapd)
> goto out_unlock;
> +*/
>
I'm having the same experience too. The one thing
that makes stuff better is not to look at the zone at all
in try_to_swap_out (as Juan points out above).
I'm trying to also see if we can do better in shrink_mmap().
Although my gprof statistics say that we can end-up spending
91% of the time skipping pages, I'm not able to comeup with
anything simple to make shrink_mmap behave better ... except
one change which makes swapping a lot less and shrink_mmap
a lot more agressive: don't skip pages based on zone's
high water mark if we are trying hard to free pages (my heuristic
was to stop skipping pages if priority in shrink_mmap was 3; YMMV).
I'm not entirely convinced that this is the right thing to do.
In all, I do think that try_to_swap_out shouldn't skip pages
based on zones. We have now evidence from 3 different "workloads"
in this direction --- my own dbench test, Juan's test above &
Benjamin's "gaming" workload.
--------------------------------------------------------------------------
Rajagopal Ananthanarayanan ("ananth")
Member Technical Staff, SGI.
--------------------------------------------------------------------------
--
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.eu.org/Linux-MM/
next prev parent reply other threads:[~2000-05-09 2:28 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <8evk0f$7jote$1@fido.engr.sgi.com>
2000-05-06 17:12 ` [DATAPOINT] pre7-6 will not swap Rajagopal Ananthanarayanan
2000-05-06 4:25 ` Benjamin Redelings I
2000-05-06 19:35 ` Linus Torvalds
2000-05-06 5:35 ` Benjamin Redelings I
2000-05-06 21:46 ` Rik van Riel
2000-05-06 22:24 ` Rajagopal Ananthanarayanan
2000-05-06 14:03 ` Benjamin Redelings I
2000-05-07 0:22 ` Rik van Riel
2000-05-07 2:23 ` Linus Torvalds
2000-05-07 17:40 ` Rik van Riel
2000-05-07 17:53 ` Linus Torvalds
2000-05-07 19:13 ` Rajagopal Ananthanarayanan
2000-05-07 19:30 ` Linus Torvalds
2000-05-08 20:40 ` gprof data for pre7-6 Rajagopal Ananthanarayanan
2000-05-09 1:52 ` [DATAPOINT] pre7-6 will not swap Quintela Carreira Juan J.
2000-05-09 2:28 ` Rajagopal Ananthanarayanan [this message]
2000-05-09 2:33 ` Linus Torvalds
2000-05-09 3:31 ` Rajagopal Ananthanarayanan
2000-05-09 15:56 ` [DATAPOINT] pre7-8 swaps with FREE mem? Benjamin Redelings I
2000-05-06 20:12 ` PG_referenced and lru_cache (cpu%) Roger Larsson
2000-05-06 18:31 ` Rik van Riel
2000-05-06 22:16 ` Roger Larsson
2000-05-05 8:07 [DATAPOINT] pre7-6 will not swap Benjamin Redelings I
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=391777D4.86460218@sgi.com \
--to=ananth@sgi.com \
--cc=andrea@suse.de \
--cc=bredelin@ucla.edu \
--cc=linux-mm@kvack.org \
--cc=quintela@vexeta.dc.fi.udc.es \
--cc=torvalds@transmeta.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.