Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759892Ab3DBDLt (ORCPT ); Mon, 1 Apr 2013 23:11:49 -0400 Received: from mx1.redhat.com ([209.132.183.28]:8173 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759360Ab3DBDLs (ORCPT ); Mon, 1 Apr 2013 23:11:48 -0400 Message-ID: <515A4BD1.1020407@redhat.com> Date: Tue, 02 Apr 2013 11:09:05 +0800 From: Zhouping Liu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: David Rientjes CC: Andrea Arcangeli , Hugh Dickins , Mel Gorman , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Amos Kong Subject: Re: THP: AnonHugePages in /proc/[pid]/smaps is correct or not? References: <383590596.664138.1364803227470.JavaMail.root@redhat.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3049 Lines: 96 On 04/02/2013 06:23 AM, David Rientjes wrote: > On Mon, 1 Apr 2013, Zhouping Liu wrote: > >> Hi all, >> >> I found THP can't correctly distinguish one anonymous hugepage map. >> >> 1. when /sys/kernel/mm/transparent_hugepage/enabled is 'always', the >> amount of THP always is one less. >> > It's not a problem with identifying an anonymous mapping as a hugepage, > setting thp enabled to "always" does not guarantee that they will always > be allocatable or that your mmap() will be 2MB aligned. Your sample code > is using mmap() instead of posix_memalign() so you'll probably only get > 100% hugepages only 1/512th of the time. I don't understand clearly the last sentence 'you'll probably only get 100% hugepages only 1/512th of the time.' could you please explain more details about 'only 1/512th of the time'? > >> 2. when /sys/kernel/mm/transparent_hugepage/enabled is 'madvise', THP can't >> distinguish any one anonymous hugepage size: >> >> Testing code: >> -------- snip -------- >> unsigned long hugepagesize = (1UL << 21); >> >> int main() >> { >> void *addr; >> int i; >> >> printf("pid is %d\n", getpid()); >> >> for (i = 0; i < 5; i++) { >> addr = mmap(NULL, hugepagesize, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANON, -1, 0); >> >> if (addr == MAP_FAILED) { >> perror("mmap"); >> return -1; >> } >> >> if (madvise(addr, hugepagesize, MADV_HUGEPAGE) == -1) { >> perror("madvise"); >> return -1; >> } >> >> memset(addr, i, hugepagesize); >> } >> >> sleep(50); >> >> return 0; >> } >> --------- snip ---------- >> >> The result is that it can't find any AnonHugePages from /proc/[pid]/smaps : >> -------------- snip ------- >> 7f0b38cd0000-7f0b396d0000 rw-p 00000000 00:00 0 >> Size: 10240 kB >> Rss: 10240 kB >> Pss: 10240 kB >> Shared_Clean: 0 kB >> Shared_Dirty: 0 kB >> Private_Clean: 0 kB >> Private_Dirty: 10240 kB >> Referenced: 10240 kB >> Anonymous: 10240 kB >> AnonHugePages: 0 kB >> Swap: 0 kB >> KernelPageSize: 4 kB >> MMUPageSize: 4 kB >> Locked: 0 kB >> VmFlags: rd wr mr mw me ac > "hg" would be shown in VmFlags if your MADV_HUGEPAGE was successful, are > you sure this is the right vma? I think it's the same issue as the above, according to the sample code, it does mmap() five times in total, and each time it map 2MB anonymous maps, as all the maps maybe aren't 2MB aligned, so "AnonHugePages" show 0 kB, and no "hg" VmFalgs. so, again, if I understand correctly, thp should tune the naturally aligned maps, such as generated by mmap()/malloc(), make such maps 'hugepagesize' aligned if the maps or vma is equal and greater than 'hugepagesize', doesn't it? Thanks, Zhouping -- 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/