From owner-cvs-src@FreeBSD.ORG Mon Feb 23 04:57:52 2004 Return-Path: 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 1A24316A4CE; Mon, 23 Feb 2004 04:57:52 -0800 (PST) Received: from gw.celabo.org (gw.celabo.org [208.42.49.153]) by mx1.FreeBSD.org (Postfix) with ESMTP id E294C43D1D; Mon, 23 Feb 2004 04:57:51 -0800 (PST) (envelope-from nectar@celabo.org) Received: from madman.celabo.org (madman.celabo.org [10.0.1.111]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "madman.celabo.org", Issuer "celabo.org CA" (verified OK)) by gw.celabo.org (Postfix) with ESMTP id 86012548A2; Mon, 23 Feb 2004 06:57:51 -0600 (CST) Received: by madman.celabo.org (Postfix, from userid 1001) id 1FBE36D455; Mon, 23 Feb 2004 06:57:51 -0600 (CST) Date: Mon, 23 Feb 2004 06:57:51 -0600 From: "Jacques A. Vidrine" To: Robert Watson Message-ID: <20040223125751.GC15721@madman.celabo.org> References: <200402230318.i1N3IIdN084914@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200402230318.i1N3IIdN084914@repoman.freebsd.org> X-Url: http://www.celabo.org/ User-Agent: Mutt/1.5.4i-ja.1 cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/drm r128_state.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Feb 2004 12:57:52 -0000 On Sun, Feb 22, 2004 at 07:18:18PM -0800, Robert Watson wrote: > rwatson 2004/02/22 19:18:18 PST > > FreeBSD src repository > > Modified files: > sys/dev/drm r128_state.c > Log: > Limit the amount of memory userspace processes can cause the kernel to > allocate via DRI on r128 devices. > > Obtained from: Thomas Biege > Reviewed by: scottl > > Revision Changes Path > 1.9 +17 -0 src/sys/dev/drm/r128_state.c FWIW The issue was discovered in the Linux kernel and reported by Alan Cox. Alan came up with the initial patch; Thomas added the `count <= 0' and the redundant `count * sizeof' checks. Cheers, -- Jacques Vidrine / nectar@celabo.org / jvidrine@verio.net / nectar@freebsd.org