linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: mel@skynet.ie (Mel Gorman)
To: Nicolas Mailhot <nicolas.mailhot@laposte.net>
Cc: Christoph Lameter <clameter@sgi.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	"bugme-daemon@kernel-bugs.osdl.org"
	<bugme-daemon@bugzilla.kernel.org>
Subject: Re: [Bug 8464] New: autoreconf: page allocation failure. order:2, mode:0x84020
Date: Sat, 12 May 2007 20:24:08 +0100	[thread overview]
Message-ID: <20070512192408.GA5769@skynet.ie> (raw)
In-Reply-To: <1178996310.6397.3.camel@rousalka.dyndns.org>

On (12/05/07 20:58), Nicolas Mailhot didst pronounce:
> Le samedi 12 mai 2007 a 20:09 +0200, Nicolas Mailhot a ecrit :
> > Le samedi 12 mai 2007 a 17:42 +0100, Mel Gorman a ecrit :
> > 
> > > order-2 (at least 19 pages but more are there) and higher pages were free
> > > and this was a NORMAL allocation. It should also be above watermarks so
> > > something screwy is happening
> > > 
> > > *peers suspiciously*
> > > 
> > > Can you try the following patch on top of the kswapd patch please? It is
> > > also available from http://www.csn.ul.ie/~mel/watermarks.patch
> > 
> > Ok, testing now
> 
> And this one failed testing too 

And same thing, you have suitable free memory. The last patch was
wrong because I forgot the !in_interrupt() part which was careless
and dumb.  Please try the following, again on top of the kswapd patch -
http://www.csn.ul.ie/~mel/watermarks-v2.patch

Thanks for all the testing, it's appreciated.

diff -rup -X /usr/src/patchset-0.6/bin//dontdiff linux-2.6.21-mm2-revertmd/mm/page_alloc.c linux-2.6.21-mm2-watermarks/mm/page_alloc.c
--- linux-2.6.21-mm2-revertmd/mm/page_alloc.c	2007-05-11 21:16:57.000000000 +0100
+++ linux-2.6.21-mm2-watermarks/mm/page_alloc.c	2007-05-12 20:20:19.000000000 +0100
@@ -1645,8 +1645,16 @@ nofail_alloc:
 	}
 
 	/* Atomic allocations - we can't balance anything */
-	if (!wait)
+	if (!wait) {
+
+		/* Attempt to allocate ignoring watermarks */
+		page = get_page_from_freelist(gfp_mask, order,
+					zonelist, ALLOC_NO_WATERMARKS);	
+		if (page)
+			goto got_pg;
+
 		goto nopage;
+	}
 
 	cond_resched();
 
-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2007-05-12 19:24 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200705102128.l4ALSI2A017437@fire-2.osdl.org>
2007-05-10 21:43 ` Andrew Morton
2007-05-10 21:49   ` Christoph Lameter
2007-05-10 22:06     ` Mel Gorman
2007-05-10 22:11       ` Christoph Lameter
2007-05-10 22:16         ` Mel Gorman
2007-05-10 22:27           ` Christoph Lameter
2007-05-10 22:44             ` Mel Gorman
2007-05-10 22:49               ` Christoph Lameter
2007-05-10 23:00                 ` Mel Gorman
2007-05-10 23:01                   ` Christoph Lameter
2007-05-11  5:56                     ` Nicolas Mailhot
2007-05-11  9:08                       ` Mel Gorman
2007-05-11 11:51                         ` Nicolas Mailhot
2007-05-11 17:38                           ` Mel Gorman
2007-05-11 17:45                             ` Nicolas Mailhot
2007-05-11 18:30                               ` Nicolas Mailhot
2007-05-11 20:36                                 ` Mel Gorman
2007-05-12  8:11                                   ` Nicolas Mailhot
2007-05-12 16:42                                     ` Mel Gorman
2007-05-12 18:09                                       ` Nicolas Mailhot
2007-05-12 18:58                                         ` Nicolas Mailhot
2007-05-12 19:24                                           ` Mel Gorman [this message]
2007-05-13  8:16                                             ` Nicolas Mailhot
2007-05-11 17:46                             ` Christoph Lameter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070512192408.GA5769@skynet.ie \
    --to=mel@skynet.ie \
    --cc=akpm@linux-foundation.org \
    --cc=bugme-daemon@bugzilla.kernel.org \
    --cc=clameter@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=nicolas.mailhot@laposte.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox