On Mon, 2 Oct 2000 ehrhardt@mathematik.uni-ulm.de wrote:
> On Mon, Oct 02, 2000 at 12:42:47AM -0300, Rik van Riel wrote:
> > --- linux-2.4.0-test9-pre7/fs/buffer.c.orig Sat Sep 30 18:09:18 2000
> > +++ linux-2.4.0-test9-pre7/fs/buffer.c Mon Oct 2 00:19:41 2000
> > @@ -706,7 +706,9 @@
> > static void refill_freelist(int size)
> > {
> > if (!grow_buffers(size)) {
> > - try_to_free_pages(GFP_BUFFER);
> > + wakeup_bdflush(1);
> > + current->policy |= SCHED_YIELD;
> > + schedule();
> > }
> > }
>
> This part looks strange! wakeup_bdflush will sleep if the
> parameter is not zero, i.e. we'll schedule twice. I doubt that
> this the intended behaviour?
Heh...
I copied back the old code because I saw some failures in
the try_to_free_pages() in this situation.
Maybe the person who wrote this code originally can comment
on this one ?
regards,
Rik
--
"What you're running that piece of shit Gnome?!?!"
-- Miguel de Icaza, UKUUG 2000
http://www.conectiva.com/ http://www.surriel.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 : Sat Oct 07 2000 - 21:00:10 EST