[Top] [All Lists]

Re: [PATCH] hugetlb: Provide a default HPAGE_SHIFT if !CONFIG_HUGETLB_PA

To: David Rientjes <>, "" <>
Subject: Re: [PATCH] hugetlb: Provide a default HPAGE_SHIFT if !CONFIG_HUGETLB_PAGE
From: David Daney <>
Date: Wed, 16 Nov 2011 14:09:12 -0800
Cc: David Daney <>, "" <>, William Irwin <>, Andrew Morton <>, "" <>, David Daney <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=xBWuxsEGLH+emo1Jo3MQwkEdicuv1Arl5KifMFUU4sY=; b=VHLA/HtWZuK4YKNiJxBJF+3aMQXZrq0bWHKsPVNU0mrrGJgU/2uSa5wSRAFz+QeFxm R1j33Rg3NOVivBX88pwPDtvRG+ffWKz8TGxXFoCP83IRG05XulgaT5dUTP8tvxOJ3NER DfWEBjelK+0pziGJqCnGqRrU9Z1iiLeHChXd8=
In-reply-to: <>
References: <> <>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20101027 Fedora/3.0.10-1.fc12 Thunderbird/3.0.10
On 11/16/2011 01:32 PM, David Rientjes wrote:
On Wed, 16 Nov 2011, David Daney wrote:

From: David Daney<>

This is required now to get MIPS kernels to compile with


I should have been more specific. The failure is in Ralf's mips-for-linux-next branch.

 Apparently there's some config option you've enabled that is causing
it to fail but I can't find it.  defconfig works fine on my mips
crosscompiler and allyesconfig is borked already in other ways.

Please look in the mips-for-linux-next branch.

This is definitely the wrong fix, anyway, and it would require a change to
arch/mips/include/asm/page.h instead since it's localized to mips,

No, all we are doing is supplying a dummy definition for HPAGE_SHIFT as we currently have for HPAGE_SIZE and HPAGE_MASK.

so nack.

Signed-off-by: David Daney<>
  include/linux/hugetlb.h |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h
index 19644e0..746d543 100644
--- a/include/linux/hugetlb.h
+++ b/include/linux/hugetlb.h
@@ -113,6 +113,7 @@ static inline void copy_huge_page(struct page *dst, struct 
page *src)
  #ifndef HPAGE_MASK
  #define HPAGE_MASK    PAGE_MASK               /* Keep the compiler happy */

Why didn't you NACK the addition of these two lines too?

Following your logic, we should remove these and patch up all the architecture specific files instead.

David Daney


  #endif /* !CONFIG_HUGETLB_PAGE */

<Prev in Thread] Current Thread [Next in Thread>