Messages in this thread Patch in this message | | | Date | Thu, 12 Apr 2001 20:18:24 +0200 (MET DST) | From | Szabolcs Szakacsits <> | Subject | Re: scheduler went mad? |
| |
On Thu, 12 Apr 2001, Marcelo Tosatti wrote:
> This patch is broken, ignore it. > Just removing wakeup_bdflush() is indeed correct. > We already wakeup bdflush at try_to_free_buffers() anyway.
I still feel a bit unconfortable about processes looping forever in __alloc_pages and because of this oom_killer also can't be moved to page fault handler where I think its place should be. I'm using the patch below.
Szaka
--- mm/page_alloc.c.orig Sat Mar 31 19:07:22 2001 +++ mm/page_alloc.c Mon Apr 2 21:05:31 2001 @@ -453,8 +453,12 @@ */ if (gfp_mask & __GFP_WAIT) { memory_pressure++; - try_to_free_pages(gfp_mask); - wakeup_bdflush(0); + if (!try_to_free_pages(gfp_mask)); + return NULL; goto try_again; } }
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |