Remove page queues locking from all sf_buf_mext()-like functions. The page

lock now suffices.

Fix a couple nearby style violations.

3 files changed, 2 insertions, 13 deletions

diff --git a/sys/kern/uipc_cow.c b/sys/kern/uipc_cow.c
index bc9a56a..853b70b 100644
--- a/sys/kern/uipc_cow.c
+++ b/sys/kern/uipc_cow.c
@@ -81,7 +81,6 @@ socow_iodone(void *addr, void *args)
 	sf_buf_free(sf);
 	/* remove COW mapping  */
 	vm_page_lock(pp);
-	vm_page_lock_queues();
 	vm_page_cowclear(pp);
 	vm_page_unwire(pp, 0);
 	/*
@@ -91,7 +90,6 @@ socow_iodone(void *addr, void *args)
 	 */
 	if (pp->wire_count == 0 && pp->object == NULL)
 		vm_page_free(pp);
-	vm_page_unlock_queues();
 	vm_page_unlock(pp);
 	socow_stats.iodone++;
 }
@@ -147,9 +145,8 @@ socow_setup(struct mbuf *m0, struct uio *uio)
 	 * Allocate an sf buf
 	 */
 	sf = sf_buf_alloc(pp, SFB_CATCH);
-	if (!sf) {
+	if (sf == NULL) {
 		vm_page_lock(pp);
-		vm_page_lock_queues();
 		vm_page_cowclear(pp);
 		vm_page_unwire(pp, 0);
 		/*
@@ -159,7 +156,6 @@ socow_setup(struct mbuf *m0, struct uio *uio)
 		 */
 		if (pp->wire_count == 0 && pp->object == NULL)
 			vm_page_free(pp);
-		vm_page_unlock_queues();
 		vm_page_unlock(pp);
 		socow_stats.fail_sf_buf++;
 		return(0);
diff --git a/sys/kern/uipc_syscalls.c b/sys/kern/uipc_syscalls.c
index dd504f8..adcb852 100644
--- a/sys/kern/uipc_syscalls.c
+++ b/sys/kern/uipc_syscalls.c
@@ -1716,7 +1716,6 @@ sf_buf_mext(void *addr, void *args)
 	m = sf_buf_page(args);
 	sf_buf_free(args);
 	vm_page_lock(m);
-	vm_page_lock_queues();
 	vm_page_unwire(m, 0);
 	/*
 	 * Check for the object going away on us. This can
@@ -1725,7 +1724,6 @@ sf_buf_mext(void *addr, void *args)
 	 */
 	if (m->wire_count == 0 && m->object == NULL)
 		vm_page_free(m);
-	vm_page_unlock_queues();
 	vm_page_unlock(m);
 	if (addr == NULL)
 		return;
@@ -2111,7 +2109,6 @@ retry_space:
 			}
 			if (error) {
 				vm_page_lock(pg);
-				vm_page_lock_queues();
 				vm_page_unwire(pg, 0);
 				/*
 				 * See if anyone else might know about
@@ -2120,10 +2117,8 @@ retry_space:
 				 */
 				if (pg->wire_count == 0 && pg->valid == 0 &&
 				    pg->busy == 0 && !(pg->oflags & VPO_BUSY) &&
-				    pg->hold_count == 0) {
+				    pg->hold_count == 0)
 					vm_page_free(pg);
-				}
-				vm_page_unlock_queues();
 				vm_page_unlock(pg);
 				VM_OBJECT_UNLOCK(obj);
 				if (error == EAGAIN)
diff --git a/sys/net/bpf_zerocopy.c b/sys/net/bpf_zerocopy.c
index 5629093..91efd64 100644
--- a/sys/net/bpf_zerocopy.c
+++ b/sys/net/bpf_zerocopy.c
@@ -113,11 +113,9 @@ zbuf_page_free(vm_page_t pp)
 {
 
 	vm_page_lock(pp);
-	vm_page_lock_queues();
 	vm_page_unwire(pp, 0);
 	if (pp->wire_count == 0 && pp->object == NULL)
 		vm_page_free(pp);
-	vm_page_unlock_queues();
 	vm_page_unlock(pp);
 }