Re: [PATCH v2 2/8] mm/page_alloc: correct to clear guard attribute in DEBUG_PAGEALLOC

2014-08-13 Thread Joonsoo Kim
On Tue, Aug 12, 2014 at 01:45:23AM +, Minchan Kim wrote: > On Wed, Aug 06, 2014 at 04:18:30PM +0900, Joonsoo Kim wrote: > > In __free_one_page(), we check the buddy page if it is guard page. > > And, if so, we should clear guard attribute on the buddy page. But, > > currently, we clear original

Re: [PATCH v2 2/8] mm/page_alloc: correct to clear guard attribute in DEBUG_PAGEALLOC

2014-08-11 Thread Minchan Kim
On Wed, Aug 06, 2014 at 04:18:30PM +0900, Joonsoo Kim wrote: > In __free_one_page(), we check the buddy page if it is guard page. > And, if so, we should clear guard attribute on the buddy page. But, > currently, we clear original page's order rather than buddy one's. > This doesn't have any proble

[PATCH v2 2/8] mm/page_alloc: correct to clear guard attribute in DEBUG_PAGEALLOC

2014-08-06 Thread Joonsoo Kim
In __free_one_page(), we check the buddy page if it is guard page. And, if so, we should clear guard attribute on the buddy page. But, currently, we clear original page's order rather than buddy one's. This doesn't have any problem, because resetting buddy's order is useless and the original page's