From owner-cvs-all@FreeBSD.ORG Sat Mar 17 18:18:09 2007 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 73E5F16A404; Sat, 17 Mar 2007 18:18:09 +0000 (UTC) (envelope-from jeff@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [69.147.83.41]) by mx1.freebsd.org (Postfix) with ESMTP id 63B0513C480; Sat, 17 Mar 2007 18:18:09 +0000 (UTC) (envelope-from jeff@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.8/8.13.8) with ESMTP id l2HII9tb008842; Sat, 17 Mar 2007 18:18:09 GMT (envelope-from jeff@repoman.freebsd.org) Received: (from jeff@localhost) by repoman.freebsd.org (8.13.8/8.13.8/Submit) id l2HII929008841; Sat, 17 Mar 2007 18:18:09 GMT (envelope-from jeff) Message-Id: <200703171818.l2HII929008841@repoman.freebsd.org> From: Jeff Roberson Date: Sat, 17 Mar 2007 18:18:09 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Cc: Subject: cvs commit: src/sys/nfsserver nfs.h nfs_serv.c nfs_srvcache.c nfs_srvsock.c nfs_srvsubs.c nfs_syscalls.c nfsm_subs.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 17 Mar 2007 18:18:09 -0000 jeff 2007-03-17 18:18:09 UTC FreeBSD src repository Modified files: sys/nfsserver nfs.h nfs_serv.c nfs_srvcache.c nfs_srvsock.c nfs_srvsubs.c nfs_syscalls.c nfsm_subs.h Log: - Turn all explicit giant acquires into conditional VFS_LOCK_GIANTs. Only ops which used namei still remained. - Implement a scheme for reducing the overhead of tracking which vops require giant by constantly reducing the number of recursive giant acquires to one, leaving us with only one vfslocked variable. - Remove all NFSD lock acquisition and release from the individual nfs ops. Careful examination has shown that they are not required. This greatly simplifies the code. Sponsored by: Isilon Systems, Inc. Discussed with: rwatson Tested by: kkenn Approved by: re Revision Changes Path 1.82 +1 -4 src/sys/nfsserver/nfs.h 1.171 +187 -554 src/sys/nfsserver/nfs_serv.c 1.44 +2 -0 src/sys/nfsserver/nfs_srvcache.c 1.102 +0 -10 src/sys/nfsserver/nfs_srvsock.c 1.146 +42 -71 src/sys/nfsserver/nfs_srvsubs.c 1.112 +2 -0 src/sys/nfsserver/nfs_syscalls.c 1.39 +2 -5 src/sys/nfsserver/nfsm_subs.h