From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vaishali Thakkar Subject: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Date: Wed, 23 Mar 2016 17:37:18 +0530 Message-ID: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> Return-path: Sender: owner-linux-mm@kvack.org List-ID: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: akpm@linux-foundation.org Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, mhocko@suse.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org, Vaishali Thakkar Current code fails to ignore the 'hugepages=' parameters when unsupported hugepagesize is specified. With this patchset, introduce new architecture independent routine hugetlb_bad_size to handle such command line options. And then call it in architecture specific code. Changes since v1: - Separated different architecture specific changes in different patches - CC'ed all arch maintainers Vaishali Thakkar (6): mm/hugetlb: Introduce hugetlb_bad_size arm64: mm: Use hugetlb_bad_size metag: mm: Use hugetlb_bad_size powerpc: mm: Use hugetlb_bad_size tile: mm: Use hugetlb_bad_size x86: mm: Use hugetlb_bad_size arch/arm64/mm/hugetlbpage.c | 1 + arch/metag/mm/hugetlbpage.c | 1 + arch/powerpc/mm/hugetlbpage.c | 6 ++++-- arch/tile/mm/hugetlbpage.c | 7 ++++++- arch/x86/mm/hugetlbpage.c | 1 + include/linux/hugetlb.h | 1 + mm/hugetlb.c | 14 +++++++++++++- 7 files changed, 27 insertions(+), 4 deletions(-) -- 2.1.4 -- 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: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Hocko Subject: Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Date: Wed, 23 Mar 2016 14:30:11 +0100 Message-ID: <20160323133011.GG7059@dhcp22.suse.cz> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <1458734844-14833-1-git-send-email-vaishali.thakkar-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org> Sender: linux-metag-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Vaishali Thakkar Cc: akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, n-horiguchi-PaJj6Psr51x8UrSeD/g0lQ@public.gmane.org, mike.kravetz-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org, hillf.zj-gPhfCIXyaqCqndwCJWfcng@public.gmane.org, baiyaowei-0p4V/sDNsUmm0O/7XYngnFaTQe2KTcn/@public.gmane.org, dingel-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, kirill.shutemov-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, paul.gortmaker-CWA4WttNNZF54TAoqtyWWQ@public.gmane.org, catalin.marinas-5wv7dgnIgG8@public.gmane.org, will.deacon-5wv7dgnIgG8@public.gmane.org, cmetcalf-d5a29ZRxExrQT0dZR+AlfA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org, linux-metag-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org, paulus-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org, mpe-Gsx/Oe8HsFggBc27wqDAHg@public.gmane.org, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org, mingo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org, x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: > Current code fails to ignore the 'hugepages=' parameters when unsupported > hugepagesize is specified. With this patchset, introduce new architecture > independent routine hugetlb_bad_size to handle such command line options. > And then call it in architecture specific code. > > Changes since v1: > - Separated different architecture specific changes in different > patches > - CC'ed all arch maintainers The hugetlb parameters parsing is a bit mess but this at least makes it behave more consistently. Feel free to add to all patches Acked-by: Michal Hocko On a side note. I have received patches with broken threading - the follow up patches are not in the single thread under this cover email. I thought this was the default behavior of git send-email but maybe your (older) version doesn't do that. --thread option would enforce that (with --no-chain-reply-to) or you can set it up in the git config. IMHO it is always better to have the patchset in the single email thread. -- Michal Hocko SUSE Labs From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vaishali Thakkar Subject: Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Date: Wed, 23 Mar 2016 21:31:39 +0530 Message-ID: <56F2BDE3.40309@oracle.com> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> <20160323133011.GG7059@dhcp22.suse.cz> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160323133011.GG7059@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Michal Hocko Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org On Wednesday 23 March 2016 07:00 PM, Michal Hocko wrote: > On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: >> Current code fails to ignore the 'hugepages=' parameters when unsupported >> hugepagesize is specified. With this patchset, introduce new architecture >> independent routine hugetlb_bad_size to handle such command line options. >> And then call it in architecture specific code. >> >> Changes since v1: >> - Separated different architecture specific changes in different >> patches >> - CC'ed all arch maintainers > The hugetlb parameters parsing is a bit mess but this at least makes it > behave more consistently. Feel free to add to all patches > Acked-by: Michal Hocko > > On a side note. I have received patches with broken threading - the > follow up patches are not in the single thread under this cover email. > I thought this was the default behavior of git send-email but maybe your > (older) version doesn't do that. --thread option would enforce that > (with --no-chain-reply-to) or you can set it up in the git config. IMHO > it is always better to have the patchset in the single email thread. > Yes, now I have set up my git config for that. Hopefully, things will work properly - patchset in a single thread from the next time. Thanks. -- Vaishali -- 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: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp1040.oracle.com (userp1040.oracle.com [156.151.31.81]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3qVTsR3VcDzDq6W for ; Wed, 23 Mar 2016 23:49:15 +1100 (AEDT) From: Vaishali Thakkar To: akpm@linux-foundation.org Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, mhocko@suse.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org, Vaishali Thakkar Subject: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Date: Wed, 23 Mar 2016 17:37:18 +0530 Message-Id: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Current code fails to ignore the 'hugepages=' parameters when unsupported hugepagesize is specified. With this patchset, introduce new architecture independent routine hugetlb_bad_size to handle such command line options. And then call it in architecture specific code. Changes since v1: - Separated different architecture specific changes in different patches - CC'ed all arch maintainers Vaishali Thakkar (6): mm/hugetlb: Introduce hugetlb_bad_size arm64: mm: Use hugetlb_bad_size metag: mm: Use hugetlb_bad_size powerpc: mm: Use hugetlb_bad_size tile: mm: Use hugetlb_bad_size x86: mm: Use hugetlb_bad_size arch/arm64/mm/hugetlbpage.c | 1 + arch/metag/mm/hugetlbpage.c | 1 + arch/powerpc/mm/hugetlbpage.c | 6 ++++-- arch/tile/mm/hugetlbpage.c | 7 ++++++- arch/x86/mm/hugetlbpage.c | 1 + include/linux/hugetlb.h | 1 + mm/hugetlb.c | 14 +++++++++++++- 7 files changed, 27 insertions(+), 4 deletions(-) -- 2.1.4 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f66.google.com (mail-wm0-f66.google.com [74.125.82.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3qVVml5XHGzDq6W for ; Thu, 24 Mar 2016 00:30:15 +1100 (AEDT) Received: by mail-wm0-f66.google.com with SMTP id r129so4417928wmr.2 for ; Wed, 23 Mar 2016 06:30:15 -0700 (PDT) Date: Wed, 23 Mar 2016 14:30:11 +0100 From: Michal Hocko To: Vaishali Thakkar Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org Subject: Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Message-ID: <20160323133011.GG7059@dhcp22.suse.cz> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: > Current code fails to ignore the 'hugepages=' parameters when unsupported > hugepagesize is specified. With this patchset, introduce new architecture > independent routine hugetlb_bad_size to handle such command line options. > And then call it in architecture specific code. > > Changes since v1: > - Separated different architecture specific changes in different > patches > - CC'ed all arch maintainers The hugetlb parameters parsing is a bit mess but this at least makes it behave more consistently. Feel free to add to all patches Acked-by: Michal Hocko On a side note. I have received patches with broken threading - the follow up patches are not in the single thread under this cover email. I thought this was the default behavior of git send-email but maybe your (older) version doesn't do that. --thread option would enforce that (with --no-chain-reply-to) or you can set it up in the git config. IMHO it is always better to have the patchset in the single email thread. -- Michal Hocko SUSE Labs From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp1040.oracle.com (userp1040.oracle.com [156.151.31.81]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3qVZ9b0qkSzDqG1 for ; Thu, 24 Mar 2016 03:03:30 +1100 (AEDT) Subject: Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize To: Michal Hocko References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> <20160323133011.GG7059@dhcp22.suse.cz> Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org From: Vaishali Thakkar Message-ID: <56F2BDE3.40309@oracle.com> Date: Wed, 23 Mar 2016 21:31:39 +0530 MIME-Version: 1.0 In-Reply-To: <20160323133011.GG7059@dhcp22.suse.cz> Content-Type: text/plain; charset=windows-1252 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wednesday 23 March 2016 07:00 PM, Michal Hocko wrote: > On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: >> Current code fails to ignore the 'hugepages=' parameters when unsupported >> hugepagesize is specified. With this patchset, introduce new architecture >> independent routine hugetlb_bad_size to handle such command line options. >> And then call it in architecture specific code. >> >> Changes since v1: >> - Separated different architecture specific changes in different >> patches >> - CC'ed all arch maintainers > The hugetlb parameters parsing is a bit mess but this at least makes it > behave more consistently. Feel free to add to all patches > Acked-by: Michal Hocko > > On a side note. I have received patches with broken threading - the > follow up patches are not in the single thread under this cover email. > I thought this was the default behavior of git send-email but maybe your > (older) version doesn't do that. --thread option would enforce that > (with --no-chain-reply-to) or you can set it up in the git config. IMHO > it is always better to have the patchset in the single email thread. > Yes, now I have set up my git config for that. Hopefully, things will work properly - patchset in a single thread from the next time. Thanks. -- Vaishali From mboxrd@z Thu Jan 1 00:00:00 1970 From: vaishali.thakkar@oracle.com (Vaishali Thakkar) Date: Wed, 23 Mar 2016 17:37:18 +0530 Subject: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Message-ID: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Current code fails to ignore the 'hugepages=' parameters when unsupported hugepagesize is specified. With this patchset, introduce new architecture independent routine hugetlb_bad_size to handle such command line options. And then call it in architecture specific code. Changes since v1: - Separated different architecture specific changes in different patches - CC'ed all arch maintainers Vaishali Thakkar (6): mm/hugetlb: Introduce hugetlb_bad_size arm64: mm: Use hugetlb_bad_size metag: mm: Use hugetlb_bad_size powerpc: mm: Use hugetlb_bad_size tile: mm: Use hugetlb_bad_size x86: mm: Use hugetlb_bad_size arch/arm64/mm/hugetlbpage.c | 1 + arch/metag/mm/hugetlbpage.c | 1 + arch/powerpc/mm/hugetlbpage.c | 6 ++++-- arch/tile/mm/hugetlbpage.c | 7 ++++++- arch/x86/mm/hugetlbpage.c | 1 + include/linux/hugetlb.h | 1 + mm/hugetlb.c | 14 +++++++++++++- 7 files changed, 27 insertions(+), 4 deletions(-) -- 2.1.4 From mboxrd@z Thu Jan 1 00:00:00 1970 From: mhocko@kernel.org (Michal Hocko) Date: Wed, 23 Mar 2016 14:30:11 +0100 Subject: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize In-Reply-To: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> Message-ID: <20160323133011.GG7059@dhcp22.suse.cz> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: > Current code fails to ignore the 'hugepages=' parameters when unsupported > hugepagesize is specified. With this patchset, introduce new architecture > independent routine hugetlb_bad_size to handle such command line options. > And then call it in architecture specific code. > > Changes since v1: > - Separated different architecture specific changes in different > patches > - CC'ed all arch maintainers The hugetlb parameters parsing is a bit mess but this at least makes it behave more consistently. Feel free to add to all patches Acked-by: Michal Hocko On a side note. I have received patches with broken threading - the follow up patches are not in the single thread under this cover email. I thought this was the default behavior of git send-email but maybe your (older) version doesn't do that. --thread option would enforce that (with --no-chain-reply-to) or you can set it up in the git config. IMHO it is always better to have the patchset in the single email thread. -- Michal Hocko SUSE Labs From mboxrd@z Thu Jan 1 00:00:00 1970 From: vaishali.thakkar@oracle.com (Vaishali Thakkar) Date: Wed, 23 Mar 2016 21:31:39 +0530 Subject: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize In-Reply-To: <20160323133011.GG7059@dhcp22.suse.cz> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> <20160323133011.GG7059@dhcp22.suse.cz> Message-ID: <56F2BDE3.40309@oracle.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 23 March 2016 07:00 PM, Michal Hocko wrote: > On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: >> Current code fails to ignore the 'hugepages=' parameters when unsupported >> hugepagesize is specified. With this patchset, introduce new architecture >> independent routine hugetlb_bad_size to handle such command line options. >> And then call it in architecture specific code. >> >> Changes since v1: >> - Separated different architecture specific changes in different >> patches >> - CC'ed all arch maintainers > The hugetlb parameters parsing is a bit mess but this at least makes it > behave more consistently. Feel free to add to all patches > Acked-by: Michal Hocko > > On a side note. I have received patches with broken threading - the > follow up patches are not in the single thread under this cover email. > I thought this was the default behavior of git send-email but maybe your > (older) version doesn't do that. --thread option would enforce that > (with --no-chain-reply-to) or you can set it up in the git config. IMHO > it is always better to have the patchset in the single email thread. > Yes, now I have set up my git config for that. Hopefully, things will work properly - patchset in a single thread from the next time. Thanks. -- Vaishali From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f44.google.com (mail-wm0-f44.google.com [74.125.82.44]) by kanga.kvack.org (Postfix) with ESMTP id A6E6C6B007E for ; Wed, 23 Mar 2016 09:30:32 -0400 (EDT) Received: by mail-wm0-f44.google.com with SMTP id l68so234064753wml.0 for ; Wed, 23 Mar 2016 06:30:32 -0700 (PDT) Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com. [74.125.82.65]) by mx.google.com with ESMTPS id l65si3784152wmb.26.2016.03.23.06.30.12 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 23 Mar 2016 06:30:12 -0700 (PDT) Received: by mail-wm0-f65.google.com with SMTP id r129so4417919wmr.2 for ; Wed, 23 Mar 2016 06:30:12 -0700 (PDT) Date: Wed, 23 Mar 2016 14:30:11 +0100 From: Michal Hocko Subject: Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Message-ID: <20160323133011.GG7059@dhcp22.suse.cz> References: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1458734844-14833-1-git-send-email-vaishali.thakkar@oracle.com> Sender: owner-linux-mm@kvack.org List-ID: To: Vaishali Thakkar Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, n-horiguchi@ah.jp.nec.com, mike.kravetz@oracle.com, hillf.zj@alibaba-inc.com, baiyaowei@cmss.chinamobile.com, dingel@linux.vnet.ibm.com, kirill.shutemov@linux.intel.com, dave.hansen@linux.intel.com, paul.gortmaker@windriver.com, catalin.marinas@arm.com, will.deacon@arm.com, cmetcalf@ezchip.com, linux-arm-kernel@lists.infradead.org, james.hogan@imgtec.com, linux-metag@vger.kernel.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, linuxppc-dev@lists.ozlabs.org, tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, x86@kernel.org On Wed 23-03-16 17:37:18, Vaishali Thakkar wrote: > Current code fails to ignore the 'hugepages=' parameters when unsupported > hugepagesize is specified. With this patchset, introduce new architecture > independent routine hugetlb_bad_size to handle such command line options. > And then call it in architecture specific code. > > Changes since v1: > - Separated different architecture specific changes in different > patches > - CC'ed all arch maintainers The hugetlb parameters parsing is a bit mess but this at least makes it behave more consistently. Feel free to add to all patches Acked-by: Michal Hocko On a side note. I have received patches with broken threading - the follow up patches are not in the single thread under this cover email. I thought this was the default behavior of git send-email but maybe your (older) version doesn't do that. --thread option would enforce that (with --no-chain-reply-to) or you can set it up in the git config. IMHO it is always better to have the patchset in the single email thread. -- Michal Hocko SUSE Labs -- 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: email@kvack.org