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

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?

Vlastimil

Thanks
Huang Shijie