Re: [patch] 2.2.15-pre3 kswapd fix

From: Rik van Riel (riel@nl.linux.org)
Date: Thu Jan 20 2000 - 21:29:23 EST

On Fri, 21 Jan 2000, Andrea Arcangeli wrote:
> On Fri, 21 Jan 2000, Rik van Riel wrote:
>
> >About Andrea's freepages.low vs. freepages.min problem,
>
> It's not a problem. It's just that low is unused and you are now making
> min unused. You are simply increasing the min value.

More than that. Min is supposed to be the absolute boundary
below which nothing can allocate memory (except for ATOMIC
allocations). It's not with the old code, that's broken.

> >+++ linux-2.2.15-pre3/mm/vmscan.c Fri Jan 21 02:46:42 2000
> >@@ -485,18 +485,16 @@
> > * the processes needing more memory will wake us
> > * up on a more timely basis.
> > */
> >- interruptible_sleep_on_timeout(&kswapd_wait, HZ);
> > while (nr_free_pages < freepages.high)
> > {
> >- if (do_try_to_free_pages(GFP_KSWAPD))
> >- {
> >- if (tsk->need_resched)
> >- schedule();
> >- continue;
> >- }
> >- tsk->state = TASK_INTERRUPTIBLE;
> >- schedule_timeout(10*HZ);
> >+ if (!do_try_to_free_pages(GFP_KSWAPD))
> >+ break;
> >+ if (tsk->need_resched)
> >+ schedule();
> > }
> >+ run_task_queue(&tq_disk);
> >+ tsk->state = TASK_INTERRUPTIBLE;
> >+ schedule_timeout(HZ);
>
> How do you get a wakeup now? :) now it's pure too slow polling.

I copied this from the 2.3 code in the expectation that
schedule_timeout(HZ); is interruptible. If it's not, then
we've just found a 2.3 bug :)

> > wake_up_interruptible(&kswapd_wait);
> >- if (gfp_mask & __GFP_WAIT)
> >+ if ((gfp_mask & __GFP_WAIT) && (nr_free_pages < (freepages.low - 4)))
>
> Again treshing_mem heuristic broken...

Please explain to me why this is broken? I've carefully
explained why this makes sense and you haven't given
any practical explanation on your point of view...

regards,

Rik 

--
The Internet is not a network of computers. It is a network
of people. That is its real strength.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/

This archive was generated by hypermail 2b29 : Sun Jan 23 2000 - 21:00:24 EST