From: "Chen, Kenneth W" <kenneth.w.chen@intel.com>
To: "'David Gibson'" <david@gibson.dropbear.id.au>,
"Zhang, Yanmin" <yanmin.zhang@intel.com>
Cc: "Andrew Morton" <akpm@osdl.org>,
"William Lee Irwin" <wli@holomorphy.com>,
<linux-kernel@vger.kernel.org>
Subject: RE: hugepage: Strict page reservation for hugepage inodes
Date: Wed, 8 Mar 2006 10:38:58 -0800 [thread overview]
Message-ID: <200603081838.k28Icwg10327@unix-os.sc.intel.com> (raw)
In-Reply-To: <20060308102314.GB32571@localhost.localdomain>
David Gibson wrote on Wednesday, March 08, 2006 2:23 AM
> Yes. This is a simplifying assumption. I know of no real application
> that will waste pages because of this behaviour. If you know one,
> maybe we will need to reconsider.
>
> > I have an idea. How about to record all the start/end address of
> > huge page mmaping of the inode? Long long ago, there was a patch at
> > http://marc.theaimsgroup.com/?l=lse-tech&m=108187931924134&w=2.
> > Of course, we need port it to the latest kernel if this idea is better.
>
> I know the patch - I was going to port it to the current kernel, but
> came up with my patch instead, because it seemed like a simpler
> approach.
I really think the Variable length reservation system is the way to go
for tracking hugetlb commit. It is more robust and in my opinion, it
is better than traverse the page cache radix tree. At least, you don't
have to worry about all the race condition there. Oh, it also can get
rid of the hugetlb_instantiation_mutex that was introduced. Someday,
people is going to scream at you for serializing hugetlb fault path.
- Ken
next prev parent reply other threads:[~2006-03-08 18:39 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-08 9:25 Zhang, Yanmin
2006-03-08 10:23 ` David Gibson
2006-03-08 18:38 ` Chen, Kenneth W [this message]
2006-03-08 23:52 ` 'David Gibson'
2006-03-09 0:19 ` Chen, Kenneth W
2006-03-09 0:30 ` 'David Gibson'
2006-03-09 0:29 ` Andrew Morton
-- strict thread matches above, loose matches on Subject: below --
2006-02-28 9:21 Zhang, Yanmin
2006-02-28 23:35 ` David Gibson
2006-02-28 8:53 Zhang, Yanmin
2006-02-28 9:14 ` David Gibson
2006-02-28 7:11 David Gibson
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=200603081838.k28Icwg10327@unix-os.sc.intel.com \
--to=kenneth.w.chen@intel.com \
--cc=akpm@osdl.org \
--cc=david@gibson.dropbear.id.au \
--cc=linux-kernel@vger.kernel.org \
--cc=wli@holomorphy.com \
--cc=yanmin.zhang@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®