On Wed, 31 Jan 2018, Anshuman Khandual wrote: > On 01/30/2018 08:06 PM, Michal Hocko wrote: > > On Tue 30-01-18 10:36:42, Anshuman Khandual wrote: > >> Allocation helper functions for migrate_pages() remmain scattered with > >> similar names making them really confusing. Rename these functions based > >> on the context for the migration and move them all into common migration > >> header. Functionality remains unchanged. I agree that their names could be made less confusing (though didn't succeed very well when I tried); and maybe a couple of them are general enough to be used from more than one callsite, and could well live in mm/migrate.c. But moving all of page migration's (currently static) new_page allocator functions away from the code that relies on their special characteristics (probably relayed to them through a private argument), and into a single header file, just seems perverse to me. And likely to be a nuisance when adding more in future: private structures having to be made public just to make them visible in that shared header file. Would it make sense to keep the various functions that may be called by rmap_walk() together in one rmap_walk.h? The different filesystems' writepage methods together in one writepage.h? I don't think so. Hugh -- 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>
