From owner-svn-src-head@FreeBSD.ORG Fri Aug 9 15:53:37 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id DA7196CB; Fri, 9 Aug 2013 15:53:37 +0000 (UTC) (envelope-from tijl@freebsd.org) Received: from mailrelay001.isp.belgacom.be (mailrelay001.isp.belgacom.be [195.238.6.51]) by mx1.freebsd.org (Postfix) with ESMTP id 0FF1F2033; Fri, 9 Aug 2013 15:53:36 +0000 (UTC) X-Belgacom-Dynamic: yes X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AmkGAHwPBVJR8mj4/2dsb2JhbABbgwY1RL5fgRkXdIIkAQEFJxMcIxALFAQJDBkPKh4GiCcIuRyOa4EwBwqEBAOUCoNWgSuQJYFfgTs6gS0 Received: from 248.104-242-81.adsl-dyn.isp.belgacom.be (HELO kalimero.tijl.coosemans.org) ([81.242.104.248]) by relay.skynet.be with ESMTP; 09 Aug 2013 17:52:26 +0200 Received: from kalimero.tijl.coosemans.org (kalimero.tijl.coosemans.org [127.0.0.1]) by kalimero.tijl.coosemans.org (8.14.7/8.14.7) with ESMTP id r79FqPXt084464; Fri, 9 Aug 2013 17:52:25 +0200 (CEST) (envelope-from tijl@FreeBSD.org) Date: Fri, 9 Aug 2013 17:52:24 +0200 From: Tijl Coosemans To: Attilio Rao Subject: Re: svn commit: r254141 - in head/sys: amd64/amd64 dev/drm2/i915 dev/drm2/ttm i386/i386 kern vm Message-ID: <20130809175224.4ee2fd11@kalimero.tijl.coosemans.org> In-Reply-To: <201308091128.r79BSuYY000651@svn.freebsd.org> References: <201308091128.r79BSuYY000651@svn.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 09 Aug 2013 15:53:37 -0000 On Fri, 9 Aug 2013 11:28:56 +0000 (UTC) Attilio Rao wrote: > Author: attilio > Date: Fri Aug 9 11:28:55 2013 > New Revision: 254141 > URL: http://svnweb.freebsd.org/changeset/base/254141 > > Log: > On all the architectures, avoid to preallocate the physical memory > for nodes used in vm_radix. > On architectures supporting direct mapping, also avoid to pre-allocate > the KVA for such nodes. > > In order to do so make the operations derived from vm_radix_insert() > to fail and handle all the deriving failure of those. > > vm_radix-wise introduce a new function called vm_radix_replace(), > which can replace a leaf node, already present, with a new one, > and take into account the possibility, during vm_radix_insert() > allocation, that the operations on the radix trie can recurse. > This means that if operations in vm_radix_insert() recursed > vm_radix_insert() will start from scratch again. > > Sponsored by: EMC / Isilon storage division > Reviewed by: alc (older version) > Reviewed by: jeff > Tested by: pho, scottl VPO_BUSY doesn't appear to be defined anywhere: cc -c -O -pipe -march=pentium4m -std=c99 -g -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -nostdinc -I. -I/usr/home/tijl/projects/freebsd/base/head/sys -I/usr/home/tijl/projects/freebsd/base/head/sys/contrib/altq -I/usr/home/tijl/projects/freebsd/base/head/sys/contrib/libfdt -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -mno-aes -mno-avx -mno-mmx -mno-sse -msoft-float -ffreestanding -fstack-protector -Werror /usr/home/tijl/projects/freebsd/base/head/sys/vm/vm_page.c /usr/home/tijl/projects/freebsd/base/head/sys/vm/vm_page.c:1205:21: error: use of undeclared identifier 'VPO_BUSY' if (mold->oflags & VPO_BUSY) { ^ /usr/home/tijl/projects/freebsd/base/head/sys/vm/vm_page.c:1206:20: error: use of undeclared identifier 'VPO_BUSY' mold->oflags &= ~VPO_BUSY; ^ 2 errors generated.