From mboxrd@z Thu Jan 1 00:00:00 1970 From: yamamoto-jCdQPDEk3idL9jVzuh4AOg@public.gmane.org (YAMAMOTO Takashi) Subject: Re: [Devel] [PATCH] memory.min_usage Date: Tue, 4 Dec 2007 16:02:57 +0900 (JST) Message-ID: <20071204070257.CA4171D0BCD@siro.lan> References: <6599ad830712032203t48e455cbjd25a40cf93cb453f@mail.gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Return-path: In-Reply-To: Your message of "Mon, 3 Dec 2007 22:03:57 -0800" <6599ad830712032203t48e455cbjd25a40cf93cb453f-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: menage-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org Cc: containers-qjLDD68F18O7TbgM5vRIOg@public.gmane.org, minoura-jCdQPDEk3idL9jVzuh4AOg@public.gmane.org, balbir-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org List-Id: containers.vger.kernel.org > On Dec 3, 2007 8:09 PM, YAMAMOTO Takashi wrote: > > +static u64 mem_cgroup_min_usage_read(struct cgroup *cont, struct cftype *cft) > > +{ > > + struct mem_cgroup *mem = mem_cgroup_from_cont(cont); > > + > > + return mem->min_usage; > > +} > > + > > This won't be atomic on a 32-bit arch, and I don't think you have any locking. > > Paul sure. probably it's better to make it a part of res_counter. YAMAMOTO Takashi