LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
To: linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] powerpc/mm: Disable registering 16G hugepage size for powernv platform
Date: Tue, 25 Jul 2017 10:37:14 +0530	[thread overview]
Message-ID: <5f0f7a59-1d8d-039a-9176-2e2b947b2318@linux.vnet.ibm.com> (raw)
In-Reply-To: <87wp6yjfia.fsf@concordia.ellerman.id.au>



On 07/24/2017 03:58 PM, Michael Ellerman wrote:
> "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> writes:
>
>> We don't support allocating them runtime. Only supported mechanism as of now is
>> HMC based hypervisor interface. So disable 16G hugetlb on powernv platform
>>
>> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
>> ---
>>
>> May be this should go to -stable ?. To show the 16G pagesize in
>> /sys/kernel/mm/hugepages/ confuse users
> It's a bit of a hack :/
>
> 16G also shows up on pseries, when there are no pages pre-allocated,
> which is also confusing.
>
> Ideally we'd only show sizes that can be allocated at runtime, OR were
> preallocated, regardless of platform.
>
> Is there an easy way to do that?
>
>


That is tricky, we are are going to enable 16G hugetlb on powernv with a 
followup patch. In that series we depend on the specifying hugetlb page 
count via kernel command line. The way above kernel command line parsing 
work is by looking at the hugetlb hstate which got allocated before and 
then assign the number of hugepages to that hstate. What you are asking 
is to fail the creating of hstate based on number of hugepages.

For now there is no way of using 16G hugepage on powernv. This patch 
address that issue by not showing 16G as a supported hugetlb page. In 
the later series, we will enable 16G hugetlb for powernv
and at which point both pseries and powernv behavior will remain consistent.

-aneesh

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

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-24  7:26 [PATCH] powerpc/mm: Disable registering 16G hugepage size for powernv platform Aneesh Kumar K.V
2017-07-24 10:28 ` Michael Ellerman
2017-07-25  5:07   ` Aneesh Kumar K.V [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=5f0f7a59-1d8d-039a-9176-2e2b947b2318@linux.vnet.ibm.com \
    --to=aneesh.kumar@linux.vnet.ibm.com \
    --cc=linuxppc-dev@lists.ozlabs.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