linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Anshuman Khandual <khandual@linux.vnet.ibm.com>
To: Mike Kravetz <mike.kravetz@oracle.com>,
	Anshuman Khandual <khandual@linux.vnet.ibm.com>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Cc: akpm@linux-foundation.org
Subject: Re: [PATCH V3] mm/madvise: Enable (soft|hard) offline of HugeTLB pages at PGD level
Date: Fri, 28 Jul 2017 11:23:34 +0530	[thread overview]
Message-ID: <c495d3d1-fd72-6f27-4397-ffbed7b2b2d3@linux.vnet.ibm.com> (raw)
In-Reply-To: <04ae16b1-8783-fb3b-4715-b96b6644566f@oracle.com>

On 07/28/2017 06:19 AM, Mike Kravetz wrote:
> On 05/16/2017 03:05 AM, Anshuman Khandual wrote:
>> Though migrating gigantic HugeTLB pages does not sound much like real
>> world use case, they can be affected by memory errors. Hence migration
>> at the PGD level HugeTLB pages should be supported just to enable soft
>> and hard offline use cases.
> 
> Hi Anshuman,
> 
> Sorry for the late question, but I just stumbled on this code when
> looking at something else.
> 
> It appears the primary motivation for these changes is to handle
> memory errors in gigantic pages.  In this case, you migrate to

Right.

> another gigantic page.  However, doesn't this assume that there is

Right.

> a pre-allocated gigantic page sitting unused that will be the target
> of the migration?  alloc_huge_page_node will not allocate a gigantic
> page.  Or, am I missing something?

Yes, its in the context of 16GB pages on POWER8 system where all the
gigantic pages are pre allocated from the platform and passed on to
the kernel through the device tree. We dont allocate these gigantic
pages on runtime.

--
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>

      reply	other threads:[~2017-07-28  5:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-26  3:57 [PATCH V2] mm/madvise: Enable (soft|hard) offline of HugeTLB pages at PGD level Anshuman Khandual
2017-05-12 21:35 ` Andrew Morton
2017-05-14  4:11   ` Anshuman Khandual
2017-05-16 10:05 ` [PATCH V3] " Anshuman Khandual
2017-07-28  0:49   ` Mike Kravetz
2017-07-28  5:53     ` Anshuman Khandual [this message]

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=c495d3d1-fd72-6f27-4397-ffbed7b2b2d3@linux.vnet.ibm.com \
    --to=khandual@linux.vnet.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mike.kravetz@oracle.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;
as well as URLs for NNTP newsgroup(s).