From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754377AbZKWSXT (ORCPT ); Mon, 23 Nov 2009 13:23:19 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751382AbZKWSXS (ORCPT ); Mon, 23 Nov 2009 13:23:18 -0500 Received: from mx2.mail.elte.hu ([157.181.151.9]:52533 "EHLO mx2.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750820AbZKWSXR (ORCPT ); Mon, 23 Nov 2009 13:23:17 -0500 Date: Mon, 23 Nov 2009 19:23:07 +0100 From: Ingo Molnar To: Rusty Russell Cc: linux-kernel@vger.kernel.org, Mike Travis , Linus Torvalds , Andrew Morton Subject: Re: [PATCH 3/6] cpumask: truncate task_struct.cpus_allowed for CONFIG_CPUMASK_OFFSTACK Message-ID: <20091123182307.GA9123@elte.hu> References: <200911191930.59400.rusty@rustcorp.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200911191930.59400.rusty@rustcorp.com.au> User-Agent: Mutt/1.5.20 (2009-08-17) X-ELTE-SpamScore: 0.0 X-ELTE-SpamLevel: X-ELTE-SpamCheck: no X-ELTE-SpamVersion: ELTE 2.0 X-ELTE-SpamCheck-Details: score=0.0 required=5.9 tests=none autolearn=no SpamAssassin version=3.2.5 _SUMMARY_ Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Rusty Russell wrote: > Turns cpus_allowed into a bitmap, and truncate it to nr_cpu_ids if > CONFIG_CPUMASK_OFFSTACK is set. > > I do this rather than the classic [0] dangling array trick, because of > INIT_TASK and references to sizeof(struct task_struct). > > Signed-off-by: Rusty Russell > --- > include/linux/init_task.h | 2 +- > include/linux/sched.h | 7 +++++-- > kernel/fork.c | 19 ++++++++++++++++++- > 3 files changed, 24 insertions(+), 4 deletions(-) > > diff --git a/include/linux/init_task.h b/include/linux/init_task.h > --- a/include/linux/init_task.h > +++ b/include/linux/init_task.h > @@ -130,7 +130,7 @@ extern struct cred init_cred; > .static_prio = MAX_PRIO-20, \ > .normal_prio = MAX_PRIO-20, \ > .policy = SCHED_NORMAL, \ > - .cpus_allowed = CPU_MASK_ALL, \ > + .cpus_allowed = CPU_BITS_ALL, \ > .mm = NULL, \ > .active_mm = &init_mm, \ > .se = { \ > diff --git a/include/linux/sched.h b/include/linux/sched.h > --- a/include/linux/sched.h > +++ b/include/linux/sched.h > @@ -1256,7 +1256,6 @@ struct task_struct { > #endif > > unsigned int policy; > - cpumask_t cpus_allowed; > > #ifdef CONFIG_TREE_PREEMPT_RCU > int rcu_read_lock_nesting; > @@ -1544,10 +1543,14 @@ struct task_struct { > unsigned long trace_recursion; > #endif /* CONFIG_TRACING */ > unsigned long stack_start; > + > + /* This has to go at the end: if CONFIG_CPUMASK_OFFSTACK=y, only > + * nr_cpu_ids bits will actually be allocated. */ > + DECLARE_BITMAP(cpus_allowed, CONFIG_NR_CPUS); (nit: please use the comment style you see elsewhere in the file.) > }; > > /* Future-safe accessor for struct task_struct's cpus_allowed. */ > -#define tsk_cpumask(tsk) (&(tsk)->cpus_allowed) > +#define tsk_cpumask(tsk) (to_cpumask((tsk)->cpus_allowed)) Please use tsk_cpus_allowed() throughout - so that people who knew what p->cpus_allowed did know what this new thing does. Also, i'm still having second thoughts about it all - could we somehow avoid all this wrappery of commonly used fields? (My main and pretty much only worry is that struct field members look so much cleaner than some wrapper intermediary.) Ingo