Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933278AbbLOIbn (ORCPT ); Tue, 15 Dec 2015 03:31:43 -0500 Received: from mx2.suse.de ([195.135.220.15]:56402 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932078AbbLOIbm (ORCPT ); Tue, 15 Dec 2015 03:31:42 -0500 Subject: Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn To: Joonsoo Kim References: <1450069341-28875-1-git-send-email-iamjoonsoo.kim@lge.com> <566E94C6.5080000@suse.cz> Cc: Andrew Morton , Aaron Lu , Mel Gorman , Rik van Riel , David Rientjes , LKML , Linux Memory Management List , Joonsoo Kim From: Vlastimil Babka Message-ID: <566FCFEB.1020305@suse.cz> Date: Tue, 15 Dec 2015 09:31:39 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: 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 Content-Length: 1386 Lines: 30 On 12/14/2015 04:26 PM, Joonsoo Kim wrote: > 2015-12-14 19:07 GMT+09:00 Vlastimil Babka : >> On 12/14/2015 06:02 AM, Joonsoo Kim wrote: >>> >> >> Acked-by: Vlastimil Babka >> >> Note that until now in compaction we've used basically an open-coded >> round_down(), and ALIGN() for rounding up. You introduce a first use of >> round_down(), and it would be nice to standardize on round_down() and >> round_up() everywhere. I think it's more obvious than open-coding and >> ALIGN() (which doesn't tell the reader if it's aligning up or down). >> Hopefully they really do the same thing and there are no caveats... > > Okay. Will send another patch for this clean-up on next spin. Great, I didn't mean that the cleanup is needed right now, but whether we agree on an idiom to use whenever doing any changes from now on. Maybe it would be best to add some defines in the top of compaction.c that would also hide away the repeated pageblock_nr_pages everywhere? Something like: #define pageblock_start(pfn) round_down(pfn, pageblock_nr_pages) #define pageblock_end(pfn) round_up((pfn)+1, pageblock_nr_pages) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/