Messages in this thread Patch in this message | | | Date | Fri, 6 Mar 2009 15:15:28 GMT | From | Markus Metzger <> | Subject | [tip:x86/urgent] x86, bts: remove bad warning |
| |
Commit-ID: 9ca0791dcaa666e8e8f4b4ca028b65b4bde9cb28 Gitweb: http://git.kernel.org/tip/9ca0791dcaa666e8e8f4b4ca028b65b4bde9cb28 Author: "Markus Metzger" <markus.t.metzger@intel.com> AuthorDate: Thu, 5 Mar 2009 08:49:54 +0100 Commit: Ingo Molnar <mingo@elte.hu> CommitDate: Fri, 6 Mar 2009 16:13:15 +0100
x86, bts: remove bad warning
In case a ptraced task is reaped (while the tracer is still attached), ds_exit_thread() is called before ptrace_exit(). The latter will release the bts_tracer and remove the thread's ds_ctx. The former will WARN() if the context is not NULL.
Oleg Nesterov submitted patches that move ptrace_exit() before exit_thread() and thus reverse the order of the above calls.
Remove the bad warning. I will add it again when Oleg's changes are in.
Signed-off-by: Markus Metzger <markus.t.metzger@intel.com> LKML-Reference: <20090305084954.A22000@sedona.ch.intel.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
--- arch/x86/kernel/ds.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/arch/x86/kernel/ds.c b/arch/x86/kernel/ds.c index 169a120..de7cdbd 100644 --- a/arch/x86/kernel/ds.c +++ b/arch/x86/kernel/ds.c @@ -1029,5 +1029,4 @@ void ds_copy_thread(struct task_struct *tsk, struct task_struct *father) void ds_exit_thread(struct task_struct *tsk) { - WARN_ON(tsk->thread.ds_ctx); }
| |