Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753464AbcCVXki (ORCPT <rfc822;w@1wt.eu>);
	Tue, 22 Mar 2016 19:40:38 -0400
Received: from TYO201.gate.nec.co.jp ([210.143.35.51]:35059 "EHLO
	tyo201.gate.nec.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753406AbcCVXkg convert rfc822-to-8bit (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 22 Mar 2016 19:40:36 -0400
From: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
To: Vaishali Thakkar <vaishali.thakkar@oracle.com>
CC: "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-mm@kvack.org" <linux-mm@kvack.org>,
        Mike Kravetz <mike.kravetz@oracle.com>,
        Hillf Danton <hillf.zj@alibaba-inc.com>,
        "Michal Hocko" <mhocko@suse.com>,
        Yaowei Bai <baiyaowei@cmss.chinamobile.com>,
        "Dominik Dingel" <dingel@linux.vnet.ibm.com>,
        "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
        Paul Gortmaker <paul.gortmaker@windriver.com>,
        Dave Hansen <dave.hansen@linux.intel.com>
Subject: Re: [PATCH 2/2] arch:mm: Use hugetlb_bad_size
Thread-Topic: [PATCH 2/2] arch:mm: Use hugetlb_bad_size
Thread-Index: AQHRhCNDjZJwh70YRkuUhsKsZ5UqXZ9liKIA
Date: Tue, 22 Mar 2016 23:38:10 +0000
Message-ID: <20160322233809.GB24819@hori1.linux.bs1.fc.nec.co.jp>
References: <1458641159-13643-1-git-send-email-vaishali.thakkar@oracle.com>
In-Reply-To: <1458641159-13643-1-git-send-email-vaishali.thakkar@oracle.com>
Accept-Language: ja-JP, en-US
Content-Language: ja-JP
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [10.128.101.8]
Content-Type: text/plain; charset="iso-2022-jp"
Content-ID: <FFF1ABDB1F1E474E9B72297C37B82DAF@gisp.nec.co.jp>
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 4166
Lines: 116

On Tue, Mar 22, 2016 at 03:35:59PM +0530, Vaishali Thakkar wrote:
> Update the setup_hugepagesz function to call the routine
> hugetlb_bad_size when unsupported hugepage size is found.
> 
> Misc:
>   - Silent 80 characters warning
> 
> Signed-off-by: Vaishali Thakkar <vaishali.thakkar@oracle.com>
> Cc: Mike Kravetz <mike.kravetz@oracle.com>
> Cc: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
> Cc: Hillf Danton <hillf.zj@alibaba-inc.com>
> Cc: Michal Hocko <mhocko@suse.com>
> Cc: Yaowei Bai <baiyaowei@cmss.chinamobile.com>
> Cc: Dominik Dingel <dingel@linux.vnet.ibm.com>
> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
> Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
> Cc: Dave Hansen <dave.hansen@linux.intel.com>

Reviewed-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>

> ---
> - Please note that the patch is tested for x86 only. But as this
>   is one line change I just changed them. So, it would be good if
>   the patch can be tested for other architectures before adding
>   this in to mainline.
> - Not sure if printk related checkpatch.pl warning should be resolved
>   with this patch as code is not consistent in architectures. May be
>   one separate patch for changing all printk's to pr_<level> kind of
>   debugging functions would be good.
> ---
>  arch/arm64/mm/hugetlbpage.c   | 1 +
>  arch/metag/mm/hugetlbpage.c   | 1 +
>  arch/powerpc/mm/hugetlbpage.c | 7 +++++--
>  arch/tile/mm/hugetlbpage.c    | 7 ++++++-
>  arch/x86/mm/hugetlbpage.c     | 1 +
>  5 files changed, 14 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/mm/hugetlbpage.c b/arch/arm64/mm/hugetlbpage.c
> index 589fd28..aa8aee7 100644
> --- a/arch/arm64/mm/hugetlbpage.c
> +++ b/arch/arm64/mm/hugetlbpage.c
> @@ -307,6 +307,7 @@ static __init int setup_hugepagesz(char *opt)
>  	} else if (ps == PUD_SIZE) {
>  		hugetlb_add_hstate(PUD_SHIFT - PAGE_SHIFT);
>  	} else {
> +		hugetlb_bad_size();
>  		pr_err("hugepagesz: Unsupported page size %lu K\n", ps >> 10);
>  		return 0;
>  	}
> diff --git a/arch/metag/mm/hugetlbpage.c b/arch/metag/mm/hugetlbpage.c
> index b38700a..db1b7da 100644
> --- a/arch/metag/mm/hugetlbpage.c
> +++ b/arch/metag/mm/hugetlbpage.c
> @@ -239,6 +239,7 @@ static __init int setup_hugepagesz(char *opt)
>  	if (ps == (1 << HPAGE_SHIFT)) {
>  		hugetlb_add_hstate(HPAGE_SHIFT - PAGE_SHIFT);
>  	} else {
> +		hugetlb_bad_size();
>  		pr_err("hugepagesz: Unsupported page size %lu M\n",
>  		       ps >> 20);
>  		return 0;
> diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
> index 6dd272b..a437ff7 100644
> --- a/arch/powerpc/mm/hugetlbpage.c
> +++ b/arch/powerpc/mm/hugetlbpage.c
> @@ -772,8 +772,11 @@ static int __init hugepage_setup_sz(char *str)
>  
>  	size = memparse(str, &str);
>  
> -	if (add_huge_page_size(size) != 0)
> -		printk(KERN_WARNING "Invalid huge page size specified(%llu)\n", size);
> +	if (add_huge_page_size(size) != 0) {
> +		hugetlb_bad_size();
> +		printk(KERN_WARNING "Invalid huge page size specified(%llu)\n",
> +		       size);
> +	}
>  
>  	return 1;
>  }
> diff --git a/arch/tile/mm/hugetlbpage.c b/arch/tile/mm/hugetlbpage.c
> index e212c64..77ceaa3 100644
> --- a/arch/tile/mm/hugetlbpage.c
> +++ b/arch/tile/mm/hugetlbpage.c
> @@ -308,11 +308,16 @@ static bool saw_hugepagesz;
>  
>  static __init int setup_hugepagesz(char *opt)
>  {
> +	int rc;
> +
>  	if (!saw_hugepagesz) {
>  		saw_hugepagesz = true;
>  		memset(huge_shift, 0, sizeof(huge_shift));
>  	}
> -	return __setup_hugepagesz(memparse(opt, NULL));
> +	rc = __setup_hugepagesz(memparse(opt, NULL));
> +	if (rc)
> +		hugetlb_bad_size();
> +	return rc;
>  }
>  __setup("hugepagesz=", setup_hugepagesz);
>  
> diff --git a/arch/x86/mm/hugetlbpage.c b/arch/x86/mm/hugetlbpage.c
> index 740d7ac..3ec44f8 100644
> --- a/arch/x86/mm/hugetlbpage.c
> +++ b/arch/x86/mm/hugetlbpage.c
> @@ -165,6 +165,7 @@ static __init int setup_hugepagesz(char *opt)
>  	} else if (ps == PUD_SIZE && cpu_has_gbpages) {
>  		hugetlb_add_hstate(PUD_SHIFT - PAGE_SHIFT);
>  	} else {
> +		hugetlb_bad_size();
>  		printk(KERN_ERR "hugepagesz: Unsupported page size %lu M\n",
>  			ps >> 20);
>  		return 0;
> -- 
> 2.1.4
>