Messages in this thread | | | Date | Wed, 11 Jun 2014 11:38:30 +0800 | From | Zhang Yanfei <> | Subject | Re: [PATCH 07/10] mm: rename allocflags_to_migratetype for clarity |
| |
On 06/11/2014 10:41 AM, Minchan Kim wrote: > On Mon, Jun 09, 2014 at 11:26:19AM +0200, Vlastimil Babka wrote: >> From: David Rientjes <rientjes@google.com> >> >> The page allocator has gfp flags (like __GFP_WAIT) and alloc flags (like >> ALLOC_CPUSET) that have separate semantics. >> >> The function allocflags_to_migratetype() actually takes gfp flags, not alloc >> flags, and returns a migratetype. Rename it to gfpflags_to_migratetype(). >> >> Signed-off-by: David Rientjes <rientjes@google.com> >> Signed-off-by: Vlastimil Babka <vbabka@suse.cz> > > I was one of person who got confused sometime.
Some names in MM really make people confused. But sometimes thinking an appropriate name is also a hard thing. Like I once wanted to change the name of function nr_free_zone_pages() and also nr_free_buffer_pages(). But it is hard to name them, so at last Andrew suggested to add the detailed function description to make it clear only.
Reviewed-by: Zhang Yanfei <zhangyanfei@cn.fujitsu.com>
> > Acked-by: Minchan Kim <minchan@kernel.org> >
-- Thanks. Zhang Yanfei
| |