From mboxrd@z Thu Jan 1 00:00:00 1970 From: Balbir Singh Subject: Re: [RFC][-mm] Add an owner to the mm_struct (v4) Date: Tue, 01 Apr 2008 21:45:09 +0530 Message-ID: <47F25F8D.7080102@linux.vnet.ibm.com> References: <20080401124312.23664.64616.sendpatchset@localhost.localdomain> <84144f020804010900s5335988ai58546874a6a2f8bd@mail.gmail.com> Reply-To: balbir@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <84144f020804010900s5335988ai58546874a6a2f8bd@mail.gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Pekka Enberg Cc: Paul Menage , Pavel Emelianov , Hugh Dickins , Sudhir Kumar , YAMAMOTO Takashi , lizf@cn.fujitsu.com, linux-kernel@vger.kernel.org, taka@valinux.co.jp, linux-mm@kvack.org, David Rientjes , Andrew Morton , KAMEZAWA Hiroyuki List-Id: linux-mm.kvack.org Pekka Enberg wrote: > Hi, > > On Tue, Apr 1, 2008 at 3:43 PM, Balbir Singh wrote: >> @@ -227,8 +227,9 @@ struct mm_struct { >> /* aio bits */ >> rwlock_t ioctx_list_lock; >> struct kioctx *ioctx_list; >> -#ifdef CONFIG_CGROUP_MEM_RES_CTLR >> - struct mem_cgroup *mem_cgroup; >> +#ifdef CONFIG_MM_OWNER >> + struct task_struct *owner; /* The thread group leader that */ >> + /* owns the mm_struct. */ >> #endif > > Yes, please. This is useful for the revokeat() patches as well. I > currently need a big ugly loop to scan each task so I can break COW of > private pages. Hi, Pekka, It's good to know that this will be useful. I think understand your use case of having to walk the entire tasklist to break COW of private pages, having the owner information right in the mm_struct is definitely useful. -- Warm Regards, Balbir Singh Linux Technology Center IBM, ISTL