From owner-cvs-all@FreeBSD.ORG Fri Jul 1 16:28:33 2005 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2B83116A41C; Fri, 1 Jul 2005 16:28:33 +0000 (GMT) (envelope-from ssouhlal@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id EEE0043D1D; Fri, 1 Jul 2005 16:28:32 +0000 (GMT) (envelope-from ssouhlal@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j61GSWxn094887; Fri, 1 Jul 2005 16:28:32 GMT (envelope-from ssouhlal@repoman.freebsd.org) Received: (from ssouhlal@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j61GSWWN094886; Fri, 1 Jul 2005 16:28:32 GMT (envelope-from ssouhlal) Message-Id: <200507011628.j61GSWWN094886@repoman.freebsd.org> From: Suleiman Souhlal Date: Fri, 1 Jul 2005 16:28:32 +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/cam/scsi scsi_target.c src/sys/kern init_main.c kern_event.c kern_fork.c sys_pipe.c tty.c uipc_socket.c uipc_socket2.c vfs_aio.c vfs_subr.c src/sys/net bpf.c if.c src/sys/sys event.h mount.h vnode.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: Fri, 01 Jul 2005 16:28:33 -0000 ssouhlal 2005-07-01 16:28:32 UTC FreeBSD src repository Modified files: sys/cam/scsi scsi_target.c sys/kern init_main.c kern_event.c kern_fork.c sys_pipe.c tty.c uipc_socket.c uipc_socket2.c vfs_aio.c vfs_subr.c sys/net bpf.c if.c sys/sys event.h mount.h vnode.h Log: Fix the recent panics/LORs/hangs created by my kqueue commit by: - Introducing the possibility of using locks different than mutexes for the knlist locking. In order to do this, we add three arguments to knlist_init() to specify the functions to use to lock, unlock and check if the lock is owned. If these arguments are NULL, we assume mtx_lock, mtx_unlock and mtx_owned, respectively. - Using the vnode lock for the knlist locking, when doing kqueue operations on a vnode. This way, we don't have to lock the vnode while holding a mutex, in filt_vfsread. Reviewed by: jmg Approved by: re (scottl), scottl (mentor override) Pointyhat to: ssouhlal Will be happy: everyone Revision Changes Path 1.67 +1 -1 src/sys/cam/scsi/scsi_target.c 1.256 +1 -1 src/sys/kern/init_main.c 1.93 +83 -28 src/sys/kern/kern_event.c 1.252 +1 -1 src/sys/kern/kern_fork.c 1.184 +4 -2 src/sys/kern/sys_pipe.c 1.250 +2 -2 src/sys/kern/tty.c 1.242 +4 -2 src/sys/kern/uipc_socket.c 1.147 +4 -2 src/sys/kern/uipc_socket2.c 1.195 +1 -1 src/sys/kern/vfs_aio.c 1.634 +49 -23 src/sys/kern/vfs_subr.c 1.153 +1 -1 src/sys/net/bpf.c 1.234 +2 -2 src/sys/net/if.c 1.32 +7 -2 src/sys/sys/event.h 1.197 +9 -2 src/sys/sys/mount.h 1.303 +3 -3 src/sys/sys/vnode.h