From mboxrd@z Thu Jan 1 00:00:00 1970 From: kirill@shutemov.name (Kirill A. Shutemov) Date: Wed, 8 May 2013 15:44:17 +0300 Subject: [RFC PATCH v2 05/11] mm: thp: Correct the HPAGE_PMD_ORDER check. In-Reply-To: <1368006763-30774-6-git-send-email-steve.capper@linaro.org> References: <1368006763-30774-1-git-send-email-steve.capper@linaro.org> <1368006763-30774-6-git-send-email-steve.capper@linaro.org> Message-ID: <20130508124417.GA29631@shutemov.name> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, May 08, 2013 at 10:52:37AM +0100, Steve Capper wrote: > All Transparent Huge Pages are allocated by the buddy allocator. > > A compile time check is in place that fails when the order of a > transparent huge page is too large to be allocated by the buddy > allocator. Unfortunately that compile time check passes when: > HPAGE_PMD_ORDER == MAX_ORDER > ( which is incorrect as the buddy allocator can only allocate > memory of order strictly less than MAX_ORDER. ) It looks confusing to me. Shouldn't we fix what MAX_ORDER means instead? -- Kirill A. Shutemov