From: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
To: akpm@linux-foundation.org, Ingo Molnar <mingo@elte.hu>,
linux-kernel@vger.kernel.org
Cc: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>,
Andi Kleen <ak@muc.de>, Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>
Subject: [patch-RFC 23/26] Prepare x86_64 for TIF_SYSCALL_TRACE async flag set in entry.S
Date: Wed, 05 Dec 2007 21:43:06 -0500 [thread overview]
Message-ID: <20071206024437.834251754@polymtl.ca> (raw)
In-Reply-To: 20071206024243.397994403@polymtl.ca
[-- Attachment #1: fix-x86_64-sysenter-trace-race.patch --]
[-- Type: text/plain, Size: 1845 bytes --]
When the flag is inactive upon syscall entry and concurrently activated before
exit, we seem to reach a state where the top of stack is incorrect upon return
to user space.
Fix this by fixing the top of stack and jumping to int_ret_from_sys_call if we
detect that thread flags has been modified.
We make sure that the thread flag read is coherent between our new test and the ALLWORK_MASK test by first saving it in a register used for both comparisons.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
CC: Andi Kleen <ak@muc.de>
CC: Thomas Gleixner <tglx@linutronix.de>
CC: Ingo Molnar <mingo@redhat.com>
CC: H. Peter Anvin <hpa@zytor.com>
---
arch/x86/kernel/entry_64.S | 12 ++++++++++++
1 file changed, 12 insertions(+)
Index: linux-2.6-lttng/arch/x86/kernel/entry_64.S
===================================================================
--- linux-2.6-lttng.orig/arch/x86/kernel/entry_64.S 2007-11-13 09:25:25.000000000 -0500
+++ linux-2.6-lttng/arch/x86/kernel/entry_64.S 2007-11-13 09:49:48.000000000 -0500
@@ -268,6 +268,8 @@ sysret_check:
/* Handle reschedules */
/* edx: work, edi: workmask */
sysret_careful:
+ testl $(_TIF_SYSCALL_TRACE|_TIF_SYSCALL_AUDIT|_TIF_SECCOMP),%edx
+ jnz ret_from_sys_call_trace
bt $TIF_NEED_RESCHED,%edx
jnc sysret_signal
TRACE_IRQS_ON
@@ -279,6 +281,16 @@ sysret_careful:
CFI_ADJUST_CFA_OFFSET -8
jmp sysret_check
+ret_from_sys_call_trace:
+ TRACE_IRQS_ON
+ sti
+ SAVE_REST
+ FIXUP_TOP_OF_STACK %rdi
+ movq %rsp,%rdi
+ LOAD_ARGS ARGOFFSET /* reload args from stack in case ptrace changed it */
+ RESTORE_REST
+ jmp int_ret_from_sys_call
+
/* Handle a signal */
sysret_signal:
TRACE_IRQS_ON
--
Mathieu Desnoyers
Computer Engineering Ph.D. Student, Ecole Polytechnique de Montreal
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68
next prev parent reply other threads:[~2007-12-06 2:53 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-06 2:42 [patch-RFC 00/26] LTTng Kernel Trace Thread Flag Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 01/26] LTTng Kernel Trace Thread Flag Alpha Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 02/26] LTTng Kernel Trace Thread Flag ARM Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 03/26] LTTng Kernel Trace Thread Flag AVR32 Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 04/26] LTTng Kernel Trace Thread Flag Blackfin Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 05/26] LTTng Kernel Trace Thread Flag Cris Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 06/26] LTTng Kernel Trace Thread Flag Frv Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 07/26] LTTng Kernel Trace Thread Flag H8300 Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 08/26] LTTng Linux Kernel Trace Thread Flags x86_32 Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 09/26] LTTng Kernel Trace Thread Flag ia64 Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 10/26] LTTng Kernel Trace Thread Flag m32r Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 11/26] LTTng Kernel Trace Thread Flag m68k Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 12/26] LTTng Kernel Trace Thread Flag m68knommu Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 13/26] LTTng Kernel Trace Thread Flag MIPS Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 14/26] LTTng Kernel Trace Thread Flag parisc Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 15/26] LTTng Kernel Trace Thread Flag powerpc Mathieu Desnoyers
2007-12-06 2:42 ` [patch-RFC 16/26] LTTng Kernel Trace Thread Flag s390 Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 17/26] LTTng Kernel Trace Thread Flag SH Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 18/26] LTTng Kernel Trace Thread Flag sh64 Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 19/26] LTTng Kernel Trace Thread Flag sparc Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 20/26] LTTng Kernel Trace Thread Flag sparc64 Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 21/26] LTTng Kernel Trace Thread Flag UML Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 22/26] LTTng Kernel Trace Thread Flag v850 Mathieu Desnoyers
2007-12-06 2:43 ` Mathieu Desnoyers [this message]
2007-12-06 2:43 ` [patch-RFC 24/26] LTTng Linux Kernel Trace Thread Flag x86_64 Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 25/26] LTTng Kernel Trace Thread Flag xtensa Mathieu Desnoyers
2007-12-06 2:43 ` [patch-RFC 26/26] LTTng Kernel Trace Thread Flag API Mathieu Desnoyers
2007-12-06 21:30 ` [patch-RFC 00/26] LTTng Kernel Trace Thread Flag Frank Ch. Eigler
2007-12-07 13:44 ` Mathieu Desnoyers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20071206024437.834251754@polymtl.ca \
--to=mathieu.desnoyers@polymtl.ca \
--cc=ak@muc.de \
--cc=akpm@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mingo@redhat.com \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox