From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Mel Gorman <mel@csn.ul.ie>
Cc: Alexey Korolev <akorolex@gmail.com>,
Eric Munson <linux-mm@mgebm.net>,
Alexey Korolev <akorolev@infradead.org>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3]HTLB mapping for drivers (take 2)
Date: Tue, 25 Aug 2009 21:00:54 +1000 [thread overview]
Message-ID: <1251198054.15197.40.camel@pasglop> (raw)
In-Reply-To: <20090825104731.GA21335@csn.ul.ie>
On Tue, 2009-08-25 at 11:47 +0100, Mel Gorman wrote:
> Why? One hugepage of default size will be one TLB entry. Each hugepage
> after that will be additional TLB entries so there is no savings on
> translation overhead.
>
> Getting contiguous pages beyond the hugepage boundary is not a matter
> for GFP flags.
Note: This patch reminds me of something else I had on the backburner
for a while and never got a chance to actually implement...
There's various cases of drivers that could have good uses of hugetlb
mappings of device memory. For example, framebuffers.
I looked at it a while back and it occured to me (and Nick) that
ideally, we should split hugetlb and hugetlbfs.
Basically, on one side, we have the (mostly arch specific) populating
and walking of page tables with hugetlb translations, associated huge
VMAs, etc...
On the other side, hugetlbfs is backing that with memory.
Ideally, the former would have some kind of "standard" ops that
hugetlbfs can hook into for the existing case (moving some stuff out of
the common data structure and splitting it in two), allowing the driver
to instanciate hugetlb VMAs that are backed up by something else,
typically a simple mapping of IOs.
Anybody wants to do that or I keep it on my back burner until the time I
finally get to do it ? :-)
Cheers,
Ben.
--
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:[~2009-08-26 4:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-17 22:24 Alexey Korolev
2009-08-18 10:29 ` Eric Munson
2009-08-19 5:48 ` Alexey Korolev
2009-08-19 10:05 ` Mel Gorman
2009-08-19 10:35 ` Eric B Munson
2009-08-20 7:03 ` Alexey Korolev
2009-08-25 10:47 ` Mel Gorman
2009-08-25 11:00 ` Benjamin Herrenschmidt [this message]
2009-08-25 11:10 ` Mel Gorman
2009-08-26 9:58 ` Benjamin Herrenschmidt
2009-08-26 10:05 ` Mel Gorman
2009-08-24 6:14 ` Alexey Korolev
2009-08-25 10:53 ` Mel Gorman
2009-08-27 12:02 ` Alexey Korolev
2009-08-27 12:50 ` 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=1251198054.15197.40.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=akorolev@infradead.org \
--cc=akorolex@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-mm@mgebm.net \
--cc=mel@csn.ul.ie \
/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