From: William Lee Irwin III <wli@holomorphy.com>
To: Hugh Dickins <hugh@veritas.com>
Cc: Adam Litke <agl@us.ibm.com>,
Andrew Morton <akpm@linux-foundation.org>,
Arjan van de Ven <arjan@infradead.org>,
Christoph Hellwig <hch@infradead.org>,
Ken Chen <kenchen@google.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/7] [RFC] hugetlb: pagetable_operations API (V2)
Date: Wed, 21 Mar 2007 10:42:10 -0700 [thread overview]
Message-ID: <20070321174210.GI2986@holomorphy.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0703211650250.9630@blonde.wat.veritas.com>
On Wed, 21 Mar 2007, William Lee Irwin III wrote:
>> What sort of uglier stuff are you concerned about this enabling? My
>> wild guess is precisely the prospective users in my queue of features
>> to implement that I've neglected on account of the lack of such an
>> interface. It might be a good idea for me to take whatever distaste for
>> them exists into account before belting out the code for them.
On Wed, Mar 21, 2007 at 05:08:00PM +0000, Hugh Dickins wrote:
> I'll probably hate it whatever it is, don't worry about me ;) I'm
> rather weary of all the grand mm schemes people are toting these days.
Breathe a sigh of relief; I've retired from the world of grand schemes.
That much should be clear from my patch output over the past few years.
A driver in its own corner of the tree wouldn't have been so ambitious.
On Wed, Mar 21, 2007 at 05:08:00PM +0000, Hugh Dickins wrote:
> I guess my distaste is for letting a door open, for all kinds of
> drivers (or odd corners of architectures etc) to take control of
> the mm in ways we've never anticipated, and which we'll forever
> after be stumbling over.
> It would be a good idea for you to belt out the code for a few of them,
> to give everyone else an idea of what's to be let in through this door:
> I haven't the faintest idea what's envisaged.
After consulting with Adam Litke it appears I should remain silent on
this front.
On Wed, Mar 21, 2007 at 05:08:00PM +0000, Hugh Dickins wrote:
> Seeing what you have, maybe everyone will react that of course
> Adam's page table ops are the way to go, or maybe the reverse.
> I would be rather surprised if the adhoc collection of divergences
> you found necessary for hugetlb turn out to have a fundamental
> applicability. Perhaps we'll need subvert_the_core_ops.
The fact something can be done with them doesn't mean it has to be
done. At least in some corners of industry, the mere opportunity
to do things considered distasteful doesn't outweigh courtesy. Users
doing distasteful things need not be merged, nor shipped or supported,
regardless of what infrastructure they could utilize.
-- wli
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2007-03-21 17:42 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-19 20:05 Adam Litke
2007-03-19 20:05 ` [PATCH 1/7] Introduce the pagetable_operations and associated helper macros Adam Litke
2007-03-20 23:24 ` Dave Hansen
2007-03-21 14:50 ` Adam Litke
2007-03-21 15:05 ` Arjan van de Ven
2007-03-21 4:18 ` Nick Piggin
2007-03-21 4:52 ` William Lee Irwin III
2007-03-21 5:07 ` Nick Piggin
2007-03-21 5:41 ` William Lee Irwin III
2007-03-21 6:51 ` Nick Piggin
2007-03-21 7:36 ` Nick Piggin
2007-03-21 10:46 ` William Lee Irwin III
2007-03-21 15:17 ` Adam Litke
2007-03-21 16:00 ` Christoph Hellwig
2007-03-21 23:03 ` Nick Piggin
2007-03-21 23:02 ` Nick Piggin
2007-03-21 23:32 ` William Lee Irwin III
2007-03-19 20:05 ` [PATCH 2/7] copy_vma for hugetlbfs Adam Litke
2007-03-19 20:05 ` [PATCH 3/7] pin_pages for hugetlb Adam Litke
2007-03-19 20:05 ` [PATCH 4/7] unmap_page_range " Adam Litke
2007-03-20 23:27 ` Dave Hansen
2007-03-19 20:05 ` [PATCH 5/7] change_protection " Adam Litke
2007-03-19 20:06 ` [PATCH 6/7] free_pgtable_range " Adam Litke
2007-03-19 20:06 ` [PATCH 7/7] hugetlbfs fault handler Adam Litke
2007-03-20 23:50 ` [PATCH 0/7] [RFC] hugetlb: pagetable_operations API (V2) Dave Hansen
2007-03-21 1:17 ` William Lee Irwin III
2007-03-21 15:55 ` Hugh Dickins
2007-03-21 16:01 ` Christoph Hellwig
2007-03-21 16:23 ` William Lee Irwin III
2007-03-21 17:08 ` Hugh Dickins
2007-03-21 17:42 ` William Lee Irwin III [this message]
2007-03-21 19:43 ` pagetable_ops: Hugetlb character device example Adam Litke
2007-03-21 19:51 ` Valdis.Kletnieks
2007-03-21 20:26 ` Adam Litke
2007-03-21 22:26 ` William Lee Irwin III
2007-03-21 22:53 ` Matt Mackall
2007-03-21 23:35 ` William Lee Irwin III
2007-03-22 0:31 ` Matt Mackall
2007-03-22 10:38 ` Christoph Hellwig
2007-03-22 15:42 ` Mel Gorman
2007-03-22 18:15 ` Christoph Hellwig
2007-03-23 14:57 ` Mel Gorman
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=20070321174210.GI2986@holomorphy.com \
--to=wli@holomorphy.com \
--cc=agl@us.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=arjan@infradead.org \
--cc=hch@infradead.org \
--cc=hugh@veritas.com \
--cc=kenchen@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox