From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754868AbcARLkw (ORCPT ); Mon, 18 Jan 2016 06:40:52 -0500 Received: from mail-wm0-f46.google.com ([74.125.82.46]:37206 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754541AbcARLkq (ORCPT ); Mon, 18 Jan 2016 06:40:46 -0500 Date: Mon, 18 Jan 2016 13:40:43 +0200 From: "Kirill A. Shutemov" To: Geert Uytterhoeven Cc: "Kirill A. Shutemov" , Andrew Morton , Andrea Arcangeli , Hugh Dickins , Dave Hansen , Mel Gorman , Rik van Riel , Vlastimil Babka , Christoph Lameter , Naoya Horiguchi , Steve Capper , "Aneesh Kumar K.V" , Johannes Weiner , Michal Hocko , Jerome Marchand , Sasha Levin , "linux-kernel@vger.kernel.org" , Linux MM Subject: Re: BUILD_BUG() in smaps_account() (was: Re: [PATCHv12 01/37] mm, proc: adjust PSS calculation) Message-ID: <20160118114043.GA14531@node.shutemov.name> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 18, 2016 at 11:09:00AM +0100, Geert Uytterhoeven wrote: > Hi Kirill, > > On Tue, Oct 6, 2015 at 5:23 PM, Kirill A. Shutemov > wrote: > > With new refcounting all subpages of the compound page are not necessary > > have the same mapcount. We need to take into account mapcount of every > > sub-page. > > > > Signed-off-by: Kirill A. Shutemov > > Tested-by: Sasha Levin > > Tested-by: Aneesh Kumar K.V > > Acked-by: Jerome Marchand > > Acked-by: Vlastimil Babka > > --- > > fs/proc/task_mmu.c | 47 +++++++++++++++++++++++++++++++---------------- > > 1 file changed, 31 insertions(+), 16 deletions(-) > > > > diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c > > index bd167675a06f..ace02a4a07db 100644 > > --- a/fs/proc/task_mmu.c > > +++ b/fs/proc/task_mmu.c > > @@ -454,9 +454,10 @@ struct mem_size_stats { > > }; > > > > static void smaps_account(struct mem_size_stats *mss, struct page *page, > > - unsigned long size, bool young, bool dirty) > > + bool compound, bool young, bool dirty) > > { > > - int mapcount; > > + int i, nr = compound ? HPAGE_PMD_NR : 1; > > If CONFIG_TRANSPARENT_HUGEPAGE is not set, we have: > > #define HPAGE_PMD_NR (1< #define HPAGE_PMD_ORDER (HPAGE_PMD_SHIFT-PAGE_SHIFT) > #define HPAGE_PMD_SHIFT ({ BUILD_BUG(); 0; }) > > Depending on compiler version and optimization level, the BUILD_BUG() may be > optimized away (smaps_account() is always called with compound = false if > CONFIG_TRANSPARENT_HUGEPAGE=n), or lead to a build failure: > > fs/built-in.o: In function `smaps_account': > task_mmu.c:(.text+0x4f8fa): undefined reference to > `__compiletime_assert_471' > > Seen with m68k/allmodconfig or allyesconfig and gcc version 4.1.2 20061115 > (prerelease) (Ubuntu 4.1.1-21). > Not seen when compiling the affected file with gcc 4.6.3 or 4.9.0, or with the > m68k defconfigs. Ughh. Please, test this: >>From 5ac27019f886eef033e84c9579e09099469ccbf9 Mon Sep 17 00:00:00 2001 From: "Kirill A. Shutemov" Date: Mon, 18 Jan 2016 14:32:49 +0300 Subject: [PATCH] mm, proc: add workaround for old compilers For THP=n, HPAGE_PMD_NR in smaps_account() expands to BUILD_BUG(). That's fine since this codepath is eliminated by modern compilers. But older compilers have not that efficient dead code elimination. It causes problem at least with gcc 4.1.2 on m68k. Let's replace HPAGE_PMD_NR with 1 << compound_order(page). Signed-off-by: Kirill A. Shutemov Reported-by: Geert Uytterhoeven --- fs/proc/task_mmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c index 65a1b6c69c11..71ffc91060f6 100644 --- a/fs/proc/task_mmu.c +++ b/fs/proc/task_mmu.c @@ -468,7 +468,7 @@ struct mem_size_stats { static void smaps_account(struct mem_size_stats *mss, struct page *page, bool compound, bool young, bool dirty) { - int i, nr = compound ? HPAGE_PMD_NR : 1; + int i, nr = compound ? 1 << compound_order(page) : 1; unsigned long size = nr * PAGE_SIZE; if (PageAnon(page)) -- Kirill A. Shutemov