sched: Remove relyance on STRUCT_ALIGNMENT

Florian reported that all of kernel/sched/ is rebuild when
CONFIG_BLK_DEV_INITRD is changed, which, while not a bug is
unexpected. This is due to us including vmlinux.lds.h.

Jakub explained that the problem is that we put the alignment
requirement on the type instead of on a variable. Type alignment is a
minimum, the compiler is free to pick any larger alignment for a
specific instance of the type (eg. the variable).

So force the type alignment on all individual variable definitions and
remove the undesired dependency on vmlinux.lds.h.

Fixes: 85c2ce9104eb ("sched, vmlinux.lds: Increase STRUCT_ALIGNMENT to 64 bytes for GCC-4.9")
Reported-by: Florian Fainelli <f.fainelli@gmail.com>
Suggested-by: Jakub Jelinek <jakub@redhat.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>

diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
index 0b45dd1068f720b61853f37f460ebdd15107d102..c6ce90fbb7ac3c078eafb7de6beea0627a1128d8 100644 (file)
--- a/kernel/sched/deadline.c
+++ b/kernel/sched/deadline.c
@@ -2522,8 +2522,8 @@ static void prio_changed_dl(struct rq *rq, struct task_struct *p,
        }
 }
 
-const struct sched_class dl_sched_class
-       __section("__dl_sched_class") = {
+DEFINE_SCHED_CLASS(dl) = {
+
        .enqueue_task           = enqueue_task_dl,
        .dequeue_task           = dequeue_task_dl,
        .yield_task             = yield_task_dl,

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index cd9a37c0601bbd09044b79e32972f4417d5fb777..f30d35a43f7370f92f63810c29c2c0f6cef776f6 100644 (file)
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -11171,8 +11171,8 @@ static unsigned int get_rr_interval_fair(struct rq *rq, struct task_struct *task
 /*
  * All the scheduling class methods:
  */
-const struct sched_class fair_sched_class
-       __section("__fair_sched_class") = {
+DEFINE_SCHED_CLASS(fair) = {
+
        .enqueue_task           = enqueue_task_fair,
        .dequeue_task           = dequeue_task_fair,
        .yield_task             = yield_task_fair,

diff --git a/kernel/sched/idle.c b/kernel/sched/idle.c
index 846743e39b3c4e2de529a0c40985ff2e6821324b..9da69c4e0ee9905dc7d4db59c64ec323288f6a5e 100644 (file)
--- a/kernel/sched/idle.c
+++ b/kernel/sched/idle.c
@@ -458,8 +458,8 @@ static void update_curr_idle(struct rq *rq)
 /*
  * Simple, special scheduling class for the per-CPU idle tasks:
  */
-const struct sched_class idle_sched_class
-       __section("__idle_sched_class") = {
+DEFINE_SCHED_CLASS(idle) = {
+
        /* no enqueue/yield_task for idle tasks */
 
        /* dequeue is not valid, we print a debug message there: */

diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index 8a3b1ba09253b09672248aa888f662e0bde6a655..9b27352e0c1badab67a455d44ca51391d5ae9aef 100644 (file)
--- a/kernel/sched/rt.c
+++ b/kernel/sched/rt.c
@@ -2431,8 +2431,8 @@ static unsigned int get_rr_interval_rt(struct rq *rq, struct task_struct *task)
                return 0;
 }
 
-const struct sched_class rt_sched_class
-       __section("__rt_sched_class") = {
+DEFINE_SCHED_CLASS(rt) = {
+
        .enqueue_task           = enqueue_task_rt,
        .dequeue_task           = dequeue_task_rt,
        .yield_task             = yield_task_rt,

diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
index 965b2968c13aa67bbc1ca361d2e9f5e66eb65016..3e45055efbc55085d63d84d7d88451d2287efb35 100644 (file)
--- a/kernel/sched/sched.h
+++ b/kernel/sched/sched.h
@@ -67,7 +67,6 @@
 #include <linux/tsacct_kern.h>
 
 #include <asm/tlb.h>
-#include <asm-generic/vmlinux.lds.h>
 
 #ifdef CONFIG_PARAVIRT
 # include <asm/paravirt.h>
@@ -1836,7 +1835,7 @@ struct sched_class {
 #ifdef CONFIG_FAIR_GROUP_SCHED
        void (*task_change_group)(struct task_struct *p, int type);
 #endif
-} __aligned(STRUCT_ALIGNMENT); /* STRUCT_ALIGN(), vmlinux.lds.h */
+};
 
 static inline void put_prev_task(struct rq *rq, struct task_struct *prev)
 {
@@ -1850,6 +1849,20 @@ static inline void set_next_task(struct rq *rq, struct task_struct *next)
        next->sched_class->set_next_task(rq, next, false);
 }
 
+
+/*
+ * Helper to define a sched_class instance; each one is placed in a separate
+ * section which is ordered by the linker script:
+ *
+ *   include/asm-generic/vmlinux.lds.h
+ *
+ * Also enforce alignment on the instance, not the type, to guarantee layout.
+ */
+#define DEFINE_SCHED_CLASS(name) \
+const struct sched_class name##_sched_class \
+       __aligned(__alignof__(struct sched_class)) \
+       __section("__" #name "_sched_class")
+
 /* Defined in include/asm-generic/vmlinux.lds.h */
 extern struct sched_class __begin_sched_classes[];
 extern struct sched_class __end_sched_classes[];

diff --git a/kernel/sched/stop_task.c b/kernel/sched/stop_task.c
index ceb5b6b125618eeda2a117efc1f805c62be28c13..91bb10cc070e953d8c9dcf7c595f7d368b289c73 100644 (file)
--- a/kernel/sched/stop_task.c
+++ b/kernel/sched/stop_task.c
@@ -109,8 +109,7 @@ static void update_curr_stop(struct rq *rq)
 /*
  * Simple, special scheduling class for the per-CPU stop tasks:
  */
-const struct sched_class stop_sched_class
-       __section("__stop_sched_class") = {
+DEFINE_SCHED_CLASS(stop) = {
 
        .enqueue_task           = enqueue_task_stop,
        .dequeue_task           = dequeue_task_stop,