From owner-cvs-src-old@FreeBSD.ORG Wed Jan 7 21:26:03 2009 Return-Path: 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 4F8DB106566C for ; Wed, 7 Jan 2009 21:26:03 +0000 (UTC) (envelope-from marius@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 3D91D8FC17 for ; Wed, 7 Jan 2009 21:26:03 +0000 (UTC) (envelope-from marius@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 n07LQ3Tc080463 for ; Wed, 7 Jan 2009 21:26:03 GMT (envelope-from marius@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n07LQ3eb080461 for cvs-src-old@freebsd.org; Wed, 7 Jan 2009 21:26:03 GMT (envelope-from marius@repoman.freebsd.org) Message-Id: <200901072126.n07LQ3eb080461@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to marius@repoman.freebsd.org using -f From: Marius Strobl Date: Wed, 7 Jan 2009 21:25:44 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/dev/dcons dcons_crom.c dcons_os.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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 07 Jan 2009 21:26:03 -0000 marius 2009-01-07 21:25:44 UTC FreeBSD src repository Modified files: sys/dev/dcons dcons_crom.c dcons_os.c Log: SVN rev 186876 on 2009-01-07 21:25:44Z by marius Check the return values of contigmalloc(9) as well as bus_dma(9) functions and stop attaching of dcons(4) and dcons_crom(4) if they indicate failure. This fixes a panic seen on sparc64 machines with no free physical memory in the requested 32-bit region but still doesn't make dcons(4)/dcons_crom(4) these work. I think the latter can be fixed by simply specifying ~0UL as the upper limit for contigmalloc(9) and letting the bounce pages and the IOMMU respectively handle limitations of the DMA engine. I didn't want to change that without the consensus of simokawa@ though, who unfortunately didn't reply so far. MFC after: 1 week Revision Changes Path 1.11 +12 -3 src/sys/dev/dcons/dcons_crom.c 1.25 +11 -7 src/sys/dev/dcons/dcons_os.c