Re: [PATCH] x86: Fix adjust_range_size_mask calling position

2013-05-28 Thread H. Peter Anvin
On 05/28/2013 04:43 PM, Yinghai Lu wrote: >> >> Sorry, I'm not sure I understand what the above paragraph is trying to say. > > We merge the continuous range with same page size allow too early. > in the case > [mem 0x0020-0x3fff] page 2M > [mem 0x4000-0x7bff] page 2M > a

Re: [PATCH] x86: Fix adjust_range_size_mask calling position

2013-05-28 Thread Yinghai Lu
On Tue, May 28, 2013 at 4:36 PM, H. Peter Anvin wrote: > On 05/28/2013 04:28 PM, Yinghai Lu wrote: >> >> Actually there is bug about calling sequence for >> adjust_range_page_size_mask(). Merge first will make >> adjust to 1g for second partial 1g range fail. >> > > Sorry, I'm not sure I understan

Re: [PATCH] x86: Fix adjust_range_size_mask calling position

2013-05-28 Thread H. Peter Anvin
On 05/28/2013 04:28 PM, Yinghai Lu wrote: > > Actually there is bug about calling sequence for > adjust_range_page_size_mask(). Merge first will make > adjust to 1g for second partial 1g range fail. > Sorry, I'm not sure I understand what the above paragraph is trying to say. -hpa --