From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754305AbYC1Jnc (ORCPT ); Fri, 28 Mar 2008 05:43:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751151AbYC1JnX (ORCPT ); Fri, 28 Mar 2008 05:43:23 -0400 Received: from fk-out-0910.google.com ([209.85.128.187]:63664 "EHLO fk-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750810AbYC1JnX (ORCPT ); Fri, 28 Mar 2008 05:43:23 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=message-id:date:from:user-agent:mime-version:to:cc:subject:references:in-reply-to:x-enigmail-version:content-type:content-transfer-encoding; b=ie5xJZC+ExFTzjwL7Rm0HmoAr1oYwsuF0AoBOTAaKdjbyVx7g35N2qyjX18N+AsAvOdRy/ZlFILKPjLLqhhKXANdbgBofu0bPsCvgW4aeKCQXN+o1NhxCYri9G3dpFtTBkjtTZgXQHaqLgfzUi3KkBFMHeldaKW5VJ+mGw1p8FA= Message-ID: <47ECBDAF.9070007@gmail.com> Date: Fri, 28 Mar 2008 10:43:11 +0100 From: Jiri Slaby User-Agent: Thunderbird 2.0.0.12 (X11/20080213) MIME-Version: 1.0 To: Balbir Singh 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 Subject: Re: [-mm] Add an owner to the mm_struct (v2) References: <20080328082316.6961.29044.sendpatchset@localhost.localdomain> <47ECBD4A.8080908@gmail.com> In-Reply-To: <47ECBD4A.8080908@gmail.com> X-Enigmail-Version: 0.95.6 Content-Type: text/plain; charset=ISO-8859-2; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/28/2008 10:41 AM, Jiri Slaby wrote: >> linux-2.6.25-rc5/include/linux/mm_types.h~memory-controller-add-mm-owner >> 2008-03-28 09:30:47.000000000 +0530 >> +++ linux-2.6.25-rc5-balbir/include/linux/mm_types.h 2008-03-28 >> 12:26:59.000000000 +0530 >> @@ -227,8 +227,10 @@ 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 >> + spinlock_t owner_lock; >> + struct task_struct *owner; /* The thread group leader that */ > > Doesn't make sense to switch them (spinlock is unsigned int on x86, > what's sizeof between and after?)? Hmm, doesn't matter, there is another pointer after it, ignore me.