From owner-cvs-src-old@FreeBSD.ORG Sat Apr 4 23:13:30 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 1AF51106564A for ; Sat, 4 Apr 2009 23:13:30 +0000 (UTC) (envelope-from alc@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 071838FC19 for ; Sat, 4 Apr 2009 23:13:30 +0000 (UTC) (envelope-from alc@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 n34NDTP3039994 for ; Sat, 4 Apr 2009 23:13:29 GMT (envelope-from alc@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n34NDTNN039993 for cvs-src-old@freebsd.org; Sat, 4 Apr 2009 23:13:29 GMT (envelope-from alc@repoman.freebsd.org) Message-Id: <200904042313.n34NDTNN039993@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to alc@repoman.freebsd.org using -f From: Alan Cox Date: Sat, 4 Apr 2009 23:12:14 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/arm/include vmparam.h src/sys/i386/include vmparam.h src/sys/vm vm_mmap.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: Sat, 04 Apr 2009 23:13:30 -0000 alc 2009-04-04 23:12:14 UTC FreeBSD src repository Modified files: sys/arm/include vmparam.h sys/i386/include vmparam.h sys/vm vm_mmap.c Log: SVN rev 190705 on 2009-04-04 23:12:14Z by alc Retire VM_PROT_READ_IS_EXEC. It was intended to be a micro-optimization, but I see no benefit from it today. VM_PROT_READ_IS_EXEC was only intended for use on processors that do not distinguish between read and execute permission. On an mmap(2) or mprotect(2), it automatically added execute permission if the caller specified permissions included read permission. The hope was that this would reduce the number of vm map entries needed to implement an address space because there would be fewer neighboring vm map entries that differed only in the presence or absence of VM_PROT_EXECUTE. (See vm/vm_mmap.c revision 1.56.) Today, I don't see any real applications that benefit from VM_PROT_READ_IS_EXEC. In any case, vm map entries are now organized as a self-adjusting binary search tree instead of an ordered list. So, the need for coalescing vm map entries is not as great as it once was. Revision Changes Path 1.14 +0 -2 src/sys/arm/include/vmparam.h 1.49 +0 -4 src/sys/i386/include/vmparam.h 1.229 +0 -12 src/sys/vm/vm_mmap.c