Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 23 Aug 2013 14:50:04 +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: r254715 - head/sys/kern
Message-ID:  <201308231450.r7NEo4iC098937@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Fri Aug 23 14:50:03 2013
New Revision: 254715
URL: http://svnweb.freebsd.org/changeset/base/254715

Log:
  Since the 253927, which removed the soft busy call for the sf page, it
  does not make sense to wait for the soft busy state of the page to
  drain.  The vm object lock is dropped immediately after, so the result
  of the wait is invalidated.
  
  It might make sense to not wait for the hard busy state as well,
  esp. for the fully valid page, but this is postponed for now.
  
  Reviewed by:	alc
  Tested by:	pho
  Sponsored by:	The FreeBSD Foundation

Modified:
  head/sys/kern/uipc_syscalls.c

Modified: head/sys/kern/uipc_syscalls.c
==============================================================================
--- head/sys/kern/uipc_syscalls.c	Fri Aug 23 14:43:32 2013	(r254714)
+++ head/sys/kern/uipc_syscalls.c	Fri Aug 23 14:50:03 2013	(r254715)
@@ -2230,7 +2230,8 @@ retry_space:
 			pindex = OFF_TO_IDX(off);
 			VM_OBJECT_WLOCK(obj);
 			pg = vm_page_grab(obj, pindex, VM_ALLOC_NOBUSY |
-			    VM_ALLOC_NORMAL | VM_ALLOC_WIRED);
+			    VM_ALLOC_IGN_SBUSY | VM_ALLOC_NORMAL |
+			    VM_ALLOC_WIRED);
 
 			/*
 			 * Check if page is valid for what we need,



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