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>