On Tue, Nov 29, 2016 at 11:44:23AM +0100, Vlastimil Babka wrote: > On 11/29/2016 09:53 AM, Huang Shijie wrote: > > On Mon, Nov 28, 2016 at 02:29:03PM +0100, Vlastimil Babka wrote: > > > On 11/14/2016 08:07 AM, Huang Shijie wrote: > > > > static inline bool gigantic_page_supported(void) { return true; } > > > > #else > > > > +static inline struct page *alloc_gigantic_page(int nid, unsigned int order) > > > > +{ > > > > + return NULL; > > > > +} > > > > > > This hunk is not explained by the description. Could belong to a later > > > patch? > > > > > > > Okay, I can create an extra patch to add the description for the > > alloc_gigantic_page(). > > Not sure about extra patch, just move it to an existing later patch that > relies on it? The whole patch set has been merged to Andrew's tree, so an extra patch is better. :) Thanks Huang Shijie -- 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>