Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-21 Thread Gerald Schaefer
On Fri, 17 Apr 2020 11:50:45 -0700 Mike Kravetz wrote: > v3 - >Used weak attribute method of defining arch_hugetlb_valid_size. > This eliminates changes to arch specific hugetlb.h files (Peter) >Updated documentation (Peter, Randy) >Fixed handling of implicitly specified gigantic

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Will Deacon
On Mon, Apr 20, 2020 at 02:40:05PM -0700, Mike Kravetz wrote: > On 4/20/20 1:29 PM, Anders Roxell wrote: > > On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: > >> On 4/20/20 8:34 AM, Qian Cai wrote: > >>> > >>> Reverted this series fixed many undefined behaviors on arm64 with the > >>> config, >

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Anders Roxell
On Mon, 20 Apr 2020 at 23:43, Mike Kravetz wrote: > > On 4/20/20 1:29 PM, Anders Roxell wrote: > > On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: > >> On 4/20/20 8:34 AM, Qian Cai wrote: > >>> > >>> Reverted this series fixed many undefined behaviors on arm64 with the > >>> config, > >> While

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Mike Kravetz
On 4/20/20 1:29 PM, Anders Roxell wrote: > On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: >> On 4/20/20 8:34 AM, Qian Cai wrote: >>> >>> Reverted this series fixed many undefined behaviors on arm64 with the >>> config, >> While rearranging the code (patch 3 in series), I made the incorrect >>

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Anders Roxell
On Mon, 20 Apr 2020 at 20:23, Mike Kravetz wrote: > > On 4/20/20 8:34 AM, Qian Cai wrote: > > > > > >> On Apr 17, 2020, at 2:50 PM, Mike Kravetz wrote: > >> > >> Longpeng(Mike) reported a weird message from hugetlb command line > >> processing > >> and proposed a solution [1]. While the propose

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Will Deacon
On Mon, Apr 20, 2020 at 11:20:23AM -0700, Mike Kravetz wrote: > On 4/20/20 8:34 AM, Qian Cai wrote: > > > > > >> On Apr 17, 2020, at 2:50 PM, Mike Kravetz wrote: > >> > >> Longpeng(Mike) reported a weird message from hugetlb command line > >> processing > >> and proposed a solution [1]. While

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Mike Kravetz
On 4/20/20 8:34 AM, Qian Cai wrote: > > >> On Apr 17, 2020, at 2:50 PM, Mike Kravetz wrote: >> >> Longpeng(Mike) reported a weird message from hugetlb command line processing >> and proposed a solution [1]. While the proposed patch does address the >> specific issue, there are other related iss

Re: [PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-20 Thread Qian Cai
> On Apr 17, 2020, at 2:50 PM, Mike Kravetz wrote: > > Longpeng(Mike) reported a weird message from hugetlb command line processing > and proposed a solution [1]. While the proposed patch does address the > specific issue, there are other related issues in command line processing. > As hugetl

[PATCH v3 0/4] Clean up hugetlb boot command line processing

2020-04-17 Thread Mike Kravetz
v3 - Used weak attribute method of defining arch_hugetlb_valid_size. This eliminates changes to arch specific hugetlb.h files (Peter) Updated documentation (Peter, Randy) Fixed handling of implicitly specified gigantic page preallocation in existing code and removed documentation