From owner-cvs-src-old@FreeBSD.ORG Thu Jun 4 14:13:14 2009 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 53DEE1065738 for ; Thu, 4 Jun 2009 14:13:13 +0000 (UTC) (envelope-from rmacklem@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id C52EA8FC1D for ; Thu, 4 Jun 2009 14:13:13 +0000 (UTC) (envelope-from rmacklem@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n54EDDRN086767 for ; Thu, 4 Jun 2009 14:13:13 GMT (envelope-from rmacklem@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n54EDDuM086766 for cvs-src-old@freebsd.org; Thu, 4 Jun 2009 14:13:13 GMT (envelope-from rmacklem@repoman.freebsd.org) Message-Id: <200906041413.n54EDDuM086766@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to rmacklem@repoman.freebsd.org using -f From: Rick Macklem Date: Thu, 4 Jun 2009 14:13:06 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/rpc svc.c svc.h svc_vc.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Jun 2009 14:13:14 -0000 rmacklem 2009-06-04 14:13:06 UTC FreeBSD src repository Modified files: sys/rpc svc.c svc.h svc_vc.c Log: SVN rev 193436 on 2009-06-04 14:13:06Z by rmacklem Fix two races in the server side krpc w.r.t upcalls: Add a flag so that soupcall_clear() is only called once to cancel an upcall. Move the test for xprt_registered in the upcall down to after the mtx_lock() of the pool mutex, to catch the case where it is unregistered while the upcall is waiting for the mutex. Also, move the mtx_destroy() of the pool mutex to after SVC_RELEASE(), so that it isn't destroyed before the upcalls are disabled. Reviewed by: dfr, jhb Tested by: pho Approved by: kib (mentor) Revision Changes Path 1.4 +5 -4 src/sys/rpc/svc.c 1.4 +1 -0 src/sys/rpc/svc.h 1.7 +14 -3 src/sys/rpc/svc_vc.c