From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 11 Mar 2005 13:25:00 -0500 From: Dave Jones Subject: Re: [PATCH] mm counter operations through macros Message-ID: <20050311182500.GA4185@redhat.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org Return-Path: To: Christoph Lameter Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org List-ID: On Fri, Mar 11, 2005 at 04:23:21AM -0800, Christoph Lameter wrote: > This patch extracts all the operations on counters protected by the > page table lock (currently rss and anon_rss) into definitions in > include/linux/sched.h. All rss operations are performed through > the following three macros: > > get_mm_counter(mm, member) -> Obtain the value of a counter > set_mm_counter(mm, member, value) -> Set the value of a counter > update_mm_counter(mm, member, value) -> Add a value to a counter Splitting this last one into inc_mm_counter() and dec_mm_counter() means you can kill off the last argument, and get some of the readability back. As it stands, I think this patch adds a bunch of obfuscation for no clear benefit. Dave -- 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: aart@kvack.org