Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 May 2010 11:53:40 +0000 (UTC)
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r207846 - head/sys/vm
Message-ID:  <201005101153.o4ABregP043975@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Mon May 10 11:53:40 2010
New Revision: 207846
URL: http://svn.freebsd.org/changeset/base/207846

Log:
  Continue cleaning the queue instead of moving to the next queue or
  bailing out if acquisition of page lock caused page position in the
  queue to change.
  
  Pointed out by:	alc

Modified:
  head/sys/vm/vm_contig.c

Modified: head/sys/vm/vm_contig.c
==============================================================================
--- head/sys/vm/vm_contig.c	Mon May 10 11:50:26 2010	(r207845)
+++ head/sys/vm/vm_contig.c	Mon May 10 11:53:40 2010	(r207846)
@@ -168,10 +168,8 @@ vm_contig_launder(int queue)
 		if ((m->flags & PG_MARKER) != 0)
 			continue;
 
-		if (!vm_pageout_page_lock(m, &next)) {
-			vm_page_unlock(m);
-			return (FALSE);
-		}
+		if (!vm_pageout_page_lock(m, &next))
+			continue;
 		KASSERT(VM_PAGE_INQUEUE2(m, queue),
 		    ("vm_contig_launder: page %p's queue is not %d", m, queue));
 		error = vm_contig_launder_page(m, &next);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201005101153.o4ABregP043975>