On 12/14/2015 04:26 PM, Joonsoo Kim wrote:
2015-12-14 19:07 GMT+09:00 Vlastimil Babka <vbabka@xxxxxxx>:
On 12/14/2015 06:02 AM, Joonsoo Kim wrote:
Acked-by: Vlastimil Babka <vbabka@xxxxxxx>
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, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>