From owner-cvs-all@FreeBSD.ORG  Fri Sep 12 21:44:47 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 76C2E10656C5;
	Fri, 12 Sep 2008 21:44:47 +0000 (UTC)
	(envelope-from attilio@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 64B478FC1A;
	Fri, 12 Sep 2008 21:44:47 +0000 (UTC)
	(envelope-from attilio@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 m8CLilfA098739;
	Fri, 12 Sep 2008 21:44:47 GMT
	(envelope-from attilio@repoman.freebsd.org)
Received: (from svn2cvs@localhost)
	by repoman.freebsd.org (8.14.3/8.14.3/Submit) id m8CLiltA098736;
	Fri, 12 Sep 2008 21:44:47 GMT
	(envelope-from attilio@repoman.freebsd.org)
Message-Id: <200809122144.m8CLiltA098736@repoman.freebsd.org>
X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to
	attilio@repoman.freebsd.org using -f
From: Attilio Rao <attilio@FreeBSD.org>
Date: Fri, 12 Sep 2008 21:44:01 +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/kern subr_witness.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 12 Sep 2008 21:44:47 -0000

attilio     2008-09-12 21:44:01 UTC

  FreeBSD src repository

  Modified files:
    sys/kern             subr_witness.c 
  Log:
  SVN rev 182984 on 2008-09-12 21:44:01Z by attilio
  
  - For any lock list we hold the head in order to reduce allocation from
    the free list and in this way avoid contention on the w_mtx.
    In order to make the code simple, we rely on the rule that when the head
    has not a child it also doesn't have other subsequent entries.
    Actually this assertion is broken because we can free all the head
    children and quit witness_unlock() with the head still allocated, with no
    children and subsequent entries present.
    Fix this by shifting the head if other entries are present and still
    freeing the object, but leaving always an head.
  - Fix witness_thread_has_locks() in order to report, correctly, if the
    lock list linked to a specific thread has children or not based on the
    above explained rule.
  - Fix a printout into DDB's "show alllocks" command in order to show,
    correctly, the process name that is really what we want.
  - Fix style(9) for a comment.
  
  Tested by:      Giovanni Trematerra <giovanni dot trematerra at gmail dot com>
  Reported by:    Marko Kiiskila <marko dot kiiskila at nokia dot com>
  Sponsored by:   Nokia
  
  Revision  Changes    Path
  1.255     +18 -5     src/sys/kern/subr_witness.c