From owner-cvs-src@FreeBSD.ORG  Tue Jun 29 12:13:19 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 C217616A4CF; Tue, 29 Jun 2004 12:13:19 +0000 (GMT)
Received: from duke.cs.duke.edu (duke.cs.duke.edu [152.3.140.1])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 804D743D1F; Tue, 29 Jun 2004 12:13:19 +0000 (GMT)
	(envelope-from gallatin@cs.duke.edu)
Received: from grasshopper.cs.duke.edu (grasshopper.cs.duke.edu
	[152.3.145.30])
	by duke.cs.duke.edu (8.12.10/8.12.10) with ESMTP id i5TCDIqM025107
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Tue, 29 Jun 2004 08:13:18 -0400 (EDT)
Received: (from gallatin@localhost)
	by grasshopper.cs.duke.edu (8.12.9p2/8.12.9/Submit) id i5TCDB0Y004212;
	Tue, 29 Jun 2004 08:13:11 -0400 (EDT)
	(envelope-from gallatin)
From: Andrew Gallatin <gallatin@cs.duke.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <16609.23767.245708.270218@grasshopper.cs.duke.edu>
Date: Tue, 29 Jun 2004 08:13:11 -0400 (EDT)
To: Bruce Evans <bde@zeta.org.au>
In-Reply-To: <20040629114614.T2908@gamplex.bde.org>
References: <200406281915.i5SJFeaV060231@repoman.freebsd.org>
	<20040628152232.A2977@grasshopper.cs.duke.edu>
	<20040628193858.GG5635@green.homeunix.org>
	<16608.30892.745161.730935@grasshopper.cs.duke.edu>
	<20040629114614.T2908@gamplex.bde.org>
X-Mailer: VM 6.75 under 21.1 (patch 12) "Channel Islands" XEmacs Lucid
cc: Brian Fundakowski Feldman <green@freebsd.org>
cc: src-committers@freebsd.org
cc: cvs-all@freebsd.org
cc: cvs-src@freebsd.org
Subject: Re: cvs commit: src/sys/vm vm_map.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: Tue, 29 Jun 2004 12:13:19 -0000


Bruce Evans writes:
 > On Mon, 28 Jun 2004, Andrew Gallatin wrote:
 > 
 > > Brian Fundakowski Feldman writes:
 > >  > On Mon, Jun 28, 2004 at 03:22:33PM -0400, Andrew Gallatin wrote:
 > >  > > Andrew Gallatin [gallatin@FreeBSD.org] wrote:
 > >  > > > gallatin    2004-06-28 19:15:40 UTC
 > >  > > >
 > >  > > >   FreeBSD src repository
 > >  > > >
 > >  > > >   Modified files:
 > >  > > >     sys/vm               vm_map.c
 > >  > > >   Log:
 > >  > > >   Fix alpha - the use of min() on longs was loosing the high bits and
 > >  > > >   returning wrong answers, leading to strange values vm2->vm_{s,t,d}size.
 > 
 > MIN() and MAX() should not be used in the kernel.  4.4BSD removed them in
 > the kernel, but FreeBSD broke this in rev.1.141 of sys/param.h.  They
 > remain removed in RELENG_4.
 > 

OK.  Then what's the correct fix? ulmin()?

Thanks,

Drew