svn commit: r212951 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

Andriy Gapon avg at FreeBSD.org
Tue Sep 21 05:58:46 UTC 2010


Author: avg
Date: Tue Sep 21 05:58:45 2010
New Revision: 212951
URL: http://svn.freebsd.org/changeset/base/212951

Log:
  zfs_map_page/zfs_unmap_page: do not use sched_pin() and SFB_CPUPRIVATE
  
  zfs_map_page/zfs_unmap_page are mostly called around potential I/O paths
  and it seems to be a not very good idea to do cpu pinning there.
  
  Suggested by:	kib
  MFC after:	2 weeks

Modified:
  head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c

Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c
==============================================================================
--- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c	Tue Sep 21 05:54:36 2010	(r212950)
+++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c	Tue Sep 21 05:58:45 2010	(r212951)
@@ -358,8 +358,7 @@ static caddr_t
 zfs_map_page(vm_page_t pp, struct sf_buf **sfp)
 {
 
-	sched_pin();
-	*sfp = sf_buf_alloc(pp, SFB_CPUPRIVATE);
+	*sfp = sf_buf_alloc(pp, 0);
 	return ((caddr_t)sf_buf_kva(*sfp));
 }
 
@@ -368,7 +367,6 @@ zfs_unmap_page(struct sf_buf *sf)
 {
 
 	sf_buf_free(sf);
-	sched_unpin();
 }
 
 


More information about the svn-src-all mailing list