From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f181.google.com (mail-pd0-f181.google.com [209.85.192.181]) by kanga.kvack.org (Postfix) with ESMTP id A9A766B0035 for ; Wed, 5 Feb 2014 17:17:29 -0500 (EST) Received: by mail-pd0-f181.google.com with SMTP id y10so896826pdj.40 for ; Wed, 05 Feb 2014 14:17:29 -0800 (PST) Received: from mail.linuxfoundation.org (mail.linuxfoundation.org. [140.211.169.12]) by mx.google.com with ESMTP id gx4si30574554pbc.51.2014.02.05.14.17.28 for ; Wed, 05 Feb 2014 14:17:28 -0800 (PST) Date: Wed, 5 Feb 2014 14:17:26 -0800 From: Andrew Morton Subject: Re: [patch 01/10] mm: vmstat: fix UP zone state accounting Message-Id: <20140205141726.e474d52258e0a09418ba1018@linux-foundation.org> In-Reply-To: <1391475222-1169-2-git-send-email-hannes@cmpxchg.org> References: <1391475222-1169-1-git-send-email-hannes@cmpxchg.org> <1391475222-1169-2-git-send-email-hannes@cmpxchg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Johannes Weiner Cc: Andi Kleen , Andrea Arcangeli , Bob Liu , Christoph Hellwig , Dave Chinner , Greg Thelen , Hugh Dickins , Jan Kara , KOSAKI Motohiro , Luigi Semenzato , Mel Gorman , Metin Doslu , Michel Lespinasse , Minchan Kim , Ozgun Erdogan , Peter Zijlstra , Rik van Riel , Roman Gushchin , Ryan Mallon , Tejun Heo , Vlastimil Babka , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org On Mon, 3 Feb 2014 19:53:33 -0500 Johannes Weiner wrote: > Fengguang Wu's build testing spotted problems with inc_zone_state() > and dec_zone_state() on UP configurations in out-of-tree patches. > > inc_zone_state() is declared but not defined, dec_zone_state() is > missing entirely. > > Just like with *_zone_page_state(), they can be defined like their > preemption-unsafe counterparts on UP. um, In file included from include/linux/mm.h:876, from include/linux/suspend.h:8, from arch/x86/kernel/asm-offsets.c:12: include/linux/vmstat.h: In function '__inc_zone_page_state': include/linux/vmstat.h:228: error: implicit declaration of function '__inc_zone_state' include/linux/vmstat.h: In function '__dec_zone_page_state': include/linux/vmstat.h:234: error: implicit declaration of function '__dec_zone_state' include/linux/vmstat.h: At top level: include/linux/vmstat.h:245: warning: conflicting types for '__inc_zone_state' include/linux/vmstat.h:245: error: static declaration of '__inc_zone_state' follows non-static declaration include/linux/vmstat.h:228: note: previous implicit declaration of '__inc_zone_state' was here include/linux/vmstat.h:251: warning: conflicting types for '__dec_zone_state' include/linux/vmstat.h:251: error: static declaration of '__dec_zone_state' follows non-static declaration include/linux/vmstat.h:234: note: previous implicit declaration of '__dec_zone_state' was here I shuffled them around: --- a/include/linux/vmstat.h~mm-vmstat-fix-up-zone-state-accounting-fix +++ a/include/linux/vmstat.h @@ -214,6 +214,18 @@ static inline void __mod_zone_page_state zone_page_state_add(delta, zone, item); } +static inline void __inc_zone_state(struct zone *zone, enum zone_stat_item item) +{ + atomic_long_inc(&zone->vm_stat[item]); + atomic_long_inc(&vm_stat[item]); +} + +static inline void __dec_zone_state(struct zone *zone, enum zone_stat_item item) +{ + atomic_long_dec(&zone->vm_stat[item]); + atomic_long_dec(&vm_stat[item]); +} + static inline void __inc_zone_page_state(struct page *page, enum zone_stat_item item) { @@ -234,18 +246,6 @@ static inline void __dec_zone_page_state #define dec_zone_page_state __dec_zone_page_state #define mod_zone_page_state __mod_zone_page_state -static inline void __inc_zone_state(struct zone *zone, enum zone_stat_item item) -{ - atomic_long_inc(&zone->vm_stat[item]); - atomic_long_inc(&vm_stat[item]); -} - -static inline void __dec_zone_state(struct zone *zone, enum zone_stat_item item) -{ - atomic_long_dec(&zone->vm_stat[item]); - atomic_long_dec(&vm_stat[item]); -} - #define inc_zone_state __inc_zone_state #define dec_zone_state __dec_zone_state _ -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org