Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754572AbZIUXqo (ORCPT ); Mon, 21 Sep 2009 19:46:44 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754513AbZIUXqn (ORCPT ); Mon, 21 Sep 2009 19:46:43 -0400 Received: from smtp-out.google.com ([216.239.45.13]:62954 "EHLO smtp-out.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754499AbZIUXqm (ORCPT ); Mon, 21 Sep 2009 19:46:42 -0400 DomainKey-Signature: a=rsa-sha1; s=beta; d=google.com; c=nofws; q=dns; h=date:from:x-x-sender:to:cc:subject:in-reply-to:message-id: references:user-agent:mime-version:content-type:x-system-of-record; b=qQzTyX1lFtJ8AAlgzT5G34nM+BI+YLHWR7UJw43QCPu8D9YsFh9YIr35FgoK/i61+ 89475s6Wg8C4QlhwfUTBw== Date: Mon, 21 Sep 2009 16:46:38 -0700 (PDT) From: David Rientjes X-X-Sender: rientjes@chino.kir.corp.google.com To: "Luck, Tony" cc: Hugh Dickins , Andrew Morton , "Yu, Fenghua" , ebmunson@us.ibm.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-man@vger.kernel.org, mtk.manpages@gmail.com, Randy Dunlap , rth@twiddle.net, ink@jurassic.park.msu.ru, linux-ia64@vger.kernel.org, Arnd Bergmann , Ulrich Drepper , Alan Cox Subject: RE: [PATCH] remove duplicate asm/mman.h files In-Reply-To: <57C9024A16AD2D4C97DC78E552063EA3E29CC3F1@orsmsx505.amr.corp.intel.com> Message-ID: References: <200909181848.42192.arnd@arndb.de> <200909211031.25369.arnd@arndb.de> <57C9024A16AD2D4C97DC78E552063EA3E29CC3F1@orsmsx505.amr.corp.intel.com> User-Agent: Alpine 1.00 (DEB 882 2007-12-20) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-System-Of-Record: true Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1903 Lines: 41 On Mon, 21 Sep 2009, Luck, Tony wrote: > >> Is it perhaps the case that some UNIX on ia64 does implement MAP_GROWSUP, > >> and these numbers in the Linux ia64 mman.h have been chosen to match that > >> reference implementation? Tony will know. But I wonder if you'd do > >> better at least to leave a MAP_GROWSUP comment on that line, so that > >> somebody doesn't go and reuse the empty slot later on. > >> > > > > Reserving the bit from future use by adding a comment may be helpful, but > > then let's do it for MAP_GROWSDOWN too. > > Tony can only speculate because this bit has been in asm/mman.h > since before I started working on Linux (it is in the 2.4.0 > version ... which is roughly when I started ... and long before > I was responsible for it). > > Perhaps it was assumed that it would be useful? Linux/ia64 does > use upwardly growing memory areas (the h/w register stack engine > saves "stack" registers to an area that grows upwards). > > But since we have survived this long without it actually being > implemented, it may be true that we don't really need it after > all. > glibc notes that both MAP_GROWSUP and MAP_GROWSDOWN are specific to Linux, yet they don't functionally do anything. While it may be true that there's no cost associated with keeping them around, I also think exporting such flags to userspace may give developers the belief that the implementation actually respects them when they're passed. Ulrich wanted to do this last year but it appears to have been dropped. Unless there's a convincing argument in the other direction, I don't see why they both can't just be removed and their bits reserved. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/