[PATCH 08/15] mm: mmzone: MIGRATE_CMA migration type added
Hillf Danton
dhillf at gmail.com
Sat Feb 4 04:09:02 EST 2012
2012/2/3 Michal Nazarewicz <mina86 at mina86.com>:
>>> +static inline bool migrate_async_suitable(int migratetype)
>
> On Fri, 03 Feb 2012 15:19:54 +0100, Hillf Danton <dhillf at gmail.com> wrote:
>>
>> Just nitpick, since the helper is not directly related to what async
>> means,
>> how about migrate_suitable(int migrate_type) ?
>
>
> I feel current name is better suited since it says that it's OK to scan this
> block if it's an asynchronous compaction run.
>
The input is the migrate type of page considered, and the async is only one
of the modes that compaction should be carried out. Plus the helper is
also used in other cases where async is entirely not concerned.
That said, the naming is not clear, if not misleading.
More information about the linux-arm-kernel
mailing list