From owner-cvs-src@FreeBSD.ORG  Sun Jul  4 15:59:25 2004
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 67BE216A4CE; Sun,  4 Jul 2004 15:59:25 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 6269443D31; Sun,  4 Jul 2004 15:59:25 +0000 (GMT)
	(envelope-from green@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.12.11/8.12.11) with ESMTP id i64FxPNH048981;
	Sun, 4 Jul 2004 15:59:25 GMT
	(envelope-from green@repoman.freebsd.org)
Received: (from green@localhost)
	by repoman.freebsd.org (8.12.11/8.12.11/Submit) id i64FxPpj048980;
	Sun, 4 Jul 2004 15:59:25 GMT
	(envelope-from green)
Message-Id: <200407041559.i64FxPpj048980@repoman.freebsd.org>
From: Brian Feldman <green@FreeBSD.org>
Date: Sun, 4 Jul 2004 15:59:25 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/vm uma_core.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 04 Jul 2004 15:59:25 -0000

green       2004-07-04 15:59:25 UTC

  FreeBSD src repository

  Modified files:
    sys/vm               uma_core.c 
  Log:
  Reextend the M_WAITOK-disabling-hack to all three of the mbuf-related
  zones, and do it by direct comparison of uma_zone_t instead of strcmp.
  
  The mbuf subsystem used to provide M_TRYWAIT/M_DONTWAIT semantics, but
  this is mostly no longer the case.  M_WAITOK has taken over the spot
  M_TRYWAIT used to have, and for mbuf things, still may return NULL if
  the code path is incorrectly holding a mutex going into mbuf allocation
  functions.
  
  The M_WAITOK/M_NOWAIT semantics are absolute; though it may deadlock
  the system to try to malloc or uma_zalloc something with a mutex held
  and M_WAITOK specified, it is absolutely required to not return NULL
  and will result in instability and/or security breaches otherwise.
  There is still room to add the WITNESS_WARN() to all cases so that
  we are notified of the possibility of deadlocks, but it cannot change
  the value of the "badness" variable and allow allocation to actually
  fail except for the specialized cases which used to be M_TRYWAIT.
  
  Revision  Changes    Path
  1.99      +4 -2      src/sys/vm/uma_core.c