From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757642AbZEKLdX (ORCPT ); Mon, 11 May 2009 07:33:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753856AbZEKLdM (ORCPT ); Mon, 11 May 2009 07:33:12 -0400 Received: from hera.kernel.org ([140.211.167.34]:37306 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750841AbZEKLdK (ORCPT ); Mon, 11 May 2009 07:33:10 -0400 Date: Mon, 11 May 2009 11:30:32 GMT From: tip-bot for Jeff Garzik To: linux-tip-commits@vger.kernel.org Cc: linux-kernel@vger.kernel.org, jgarzik@redhat.com, hpa@zytor.com, mingo@redhat.com, akpm@linux-foundation.org, jeff@garzik.org, tglx@linutronix.de, mingo@elte.hu Reply-To: mingo@redhat.com, hpa@zytor.com, jgarzik@redhat.com, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, jeff@garzik.org, tglx@linutronix.de, mingo@elte.hu In-Reply-To: <20090508193841.GA13831@havoc.gtf.org> References: <20090508193841.GA13831@havoc.gtf.org> Subject: [tip:sched/core] kernel/{sched, smp}.c: fix static decl prior to struct declaration Message-ID: Git-Commit-ID: 125423a9a380c207aaf3df016fe452d1e7da403e X-Mailer: tip-git-log-daemon MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.0 (hera.kernel.org [127.0.0.1]); Mon, 11 May 2009 11:30:34 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: 125423a9a380c207aaf3df016fe452d1e7da403e Gitweb: http://git.kernel.org/tip/125423a9a380c207aaf3df016fe452d1e7da403e Author: Jeff Garzik AuthorDate: Fri, 8 May 2009 15:38:41 -0400 Committer: Ingo Molnar CommitDate: Mon, 11 May 2009 13:00:19 +0200 kernel/{sched, smp}.c: fix static decl prior to struct declaration According to C99 6.9.2p3, any declaration "static struct foo my_foo;" must follow the definition of struct foo. Apparently, gcc's lack of warning is a bug. v2: - fix ifdef imbalance, by moving entire USER_SCHED code block - indent cpp directives, to indicate nesting [ Impact: cleanup ] Signed-off-by: Jeff Garzik Cc: viro@zeniv.linux.org.uk Cc: Andrew Morton Cc: roland@redhat.com LKML-Reference: <20090508193841.GA13831@havoc.gtf.org> Signed-off-by: Ingo Molnar --- kernel/sched.c | 60 ++++++++++++++++++++++++++++---------------------------- kernel/smp.c | 4 +- 2 files changed, 32 insertions(+), 32 deletions(-) diff --git a/kernel/sched.c b/kernel/sched.c index 8908d19..7d1ea21 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -303,36 +303,6 @@ struct task_group { struct list_head children; }; -#ifdef CONFIG_USER_SCHED - -/* Helper function to pass uid information to create_sched_user() */ -void set_tg_uid(struct user_struct *user) -{ - user->tg->uid = user->uid; -} - -/* - * Root task group. - * Every UID task group (including init_task_group aka UID-0) will - * be a child to this group. - */ -struct task_group root_task_group; - -#ifdef CONFIG_FAIR_GROUP_SCHED -/* Default task group's sched entity on each cpu */ -static DEFINE_PER_CPU(struct sched_entity, init_sched_entity); -/* Default task group's cfs_rq on each cpu */ -static DEFINE_PER_CPU(struct cfs_rq, init_cfs_rq) ____cacheline_aligned_in_smp; -#endif /* CONFIG_FAIR_GROUP_SCHED */ - -#ifdef CONFIG_RT_GROUP_SCHED -static DEFINE_PER_CPU(struct sched_rt_entity, init_sched_rt_entity); -static DEFINE_PER_CPU(struct rt_rq, init_rt_rq) ____cacheline_aligned_in_smp; -#endif /* CONFIG_RT_GROUP_SCHED */ -#else /* !CONFIG_USER_SCHED */ -#define root_task_group init_task_group -#endif /* CONFIG_USER_SCHED */ - /* task_group_lock serializes add/remove of task groups and also changes to * a task group's cpu shares. */ @@ -663,6 +633,36 @@ struct rq { static DEFINE_PER_CPU_SHARED_ALIGNED(struct rq, runqueues); +#ifdef CONFIG_USER_SCHED + +/* Helper function to pass uid information to create_sched_user() */ +void set_tg_uid(struct user_struct *user) +{ + user->tg->uid = user->uid; +} + +/* + * Root task group. + * Every UID task group (including init_task_group aka UID-0) will + * be a child to this group. + */ +struct task_group root_task_group; + +# ifdef CONFIG_FAIR_GROUP_SCHED +/* Default task group's sched entity on each cpu */ +static DEFINE_PER_CPU(struct sched_entity, init_sched_entity); +/* Default task group's cfs_rq on each cpu */ +static DEFINE_PER_CPU(struct cfs_rq, init_cfs_rq) ____cacheline_aligned_in_smp; +# endif /* CONFIG_FAIR_GROUP_SCHED */ + +# ifdef CONFIG_RT_GROUP_SCHED +static DEFINE_PER_CPU(struct sched_rt_entity, init_sched_rt_entity); +static DEFINE_PER_CPU(struct rt_rq, init_rt_rq) ____cacheline_aligned_in_smp; +# endif /* CONFIG_RT_GROUP_SCHED */ +#else /* !CONFIG_USER_SCHED */ +# define root_task_group init_task_group +#endif /* CONFIG_USER_SCHED */ + static inline void check_preempt_curr(struct rq *rq, struct task_struct *p, int sync) { rq->curr->sched_class->check_preempt_curr(rq, p, sync); diff --git a/kernel/smp.c b/kernel/smp.c index 858baac..aba7bda 100644 --- a/kernel/smp.c +++ b/kernel/smp.c @@ -12,8 +12,6 @@ #include #include -static DEFINE_PER_CPU(struct call_single_queue, call_single_queue); - static struct { struct list_head queue; spinlock_t lock; @@ -39,6 +37,8 @@ struct call_single_queue { spinlock_t lock; }; +static DEFINE_PER_CPU(struct call_single_queue, call_single_queue); + static DEFINE_PER_CPU(struct call_function_data, cfd_data) = { .lock = __SPIN_LOCK_UNLOCKED(cfd_data.lock), };