2008-12-01 09:35:51

by Gene Heskett

[permalink] [raw]
Subject: Swappiness fix?

Greetings;

Has that patch someone pointed me to made it into -rc6? I have a bit under 9
hours uptime on a 4Gb machine, with a gimp session, email and web browsing,
and I see I'm 27 megs into swap. I do not have the patch now, a drive died.
However, I may still have the email that contained it, I'll check that.

That patch, IMO should be fast lane'd to make it to .28.

--
Cheers, Gene
"There are four boxes to be used in defense of liberty:
soap, ballot, jury, and ammo. Please use in that order."
-Ed Howdershelt (Author)
Neutrinos have bad breadth.


2008-12-01 09:44:27

by KOSAKI Motohiro

[permalink] [raw]
Subject: Re: Swappiness fix?

Hi

> Greetings;
>
> Has that patch someone pointed me to made it into -rc6? I have a bit under 9
> hours uptime on a 4Gb machine, with a gimp session, email and web browsing,
> and I see I'm 27 megs into swap. I do not have the patch now, a drive died.
> However, I may still have the email that contained it, I'll check that.
>
> That patch, IMO should be fast lane'd to make it to .28.

Could you please try to mmotm?
As far as I know, nobody give Tested-by to the fixing patch.

if you can test it, I'm very glad.
thanks.


2008-12-02 06:51:16

by Andrew Morton

[permalink] [raw]
Subject: Re: Swappiness fix?

On Mon, 01 Dec 2008 04:35:41 -0500 Gene Heskett <[email protected]> wrote:

> Greetings;
>
> Has that patch someone pointed me to made it into -rc6? I have a bit under 9
> hours uptime on a 4Gb machine, with a gimp session, email and web browsing,
> and I see I'm 27 megs into swap. I do not have the patch now, a drive died.
> However, I may still have the email that contained it, I'll check that.

This?

From: Rik van Riel <[email protected]>

Count the insertion of new pages in the statistics used to drive the
pageout scanning code. This should help the kernel quickly evict
streaming file IO.

We count on the fact that new file pages start on the inactive file LRU
and new anonymous pages start on the active anon list. This means
streaming file IO will increment the recent scanned file statistic, while
leaving the recent rotated file statistic alone, driving pageout scanning
to the file LRUs.

Pageout activity does its own list manipulation.

Signed-off-by: Rik van Riel <[email protected]>
Cc: KAMEZAWA Hiroyuki <[email protected]>
Cc: KOSAKI Motohiro <[email protected]>
Cc: Gene Heskett <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
---

mm/swap.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)

diff -puN mm/swap.c~vmscan-evict-streaming-io-first mm/swap.c
--- a/mm/swap.c~vmscan-evict-streaming-io-first
+++ a/mm/swap.c
@@ -445,6 +445,7 @@ void ____pagevec_lru_add(struct pagevec
for (i = 0; i < pagevec_count(pvec); i++) {
struct page *page = pvec->pages[i];
struct zone *pagezone = page_zone(page);
+ int file;

if (pagezone != zone) {
if (zone)
@@ -456,8 +457,12 @@ void ____pagevec_lru_add(struct pagevec
VM_BUG_ON(PageUnevictable(page));
VM_BUG_ON(PageLRU(page));
SetPageLRU(page);
- if (is_active_lru(lru))
+ file = is_file_lru(lru);
+ zone->recent_scanned[file]++;
+ if (is_active_lru(lru)) {
SetPageActive(page);
+ zone->recent_rotated[file]++;
+ }
add_page_to_lru_list(zone, page, lru);
}
if (zone)
_

> That patch, IMO should be fast lane'd to make it to .28.

I have it sitting here marked "awaits Gene testing" ;)