From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752224AbbJMHXK (ORCPT ); Tue, 13 Oct 2015 03:23:10 -0400 Received: from mx2.suse.de ([195.135.220.15]:50243 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751769AbbJMHXJ (ORCPT ); Tue, 13 Oct 2015 03:23:09 -0400 Subject: Re: [PATCH V2] mm, page_alloc: reserve pageblocks for high-order atomic allocations on demand -fix To: yalin wang , akpm@linux-foundation.org, mgorman@techsingularity.net, mhocko@suse.com, rientjes@google.com, js1304@gmail.com, kirill.shutemov@linux.intel.com, hannes@cmpxchg.org, alexander.h.duyck@redhat.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <1444700544-22666-1-git-send-email-yalin.wang2010@gmail.com> From: Vlastimil Babka Message-ID: <561CB159.8060409@suse.cz> Date: Tue, 13 Oct 2015 09:23:05 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <1444700544-22666-1-git-send-email-yalin.wang2010@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/13/2015 03:42 AM, yalin wang wrote: > There is a redundant check and a memory leak introduced by a patch in > mmotm. This patch removes an unlikely(order) check as we are sure order > is not zero at the time. It also checks if a page is already allocated > to avoid a memory leak. > > This is a fix to the mmotm patch > mm-page_alloc-reserve-pageblocks-for-high-order-atomic-allocations-on-demand.patch > > Signed-off-by: yalin wang > Acked-by: Mel Gorman Acked-by: Vlastimil Babka > --- > mm/page_alloc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index 0d6f540..043b691 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -2241,13 +2241,13 @@ struct page *buffered_rmqueue(struct zone *preferred_zone, > spin_lock_irqsave(&zone->lock, flags); > > page = NULL; > - if (unlikely(order) && (alloc_flags & ALLOC_HARDER)) { > + if (alloc_flags & ALLOC_HARDER) { > page = __rmqueue_smallest(zone, order, MIGRATE_HIGHATOMIC); > if (page) > trace_mm_page_alloc_zone_locked(page, order, migratetype); > } > - > - page = __rmqueue(zone, order, migratetype, gfp_flags); > + if (!page) > + page = __rmqueue(zone, order, migratetype, gfp_flags); > spin_unlock(&zone->lock); > if (!page) > goto failed; >