From: "Chen, Kenneth W" <kenneth.w.chen@intel.com>
To: "'David Gibson'" <david@gibson.dropbear.id.au>
Cc: "Zhang, Yanmin" <yanmin.zhang@intel.com>,
"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 16:19:13 -0800 [thread overview]
Message-ID: <200603090019.k290JDg13362@unix-os.sc.intel.com> (raw)
In-Reply-To: <20060308235207.GB17590@localhost.localdomain>
David Gibson wrote on Wednesday, March 08, 2006 3:52 PM
> But I don't see that recording all the mapped ranges will avoid the
> need for the fault serialization. At least the version of apw's
> reservation patch I looked at most recently would certainly still
> suffer from the alloc/instantiate race on the last hugepage in the
> system.
No, it doesn't. Because with strict commit accounting, you know that
every hugetlb page is accounted for. So there is no backout path for
multiple instantiation race. Thread that lost in the race will always
go back to retry in hugetlb_no_page(). And since reservation is also
accounted in a global variable, total hugetlb pool won't fall below
what was reserved plus what is in use. Even if sys admin tries to
reduce hugetlb pool, kernel won't release any pages that are reserved.
- Ken
next prev parent reply other threads:[~2006-03-09 0:19 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
2006-03-08 23:52 ` 'David Gibson'
2006-03-09 0:19 ` Chen, Kenneth W [this message]
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=200603090019.k290JDg13362@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
Powered by JetHome