diff options
author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2016-12-26 11:55:36 -0500 |
---|---|---|
committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2016-12-26 11:55:36 -0500 |
commit | fabb76afd0b303b6b57a5d49af979134f6d222bd (patch) | |
tree | 3f9910bafa304b592ebcf31fbd138290eb731c78 | |
parent | 69022ed7dd23b951e3e4ddb1ac670e7a1ab2322e (diff) | |
download | 4.9-rt-patches-fabb76afd0b303b6b57a5d49af979134f6d222bd.tar.gz |
Revert "x86: temp backport; make thread_info arch specific again"
This reverts commit 89970dc5aed91bca6c41e9aa43cad7645cf37ddc.
As of -rc2 the commit is present in mainline so the backport is
no longer needed for x86-rt.
-rw-r--r-- | patches/sched-core-x86-Make-struct-thread_info-arch-specific.patch | 91 | ||||
-rw-r--r-- | patches/series | 1 |
2 files changed, 0 insertions, 92 deletions
diff --git a/patches/sched-core-x86-Make-struct-thread_info-arch-specific.patch b/patches/sched-core-x86-Make-struct-thread_info-arch-specific.patch deleted file mode 100644 index ebfa5cc91637f..0000000000000 --- a/patches/sched-core-x86-Make-struct-thread_info-arch-specific.patch +++ /dev/null @@ -1,91 +0,0 @@ -From c8061485a0d7569a865a3cc3c63347b0f42b3765 Mon Sep 17 00:00:00 2001 -From: Heiko Carstens <heiko.carstens@de.ibm.com> -Date: Wed, 19 Oct 2016 19:28:11 +0100 -Subject: [PATCH] sched/core, x86: Make struct thread_info arch specific again - -commit c8061485a0d7569a865a3cc3c63347b0f42b3765 upstream. - -The following commit: - - c65eacbe290b ("sched/core: Allow putting thread_info into task_struct") - -... made 'struct thread_info' a generic struct with only a -single ::flags member, if CONFIG_THREAD_INFO_IN_TASK_STRUCT=y is -selected. - -This change however seems to be quite x86 centric, since at least the -generic preemption code (asm-generic/preempt.h) assumes that struct -thread_info also has a preempt_count member, which apparently was not -true for x86. - -We could add a bit more #ifdefs to solve this problem too, but it seems -to be much simpler to make struct thread_info arch specific -again. This also makes the conversion to THREAD_INFO_IN_TASK_STRUCT a -bit easier for architectures that have a couple of arch specific stuff -in their thread_info definition. - -The arch specific stuff _could_ be moved to thread_struct. However -keeping them in thread_info makes it easier: accessing thread_info -members is simple, since it is at the beginning of the task_struct, -while the thread_struct is at the end. At least on s390 the offsets -needed to access members of the thread_struct (with task_struct as -base) are too large for various asm instructions. This is not a -problem when keeping these members within thread_info. - -Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> -Signed-off-by: Mark Rutland <mark.rutland@arm.com> -Acked-by: Thomas Gleixner <tglx@linutronix.de> -Cc: Andrew Morton <akpm@linux-foundation.org> -Cc: Andy Lutomirski <luto@kernel.org> -Cc: Linus Torvalds <torvalds@linux-foundation.org> -Cc: Peter Zijlstra <peterz@infradead.org> -Cc: keescook@chromium.org -Cc: linux-arch@vger.kernel.org -Link: http://lkml.kernel.org/r/1476901693-8492-2-git-send-email-mark.rutland@arm.com -Signed-off-by: Ingo Molnar <mingo@kernel.org> - -diff --git a/arch/x86/include/asm/thread_info.h b/arch/x86/include/asm/thread_info.h -index 2aaca53c0974..ad6f5eb07a95 100644 ---- a/arch/x86/include/asm/thread_info.h -+++ b/arch/x86/include/asm/thread_info.h -@@ -52,6 +52,15 @@ struct task_struct; - #include <asm/cpufeature.h> - #include <linux/atomic.h> - -+struct thread_info { -+ unsigned long flags; /* low level flags */ -+}; -+ -+#define INIT_THREAD_INFO(tsk) \ -+{ \ -+ .flags = 0, \ -+} -+ - #define init_stack (init_thread_union.stack) - - #else /* !__ASSEMBLY__ */ -diff --git a/include/linux/thread_info.h b/include/linux/thread_info.h -index 45f004e9cc59..2873baf5372a 100644 ---- a/include/linux/thread_info.h -+++ b/include/linux/thread_info.h -@@ -14,17 +14,6 @@ struct timespec; - struct compat_timespec; - - #ifdef CONFIG_THREAD_INFO_IN_TASK --struct thread_info { -- unsigned long flags; /* low level flags */ --}; -- --#define INIT_THREAD_INFO(tsk) \ --{ \ -- .flags = 0, \ --} --#endif -- --#ifdef CONFIG_THREAD_INFO_IN_TASK - #define current_thread_info() ((struct thread_info *)current) - #endif - --- -2.10.1 - diff --git a/patches/series b/patches/series index 463203b04324e..4214437644217 100644 --- a/patches/series +++ b/patches/series @@ -535,7 +535,6 @@ rcu-enable-rcu_normal_after_boot-by-default-for-RT.patch # PREEMPT LAZY preempt-lazy-support.patch ftrace-Fix-trace-header-alignment.patch -sched-core-x86-Make-struct-thread_info-arch-specific.patch x86-preempt-lazy.patch arm-preempt-lazy-support.patch powerpc-preempt-lazy-support.patch |