From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760282AbcAKNv0 (ORCPT ); Mon, 11 Jan 2016 08:51:26 -0500 Received: from mx2.suse.de ([195.135.220.15]:54772 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759550AbcAKNvX (ORCPT ); Mon, 11 Jan 2016 08:51:23 -0500 Subject: Re: [PATCH] mm/page_alloc: remove unused struct zone *z variable To: "Kirill A. Shutemov" , Alexander Kuleshov , Robin Holt References: <1452239948-1012-1-git-send-email-kuleshovmail@gmail.com> <20160108232942.GB13046@node.shutemov.name> Cc: Andrew Morton , Mel Gorman , Michal Hocko , David Rientjes , Joonsoo Kim , Yaowei Bai , Xishi Qiu , Alexander Duyck , "Kirill A . Shutemov" , Johannes Weiner , linux-mm@kvack.org, linux-kernel@vger.kernel.org From: Vlastimil Babka Message-ID: <5693B347.7050905@suse.cz> Date: Mon, 11 Jan 2016 14:51:03 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20160108232942.GB13046@node.shutemov.name> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/09/2016 12:29 AM, Kirill A. Shutemov wrote: > On Fri, Jan 08, 2016 at 01:59:08PM +0600, Alexander Kuleshov wrote: >> This patch removes unused struct zone *z variable which is >> appeared in 86051ca5eaf5 (mm: fix usemap initialization) > > I guess it's a fix for 1e8ce83cd17f (mm: meminit: move page initialization > into a separate function). Yeah but it's not a bug, so a tag like that would be just noise. >> >> Signed-off-by: Alexander Kuleshov > > Acked-by: Kirill A. Shutemov Acked-by: Vlastimil Babka >> --- >> mm/page_alloc.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/mm/page_alloc.c b/mm/page_alloc.c >> index 9d666df..9bde098 100644 >> --- a/mm/page_alloc.c >> +++ b/mm/page_alloc.c >> @@ -4471,13 +4471,11 @@ void __meminit memmap_init_zone(unsigned long size, int nid, unsigned long zone, >> pg_data_t *pgdat = NODE_DATA(nid); >> unsigned long end_pfn = start_pfn + size; >> unsigned long pfn; >> - struct zone *z; >> unsigned long nr_initialised = 0; >> >> if (highest_memmap_pfn < end_pfn - 1) >> highest_memmap_pfn = end_pfn - 1; >> >> - z = &pgdat->node_zones[zone]; >> for (pfn = start_pfn; pfn < end_pfn; pfn++) { >> /* >> * There can be holes in boot-time mem_map[]s >> -- >> 2.6.2.485.g1bc8fea >> >> -- >> To unsubscribe, send a message with 'unsubscribe linux-mm' in >> the body to majordomo@kvack.org. For more info on Linux MM, >> see: http://www.linux-mm.org/ . >> Don't email: email@kvack.org >