Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755548Ab1FTRZQ (ORCPT ); Mon, 20 Jun 2011 13:25:16 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42184 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753625Ab1FTRZN (ORCPT ); Mon, 20 Jun 2011 13:25:13 -0400 Message-ID: <4DFF8271.20301@redhat.com> Date: Tue, 21 Jun 2011 01:25:05 +0800 From: Cong Wang User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc14 Thunderbird/3.1.10 MIME-Version: 1.0 To: Andrea Arcangeli CC: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Rik van Riel , Johannes Weiner , KAMEZAWA Hiroyuki , linux-mm@kvack.org Subject: Re: [PATCH 3/3] mm: print information when THP is disabled automatically References: <1308587683-2555-1-git-send-email-amwang@redhat.com> <1308587683-2555-3-git-send-email-amwang@redhat.com> <20110620165425.GF20843@redhat.com> In-Reply-To: <20110620165425.GF20843@redhat.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1643 Lines: 39 于 2011年06月21日 00:54, Andrea Arcangeli 写道: > On Tue, Jun 21, 2011 at 12:34:30AM +0800, Amerigo Wang wrote: >> Print information when THP is disabled automatically so that >> users can find this info in dmesg. >> >> Signed-off-by: WANG Cong >> --- >> mm/huge_memory.c | 5 ++++- >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/mm/huge_memory.c b/mm/huge_memory.c >> index 7fb44cc..07679da 100644 >> --- a/mm/huge_memory.c >> +++ b/mm/huge_memory.c >> @@ -544,8 +544,11 @@ static int __init hugepage_init(void) >> * where the extra memory used could hurt more than TLB overhead >> * is likely to save. The admin can still enable it through /sys. >> */ >> - if (totalram_pages< (CONFIG_TRANSPARENT_HUGEPAGE_THRESHOLD<< (20 - PAGE_SHIFT))) >> + if (totalram_pages< (CONFIG_TRANSPARENT_HUGEPAGE_THRESHOLD >> + << (20 - PAGE_SHIFT))) { >> + printk(KERN_INFO "hugepage: disabled auotmatically\n"); > > typo automatically. I'd suggest to change the prefix from "hugepage:" > to "THP:" to avoid the risk of possible confusion with hugetlbfs > support. Maybe you could print the minimal threshold too ("disabled > automatically with less than %dMB of RAM"). Well, the "hugepage:" prefix is copied from other printk messages in the same function. ;-) Yeah, it would be nice to print the threshold too. Thanks for your reply. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/