Re: [PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-04-10 Thread Mike Kravetz
On 04/10/2017 02:38 PM, Vegard Nossum wrote: > On 29 March 2017 at 23:08, Mike Kravetz wrote: >> Changes to hugetlbfs reservation maps is a two step process. The first >> step is a call to region_chg to determine what needs to be changed, and >> prepare that change. This should be followed by a

Re: [PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-04-10 Thread Vegard Nossum
On 29 March 2017 at 23:08, Mike Kravetz wrote: > Changes to hugetlbfs reservation maps is a two step process. The first > step is a call to region_chg to determine what needs to be changed, and > prepare that change. This should be followed by a call to call to > region_add to commit the change,

Re: [PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-03-30 Thread Mike Kravetz
On 03/30/2017 05:28 AM, Dmitry Vyukov wrote: > On Wed, Mar 29, 2017 at 11:17 PM, Andrew Morton > wrote: >> On Wed, 29 Mar 2017 14:08:02 -0700 Mike Kravetz >> wrote: >> >>> >>> syzkaller fuzzer found this bug, that resulted in the following: >> >> I'll change the above to >> >> : syzkaller fuzzer

Re: [PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-03-30 Thread Dmitry Vyukov
On Wed, Mar 29, 2017 at 11:17 PM, Andrew Morton wrote: > On Wed, 29 Mar 2017 14:08:02 -0700 Mike Kravetz > wrote: > >> Resending because of typo in Andrew's e-mail when first sent >> >> Changes to hugetlbfs reservation maps is a two step process. The first >> step is a call to region_chg to det

Re: [PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-03-29 Thread Andrew Morton
On Wed, 29 Mar 2017 14:08:02 -0700 Mike Kravetz wrote: > Resending because of typo in Andrew's e-mail when first sent > > Changes to hugetlbfs reservation maps is a two step process. The first > step is a call to region_chg to determine what needs to be changed, and > prepare that change. This

[PATCH RESEND] mm/hugetlb: Don't call region_abort if region_chg fails

2017-03-29 Thread Mike Kravetz
Resending because of typo in Andrew's e-mail when first sent Changes to hugetlbfs reservation maps is a two step process. The first step is a call to region_chg to determine what needs to be changed, and prepare that change. This should be followed by a call to call to region_add to commit the c