[lkml]   [2000]   [Sep]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: More on 2.2.18pre2aa2
On Tue, 12 Sep 2000, David S. Miller wrote:
> From: Rik van Riel <>
> 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);

Urghhh indeed. So /that's/ why drop_behind() didn't
seem to do its work ;)

Thanks for pointing out this obvious thinko. I hope
more people will take a look at my VM patch and try
to poke holes in it...

(until now, only 3 people seem to have bothered with
looking at the code in detail)


"What you're running that piece of shit Gnome?!?!"
-- Miguel de Icaza, UKUUG 2000

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:38    [from the cache]
©2003-2014 Jasper Spaans. hosted at Digital Ocean