From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 11 Mar 2005 16:35:19 -0800 (PST) From: Christoph Lameter Subject: Re: [PATCH] mm counter operations through macros In-Reply-To: <20050311145226.6ee4a951.akpm@osdl.org> Message-ID: References: <20050311182500.GA4185@redhat.com> <20050311145226.6ee4a951.akpm@osdl.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Andrew Morton Cc: davej@redhat.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org List-ID: On Fri, 11 Mar 2005, Andrew Morton wrote: > > +#define set_mm_counter(mm, member, value) (mm)->member = (value) > > +#define get_mm_counter(mm, member) ((mm)->member) > > +#define update_mm_counter(mm, member, value) (mm)->member += (value) > > +#define inc_mm_counter(mm, member) (mm)->member++ > > +#define dec_mm_counter(mm, member) (mm)->member-- > > +#define MM_COUNTER_T unsigned long > > Would prefer `mm_counter_t' here. > > Why not a typedef? Ok typedef it is. > > @@ -231,9 +237,13 @@ struct mm_struct { > > unsigned long start_code, end_code, start_data, end_data; > > unsigned long start_brk, brk, start_stack; > > unsigned long arg_start, arg_end, env_start, env_end; > > - unsigned long rss, anon_rss, total_vm, locked_vm, shared_vm; > > + unsigned long total_vm, locked_vm, shared_vm; > > unsigned long exec_vm, stack_vm, reserved_vm, def_flags, nr_ptes; > > > > + /* Special counters protected by the page_table_lock */ > > + MM_COUNTER_T rss; > > + MM_COUNTER_T anon_rss; > > + > > Why were only two counters converted? Because only these two counters are protected by the page table lock. > Could I suggest that you rename all these counters, so that code which > fails to use the macros won't compile? Ok. will make them mm_xx as they were in previous patches. -- 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