vm_physseg_find and VM_PAGE_TO_PHYS are both called many times in your

average arch port. They are also inline. This does not help, de-inline them.

shaves about 1k on i386 and amd64 bsd.mp. Probably similar amounts of
most architectures.

"no issue" beck@ "Nuke nuke nuke... make them functions" weingart@ "this
is good" art@

1 files changed, 3 insertions, 99 deletions

diff --git a/sys/uvm/uvm_page.h b/sys/uvm/uvm_page.h
index 5fbd32e8d2e..c60caf0fabb 100644
--- a/sys/uvm/uvm_page.h
+++ b/sys/uvm/uvm_page.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: uvm_page.h,v 1.27 2009/01/20 20:20:51 ariane Exp $	*/
+/*	$OpenBSD: uvm_page.h,v 1.28 2009/03/24 16:29:42 oga Exp $	*/
 /*	$NetBSD: uvm_page.h,v 1.19 2000/12/28 08:24:55 chs Exp $	*/
 
 /* 
@@ -273,8 +273,8 @@ PAGE_INLINE void uvm_pagezero(struct vm_page *);
 
 PAGE_INLINE int uvm_page_lookup_freelist(struct vm_page *);
 
-static struct vm_page *PHYS_TO_VM_PAGE(paddr_t);
-static int vm_physseg_find(paddr_t, int *);
+struct vm_page	*PHYS_TO_VM_PAGE(paddr_t);
+int	vm_physseg_find(paddr_t, int *);
 
 /*
  * macros
@@ -292,102 +292,6 @@ static int vm_physseg_find(paddr_t, int *);
 
 #define VM_PAGE_TO_PHYS(entry)	((entry)->phys_addr)
 
-/*
- * when VM_PHYSSEG_MAX is 1, we can simplify these functions
- */
-
-/*
- * vm_physseg_find: find vm_physseg structure that belongs to a PA
- */
-static __inline int
-vm_physseg_find(pframe, offp)
-	paddr_t pframe;
-	int	*offp;
-{
-#if VM_PHYSSEG_MAX == 1
-
-	/* 'contig' case */
-	if (pframe >= vm_physmem[0].start && pframe < vm_physmem[0].end) {
-		if (offp)
-			*offp = pframe - vm_physmem[0].start;
-		return(0);
-	}
-	return(-1);
-
-#elif (VM_PHYSSEG_STRAT == VM_PSTRAT_BSEARCH)
-	/* binary search for it */
-	int	start, len, try;
-
-	/*
-	 * if try is too large (thus target is less than than try) we reduce
-	 * the length to trunc(len/2) [i.e. everything smaller than "try"]
-	 *
-	 * if the try is too small (thus target is greater than try) then
-	 * we set the new start to be (try + 1).   this means we need to
-	 * reduce the length to (round(len/2) - 1).
-	 *
-	 * note "adjust" below which takes advantage of the fact that
-	 *  (round(len/2) - 1) == trunc((len - 1) / 2)
-	 * for any value of len we may have
-	 */
-
-	for (start = 0, len = vm_nphysseg ; len != 0 ; len = len / 2) {
-		try = start + (len / 2);	/* try in the middle */
-
-		/* start past our try? */
-		if (pframe >= vm_physmem[try].start) {
-			/* was try correct? */
-			if (pframe < vm_physmem[try].end) {
-				if (offp)
-					*offp = pframe - vm_physmem[try].start;
-				return(try);            /* got it */
-			}
-			start = try + 1;	/* next time, start here */
-			len--;			/* "adjust" */
-		} else {
-			/*
-			 * pframe before try, just reduce length of
-			 * region, done in "for" loop
-			 */
-		}
-	}
-	return(-1);
-
-#else
-	/* linear search for it */
-	int	lcv;
-
-	for (lcv = 0; lcv < vm_nphysseg; lcv++) {
-		if (pframe >= vm_physmem[lcv].start &&
-		    pframe < vm_physmem[lcv].end) {
-			if (offp)
-				*offp = pframe - vm_physmem[lcv].start;
-			return(lcv);		   /* got it */
-		}
-	}
-	return(-1);
-
-#endif
-}
-
-/*
- * PHYS_TO_VM_PAGE: find vm_page for a PA.   used by MI code to get vm_pages
- * back from an I/O mapping (ugh!).   used in some MD code as well.
- */
-static __inline struct vm_page *
-PHYS_TO_VM_PAGE(pa)
-	paddr_t pa;
-{
-	paddr_t pf = atop(pa);
-	int	off;
-	int	psi;
-
-	psi = vm_physseg_find(pf, &off);
-	if (psi != -1)
-		return(&vm_physmem[psi].pgs[off]);
-	return(NULL);
-}
-
 #define VM_PAGE_IS_FREE(entry)  ((entry)->pg_flags & PQ_FREE)
 
 #endif /* _KERNEL */