On Tue, Nov 29, 2016 at 11:44:23AM +0100, Vlastimil Babka wrote:

On 11/29/2016 09:53 AM, Huang Shijie wrote:

quoted

On Mon, Nov 28, 2016 at 02:29:03PM +0100, Vlastimil Babka wrote:

quoted

On 11/14/2016 08:07 AM, Huang Shijie wrote:

quoted

 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