From owner-svn-src-user@FreeBSD.ORG Mon May 27 16:23:49 2013 Return-Path: Delivered-To: svn-src-user@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 626C92CC; Mon, 27 May 2013 16:23:49 +0000 (UTC) (envelope-from attilio@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) by mx1.freebsd.org (Postfix) with ESMTP id 3AB57D38; Mon, 27 May 2013 16:23:49 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.6/8.14.6) with ESMTP id r4RGNnHA064834; Mon, 27 May 2013 16:23:49 GMT (envelope-from attilio@svn.freebsd.org) Received: (from attilio@localhost) by svn.freebsd.org (8.14.6/8.14.5/Submit) id r4RGNmOn064832; Mon, 27 May 2013 16:23:48 GMT (envelope-from attilio@svn.freebsd.org) Message-Id: <201305271623.r4RGNmOn064832@svn.freebsd.org> From: Attilio Rao Date: Mon, 27 May 2013 16:23:48 +0000 (UTC) To: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: svn commit: r251031 - in user/attilio/vmobj-readlock/sys: fs/tmpfs kern X-SVN-Group: user MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-user@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the experimental " user" src tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 May 2013 16:23:49 -0000 Author: attilio Date: Mon May 27 16:23:48 2013 New Revision: 251031 URL: http://svnweb.freebsd.org/changeset/base/251031 Log: Use vm_page_sleep_if_busy() in 2 places where it is not used yet. Sponsored by: EMC / Isilon storage division Modified: user/attilio/vmobj-readlock/sys/fs/tmpfs/tmpfs_subr.c user/attilio/vmobj-readlock/sys/kern/uipc_shm.c Modified: user/attilio/vmobj-readlock/sys/fs/tmpfs/tmpfs_subr.c ============================================================================== --- user/attilio/vmobj-readlock/sys/fs/tmpfs/tmpfs_subr.c Mon May 27 16:16:28 2013 (r251030) +++ user/attilio/vmobj-readlock/sys/fs/tmpfs/tmpfs_subr.c Mon May 27 16:23:48 2013 (r251031) @@ -1329,14 +1329,8 @@ tmpfs_reg_resize(struct vnode *vp, off_t retry: m = vm_page_lookup(uobj, idx); if (m != NULL) { - if ((m->oflags & VPO_BUSY) != 0 || - m->busy != 0) { - vm_page_lock(m); - VM_OBJECT_WUNLOCK(uobj); - vm_page_sleep(m, "tmfssz"); - VM_OBJECT_WLOCK(uobj); + if (vm_page_sleep_if_busy(m, "tmfssz")) goto retry; - } MPASS(m->valid == VM_PAGE_BITS_ALL); } else if (vm_pager_has_page(uobj, idx, NULL, NULL)) { m = vm_page_alloc(uobj, idx, VM_ALLOC_NORMAL); Modified: user/attilio/vmobj-readlock/sys/kern/uipc_shm.c ============================================================================== --- user/attilio/vmobj-readlock/sys/kern/uipc_shm.c Mon May 27 16:16:28 2013 (r251030) +++ user/attilio/vmobj-readlock/sys/kern/uipc_shm.c Mon May 27 16:23:48 2013 (r251031) @@ -280,16 +280,9 @@ shm_dotruncate(struct shmfd *shmfd, off_ idx = OFF_TO_IDX(length); retry: m = vm_page_lookup(object, idx); - if (m != NULL) { - if ((m->oflags & VPO_BUSY) != 0 || - m->busy != 0) { - vm_page_lock(m); - VM_OBJECT_WUNLOCK(object); - vm_page_sleep(m, "shmtrc"); - VM_OBJECT_WLOCK(object); - goto retry; - } - } else if (vm_pager_has_page(object, idx, NULL, NULL)) { + if (m != NULL && vm_page_sleep_if_busy(m, "shmtrc")) + goto retry; + else if (vm_pager_has_page(object, idx, NULL, NULL)) { m = vm_page_alloc(object, idx, VM_ALLOC_NORMAL); if (m == NULL) { VM_OBJECT_WUNLOCK(object);