From owner-cvs-src-old@FreeBSD.ORG  Thu Jun 25 00:29:07 2009
Return-Path: <owner-cvs-src-old@FreeBSD.ORG>
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 A016910656C7
	for <cvs-src-old@freebsd.org>; Thu, 25 Jun 2009 00:29:07 +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 6EE4A8FC18
	for <cvs-src-old@freebsd.org>; Thu, 25 Jun 2009 00:29:07 +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 n5P0T7pj086858
	for <cvs-src-old@freebsd.org>; Thu, 25 Jun 2009 00:29:07 GMT
	(envelope-from rmacklem@repoman.freebsd.org)
Received: (from svn2cvs@localhost)
	by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n5P0T7GJ086857
	for cvs-src-old@freebsd.org; Thu, 25 Jun 2009 00:29:07 GMT
	(envelope-from rmacklem@repoman.freebsd.org)
Message-Id: <200906250029.n5P0T7GJ086857@repoman.freebsd.org>
X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to
	rmacklem@repoman.freebsd.org using -f
From: Rick Macklem <rmacklem@FreeBSD.org>
Date: Thu, 25 Jun 2009 00:28:43 +0000 (UTC)
To: cvs-src-old@freebsd.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/rpc clnt_rc.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
	<cvs-src-old.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src-old>,
	<mailto:cvs-src-old-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src-old>
List-Post: <mailto:cvs-src-old@freebsd.org>
List-Help: <mailto:cvs-src-old-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src-old>,
	<mailto:cvs-src-old-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 25 Jun 2009 00:29:08 -0000

rmacklem    2009-06-25 00:28:43 UTC

  FreeBSD src repository

  Modified files:
    sys/rpc              clnt_rc.c 
  Log:
  SVN rev 194934 on 2009-06-25 00:28:43Z by rmacklem
  
  Fix two known problems in clnt_rc.c, plus issues w.r.t. smp noted
  during reading of the code. Change the code so that it never accesses
  rc_connecting, rc_closed or rc_client when the rc_lock mutex is not held.
  Also, it now performs the CLNT_CLOSE(client) and CLNT_RELEASE(client)
  calls after the rc_lock mutex has been released, since those calls do
  msleep()s with another mutex held. Change clnt_reconnect_call() so that
  releasing the reference count is delayed until after the
  "if (rc->rc_client == client)" check, so that rc_client cannot have been
  recycled.
  
  Tested by:      pho
  Reviewed by:    dfr
  Approved by:    kib (mentor)
  
  Revision  Changes    Path
  1.9       +74 -42    src/sys/rpc/clnt_rc.c