Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1681267

Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification for powerpc

Show all headers | View raw

On Tuesday 04 July 2017 01:35 AM, Victor Aoqui wrote:
> Implemented default hugepage size verification (default_hugepagesz=)
> in order to allow allocation of defined number of pages (hugepages=)
> only for supported hugepage sizes.
> 
> Signed-off-by: Victor Aoqui <victora@br.ibm.com>
> ---
>   arch/powerpc/mm/hugetlbpage.c | 15 +++++++++++++++
>   1 file changed, 15 insertions(+)
> 
> diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
> index a4f33de..464e72e 100644
> --- a/arch/powerpc/mm/hugetlbpage.c
> +++ b/arch/powerpc/mm/hugetlbpage.c
> @@ -797,6 +797,21 @@ static int __init hugepage_setup_sz(char *str)
>   }
>   __setup("hugepagesz=", hugepage_setup_sz);
> 
> +static int __init default_hugepage_setup_sz(char *str)
> +{
> +        unsigned long long size;
> +
> +        size = memparse(str, &str);
> +
> +        if (add_huge_page_size(size) != 0) {
> +                hugetlb_bad_size();
> +                pr_err("Invalid default huge page size specified(%llu)\n", size);
> +        }
> +
> +        return 1;
> +}
> +__setup("default_hugepagesz=", default_hugepage_setup_sz);

isn't that a behavior change in what we have now ? . Right now if size 
specified is not supported, we fallback to HPAGE_SIZE.

mm/hugetlb.c

	if (!size_to_hstate(default_hstate_size)) {
		default_hstate_size = HPAGE_SIZE;
		if (!size_to_hstate(default_hstate_size))
			hugetlb_add_hstate(HUGETLB_PAGE_ORDER);
	}


> +
>   struct kmem_cache *hugepte_cache;
>   static int __init hugetlbpage_init(void)
>   {
> 

Even if we want to do this, this should be done in generic code and 
should not be powerpc specific

-aneesh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] powerpc/mm: Implemented default_hugepagesz verification for powerpc Victor Aoqui <victora@br.ibm.com> - 2017-07-03 22:10 +0200
  Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification  for powerpc "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> - 2017-07-05 06:30 +0200
    Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification  for powerpc victora <victora@linux.vnet.ibm.com> - 2017-07-05 18:10 +0200
      Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification  for powerpc victora <victora@linux.vnet.ibm.com> - 2017-07-12 17:20 +0200
  Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification  for powerpc Anshuman Khandual <khandual@linux.vnet.ibm.com> - 2017-07-05 06:40 +0200
    Re: [PATCH] powerpc/mm: Implemented default_hugepagesz verification  for powerpc victora <victora@linux.vnet.ibm.com> - 2017-07-05 18:10 +0200

csiph-web