All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: David.Mosberger@acm.org
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linus Torvalds <torvalds@transmeta.com>
Subject: Re: [patch] remove hugetlb syscalls
Date: Thu, 14 Nov 2002 14:28:32 -0500	[thread overview]
Message-ID: <3DD3F960.6000501@pobox.com> (raw)
In-Reply-To: <Pine.LNX.4.44L.0211132239370.3817-100000@imladris.surriel.com>

David Mosberger-Tang wrote:

> But that's excactly the point.  The hugepage interface returns a
> different kind of virtual memory.  There are tons of programs out
> there using mmap().  If such a program gets fed a path to the
> hugepagefs, it might end up with huge pages without knowing anything
> about huge pages.  For the most part, that might work fine, but it
> could lead to subtle failures.


Yeah, that was one of Linus's points about the syscalls, in a private 
email.  I mentioned how the new syscalls were in poor taste, when 
existing syscalls would work fine, and he flamed me right back ;-)

One of his main points to me was exactly what you are elucidating: 
there are subtle differences between normal pages and superpages that 
are exposed to userland, and we should make that explicit [with the 
syscalls] rather than hide it [with hugetlbfs/mmap/etc.].  So I think 
this is further indication Linus has a very valid point ;-)

However, that said, I think hugetlbfs will almost always get used in 
preference to the syscalls, so leaving them in may be more a statement 
of technical correctness/cleanliness than anything else.

[tangent warning]
This whole hugetlb affair is unfortunately pretty ugly, and this thread 
is just one component of that.  All these discussions occurred off-list, 
and it's _still_ a political football.  Sigh.  I just hope that the 
furor dies down soon, that smart technical [apolitical] decisions are 
made, and future discussions are at least CC'd to lkml.

	Jeff




  parent reply	other threads:[~2002-11-14 19:21 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-13 23:45 [patch] remove hugetlb syscalls Benjamin LaHaise
2002-11-13 23:45 ` Benjamin LaHaise
2002-11-14  0:42 ` Rik van Riel
2002-11-14  0:42   ` Rik van Riel
2002-11-14  8:52   ` dada1
2002-11-14  8:52     ` dada1
2002-11-14 14:13     ` Christoph Hellwig
2002-11-14 14:13       ` Christoph Hellwig
     [not found]       ` <ugel9oa vk4.fsf@panda.mostang.com>
2002-11-14 15:13       ` dada1
2002-11-14 15:13         ` dada1
2002-11-14 15:31         ` Benjamin LaHaise
2002-11-14 15:31           ` Benjamin LaHaise
2002-11-14 15:38           ` dada1
2002-11-14 15:38             ` dada1
     [not found]           ` <3DD3FED2.2010901@unix-os.sc.intel.com>
2002-11-14 20:01             ` Benjamin LaHaise
2002-11-14 20:01               ` Benjamin LaHaise
2002-11-14 21:06             ` Benjamin LaHaise
2002-11-14 21:06               ` Benjamin LaHaise
2002-11-14 20:11           ` Rohit Seth
2002-11-14 20:11             ` Rohit Seth
2002-11-14 20:36             ` William Lee Irwin III
2002-11-14 20:36               ` William Lee Irwin III
2002-11-14 17:51       ` David Mosberger-Tang
2002-11-14 18:31         ` Alan Cox
2002-11-14 18:53           ` David Mosberger-Tang
2002-11-14 19:52             ` Alan Cox
2002-11-14 20:14               ` Alan Cox
2002-11-16 18:23                 ` Jan Niehusmann
2002-11-14 20:34         ` William Lee Irwin III
2002-11-14 21:31           ` David Mosberger-Tang
2002-11-14 21:38             ` William Lee Irwin III
2002-11-14 21:46               ` David Mosberger-Tang
2002-11-14 19:28   ` Jeff Garzik [this message]
2002-11-14 20:15     ` Alan Cox
2002-11-16 18:00     ` Rik van Riel
2002-11-16 18:15       ` Linus Torvalds
2002-11-14 20:30 ` William Lee Irwin III
2002-11-14 20:30   ` William Lee Irwin III
2002-11-14 20:48   ` Benjamin LaHaise
2002-11-14 20:48     ` Benjamin LaHaise
2002-11-14 21:02     ` William Lee Irwin III
2002-11-14 21:02       ` William Lee Irwin III
2002-11-14 21:11       ` Benjamin LaHaise
2002-11-14 21:11         ` Benjamin LaHaise
2002-11-14 21:31         ` William Lee Irwin III
2002-11-14 21:31           ` William Lee Irwin III
2002-11-14 21:40         ` Rohit Seth
2002-11-14 21:40           ` Rohit Seth
2002-11-14 21:59           ` Benjamin LaHaise
2002-11-14 21:59             ` Benjamin LaHaise
2002-11-14 21:06   ` Benjamin LaHaise
2002-11-14 21:06     ` Benjamin LaHaise
2002-11-14 21:06   ` Benjamin LaHaise
2002-11-14 21:06     ` Benjamin LaHaise
  -- strict thread matches above, loose matches on Subject: below --
2002-11-14 22:12 Seth, Rohit
2002-11-14 22:12 ` Seth, Rohit

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3DD3F960.6000501@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=David.Mosberger@acm.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@transmeta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.