Date: Tue, 12 Sep 2000 04:23:05 -0300 (BRST)
From: Rik van Riel <riel@conectiva.com.br>
I've just uploaded a new snapshot of my new VM for
2.4 to my home page, this version contains a
wakeup_kswapd() function (copied from wakeup_bdflush)
and should balance memory a bit better.
How can drop_behind() work properly?
You do not recompute the hash chain head for each decreasing 'index'
in the main while loop, and thus you search potentially the wrong hash
chain each time.
Thus, you need to change:
+ page = __find_page_nolock(mapping, index, *hash);
to something more like:
+ hash = page_hash(mapping, index);
+ page = __find_page_nolock(mapping, index, *hash);
and remove the now-spurious local variable initialization of
'hash' at the top of this function.
Also, why this?
+ page = NULL;
That's spurious, you set it on the next line, probably this is
from some older revision of this function :-)
Later,
David S. Miller
davem@redhat.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Fri Sep 15 2000 - 21:00:17 EST