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


Groups > linux.kernel > #1363401 > unrolled thread

[PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize

Started byVaishali Thakkar <vaishali.thakkar@oracle.com>
First post2016-03-23 13:50 +0100
Last post2016-03-23 17:10 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize Vaishali Thakkar <vaishali.thakkar@oracle.com> - 2016-03-23 13:50 +0100
    Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for  invalid hugepagesize Michal Hocko <mhocko@kernel.org> - 2016-03-23 14:40 +0100
      Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for  invalid hugepagesize Vaishali Thakkar <vaishali.thakkar@oracle.com> - 2016-03-23 17:10 +0100

#1363401 — [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize

FromVaishali Thakkar <vaishali.thakkar@oracle.com>
Date2016-03-23 13:50 +0100
Subject[PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize
Message-ID<rfQox-3pU-3@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1363423 — Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize

FromMichal Hocko <mhocko@kernel.org>
Date2016-03-23 14:40 +0100
SubjectRe: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize
Message-ID<rfRaV-42Z-3@gated-at.bofh.it>
In reply to#1363401
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 <mhocko@suse.com>

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

[toc] | [prev] | [next] | [standalone]


#1363514 — Re: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize

FromVaishali Thakkar <vaishali.thakkar@oracle.com>
Date2016-03-23 17:10 +0100
SubjectRe: [PATCH v2 0/6] mm/hugetlb: Fix commandline parsing behavior for invalid hugepagesize
Message-ID<rfTw5-5WF-11@gated-at.bofh.it>
In reply to#1363423

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 <mhocko@suse.com>
>
> 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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web