Merge branches 'x86/acpi', 'x86/apic', 'x86/cpudetect', 'x86/headers', 'x86/paravirt...
[safe/jmp/linux-2.6] / arch / x86 / kernel / ptrace.c
index 58ce4b5..d2f7cd5 100644 (file)
@@ -14,6 +14,7 @@
 #include <linux/errno.h>
 #include <linux/ptrace.h>
 #include <linux/regset.h>
+#include <linux/tracehook.h>
 #include <linux/user.h>
 #include <linux/elf.h>
 #include <linux/security.h>
@@ -39,7 +40,9 @@ enum x86_regset {
        REGSET_GENERAL,
        REGSET_FP,
        REGSET_XFP,
+       REGSET_IOPERM64 = REGSET_XFP,
        REGSET_TLS,
+       REGSET_IOPERM32,
 };
 
 /*
@@ -72,10 +75,7 @@ static inline bool invalid_selector(u16 value)
 static unsigned long *pt_regs_access(struct pt_regs *regs, unsigned long regno)
 {
        BUILD_BUG_ON(offsetof(struct pt_regs, bx) != 0);
-       regno >>= 2;
-       if (regno > FS)
-               --regno;
-       return &regs->bx + regno;
+       return &regs->bx + (regno >> 2);
 }
 
 static u16 get_segment_reg(struct task_struct *task, unsigned long offset)
@@ -87,9 +87,10 @@ static u16 get_segment_reg(struct task_struct *task, unsigned long offset)
        if (offset != offsetof(struct user_regs_struct, gs))
                retval = *pt_regs_access(task_pt_regs(task), offset);
        else {
-               retval = task->thread.gs;
                if (task == current)
-                       savesegment(gs, retval);
+                       retval = get_user_gs(task_pt_regs(task));
+               else
+                       retval = task_user_gs(task);
        }
        return retval;
 }
@@ -123,13 +124,10 @@ static int set_segment_reg(struct task_struct *task,
                break;
 
        case offsetof(struct user_regs_struct, gs):
-               task->thread.gs = value;
                if (task == current)
-                       /*
-                        * The user-mode %gs is not affected by
-                        * kernel entry, so we must update the CPU.
-                        */
-                       loadsegment(gs, value);
+                       set_user_gs(task_pt_regs(task), value);
+               else
+                       task_user_gs(task) = value;
        }
 
        return 0;
@@ -554,159 +552,115 @@ static int ptrace_set_debugreg(struct task_struct *child,
        return 0;
 }
 
-#ifdef CONFIG_X86_PTRACE_BTS
-/*
- * The configuration for a particular BTS hardware implementation.
- */
-struct bts_configuration {
-       /* the size of a BTS record in bytes; at most BTS_MAX_RECORD_SIZE */
-       unsigned char  sizeof_bts;
-       /* the size of a field in the BTS record in bytes */
-       unsigned char  sizeof_field;
-       /* a bitmask to enable/disable BTS in DEBUGCTL MSR */
-       unsigned long debugctl_mask;
-};
-static struct bts_configuration bts_cfg;
-
-#define BTS_MAX_RECORD_SIZE (8 * 3)
-
-
 /*
- * Branch Trace Store (BTS) uses the following format. Different
- * architectures vary in the size of those fields.
- * - source linear address
- * - destination linear address
- * - flags
- *
- * Later architectures use 64bit pointers throughout, whereas earlier
- * architectures use 32bit pointers in 32bit mode.
- *
- * We compute the base address for the first 8 fields based on:
- * - the field size stored in the DS configuration
- * - the relative field position
- *
- * In order to store additional information in the BTS buffer, we use
- * a special source address to indicate that the record requires
- * special interpretation.
- *
- * Netburst indicated via a bit in the flags field whether the branch
- * was predicted; this is ignored.
+ * These access the current or another (stopped) task's io permission
+ * bitmap for debugging or core dump.
  */
-
-enum bts_field {
-       bts_from = 0,
-       bts_to,
-       bts_flags,
-
-       bts_escape = (unsigned long)-1,
-       bts_qual = bts_to,
-       bts_jiffies = bts_flags
-};
-
-static inline unsigned long bts_get(const char *base, enum bts_field field)
+static int ioperm_active(struct task_struct *target,
+                        const struct user_regset *regset)
 {
-       base += (bts_cfg.sizeof_field * field);
-       return *(unsigned long *)base;
+       return target->thread.io_bitmap_max / regset->size;
 }
 
-static inline void bts_set(char *base, enum bts_field field, unsigned long val)
+static int ioperm_get(struct task_struct *target,
+                     const struct user_regset *regset,
+                     unsigned int pos, unsigned int count,
+                     void *kbuf, void __user *ubuf)
 {
-       base += (bts_cfg.sizeof_field * field);;
-       (*(unsigned long *)base) = val;
-}
+       if (!target->thread.io_bitmap_ptr)
+               return -ENXIO;
 
-/*
- * Translate a BTS record from the raw format into the bts_struct format
- *
- * out (out): bts_struct interpretation
- * raw: raw BTS record
- */
-static void ptrace_bts_translate_record(struct bts_struct *out, const void *raw)
-{
-       memset(out, 0, sizeof(*out));
-       if (bts_get(raw, bts_from) == bts_escape) {
-               out->qualifier       = bts_get(raw, bts_qual);
-               out->variant.jiffies = bts_get(raw, bts_jiffies);
-       } else {
-               out->qualifier = BTS_BRANCH;
-               out->variant.lbr.from_ip = bts_get(raw, bts_from);
-               out->variant.lbr.to_ip   = bts_get(raw, bts_to);
-       }
+       return user_regset_copyout(&pos, &count, &kbuf, &ubuf,
+                                  target->thread.io_bitmap_ptr,
+                                  0, IO_BITMAP_BYTES);
 }
 
+#ifdef CONFIG_X86_PTRACE_BTS
 static int ptrace_bts_read_record(struct task_struct *child, size_t index,
                                  struct bts_struct __user *out)
 {
-       struct bts_struct ret;
-       const void *bts_record;
-       size_t bts_index, bts_end;
+       const struct bts_trace *trace;
+       struct bts_struct bts;
+       const unsigned char *at;
        int error;
 
-       error = ds_get_bts_end(child, &bts_end);
-       if (error < 0)
-               return error;
-
-       if (bts_end <= index)
-               return -EINVAL;
+       trace = ds_read_bts(child->bts);
+       if (!trace)
+               return -EPERM;
 
-       error = ds_get_bts_index(child, &bts_index);
-       if (error < 0)
-               return error;
+       at = trace->ds.top - ((index + 1) * trace->ds.size);
+       if ((void *)at < trace->ds.begin)
+               at += (trace->ds.n * trace->ds.size);
 
-       /* translate the ptrace bts index into the ds bts index */
-       bts_index += bts_end - (index + 1);
-       if (bts_end <= bts_index)
-               bts_index -= bts_end;
+       if (!trace->read)
+               return -EOPNOTSUPP;
 
-       error = ds_access_bts(child, bts_index, &bts_record);
+       error = trace->read(child->bts, at, &bts);
        if (error < 0)
                return error;
 
-       ptrace_bts_translate_record(&ret, bts_record);
-
-       if (copy_to_user(out, &ret, sizeof(ret)))
+       if (copy_to_user(out, &bts, sizeof(bts)))
                return -EFAULT;
 
-       return sizeof(ret);
+       return sizeof(bts);
 }
 
 static int ptrace_bts_drain(struct task_struct *child,
                            long size,
                            struct bts_struct __user *out)
 {
-       struct bts_struct ret;
-       const unsigned char *raw;
-       size_t end, i;
-       int error;
+       const struct bts_trace *trace;
+       const unsigned char *at;
+       int error, drained = 0;
 
-       error = ds_get_bts_index(child, &end);
-       if (error < 0)
-               return error;
+       trace = ds_read_bts(child->bts);
+       if (!trace)
+               return -EPERM;
 
-       if (size < (end * sizeof(struct bts_struct)))
+       if (!trace->read)
+               return -EOPNOTSUPP;
+
+       if (size < (trace->ds.top - trace->ds.begin))
                return -EIO;
 
-       error = ds_access_bts(child, 0, (const void **)&raw);
-       if (error < 0)
-               return error;
+       for (at = trace->ds.begin; (void *)at < trace->ds.top;
+            out++, drained++, at += trace->ds.size) {
+               struct bts_struct bts;
+               int error;
 
-       for (i = 0; i < end; i++, out++, raw += bts_cfg.sizeof_bts) {
-               ptrace_bts_translate_record(&ret, raw);
+               error = trace->read(child->bts, at, &bts);
+               if (error < 0)
+                       return error;
 
-               if (copy_to_user(out, &ret, sizeof(ret)))
+               if (copy_to_user(out, &bts, sizeof(bts)))
                        return -EFAULT;
        }
 
-       error = ds_clear_bts(child);
+       memset(trace->ds.begin, 0, trace->ds.n * trace->ds.size);
+
+       error = ds_reset_bts(child->bts);
        if (error < 0)
                return error;
 
-       return end;
+       return drained;
+}
+
+static int ptrace_bts_allocate_buffer(struct task_struct *child, size_t size)
+{
+       child->bts_buffer = alloc_locked_buffer(size);
+       if (!child->bts_buffer)
+               return -ENOMEM;
+
+       child->bts_size = size;
+
+       return 0;
 }
 
-static void ptrace_bts_ovfl(struct task_struct *child)
+static void ptrace_bts_free_buffer(struct task_struct *child)
 {
-       send_sig(child->thread.bts_ovfl_signal, child, 0);
+       free_locked_buffer(child->bts_buffer, child->bts_size);
+       child->bts_buffer = NULL;
+       child->bts_size = 0;
 }
 
 static int ptrace_bts_config(struct task_struct *child,
@@ -714,114 +668,86 @@ static int ptrace_bts_config(struct task_struct *child,
                             const struct ptrace_bts_config __user *ucfg)
 {
        struct ptrace_bts_config cfg;
-       int error = 0;
-
-       error = -EOPNOTSUPP;
-       if (!bts_cfg.sizeof_bts)
-               goto errout;
+       unsigned int flags = 0;
 
-       error = -EIO;
        if (cfg_size < sizeof(cfg))
-               goto errout;
+               return -EIO;
 
-       error = -EFAULT;
        if (copy_from_user(&cfg, ucfg, sizeof(cfg)))
-               goto errout;
+               return -EFAULT;
 
-       error = -EINVAL;
-       if ((cfg.flags & PTRACE_BTS_O_SIGNAL) &&
-           !(cfg.flags & PTRACE_BTS_O_ALLOC))
-               goto errout;
+       if (child->bts) {
+               ds_release_bts(child->bts);
+               child->bts = NULL;
+       }
 
-       if (cfg.flags & PTRACE_BTS_O_ALLOC) {
-               ds_ovfl_callback_t ovfl = 0;
-               unsigned int sig = 0;
+       if (cfg.flags & PTRACE_BTS_O_SIGNAL) {
+               if (!cfg.signal)
+                       return -EINVAL;
 
-               /* we ignore the error in case we were not tracing child */
-               (void)ds_release_bts(child);
+               return -EOPNOTSUPP;
 
-               if (cfg.flags & PTRACE_BTS_O_SIGNAL) {
-                       if (!cfg.signal)
-                               goto errout;
+               child->thread.bts_ovfl_signal = cfg.signal;
+       }
 
-                       sig  = cfg.signal;
-                       ovfl = ptrace_bts_ovfl;
-               }
+       if ((cfg.flags & PTRACE_BTS_O_ALLOC) &&
+           (cfg.size != child->bts_size)) {
+               int error;
 
-               error = ds_request_bts(child, /* base = */ 0, cfg.size, ovfl);
-               if (error < 0)
-                       goto errout;
+               ptrace_bts_free_buffer(child);
 
-               child->thread.bts_ovfl_signal = sig;
+               error = ptrace_bts_allocate_buffer(child, cfg.size);
+               if (error < 0)
+                       return error;
        }
 
-       error = -EINVAL;
-       if (!child->thread.ds_ctx && cfg.flags)
-               goto errout;
-
        if (cfg.flags & PTRACE_BTS_O_TRACE)
-               child->thread.debugctlmsr |= bts_cfg.debugctl_mask;
-       else
-               child->thread.debugctlmsr &= ~bts_cfg.debugctl_mask;
+               flags |= BTS_USER;
 
        if (cfg.flags & PTRACE_BTS_O_SCHED)
-               set_tsk_thread_flag(child, TIF_BTS_TRACE_TS);
-       else
-               clear_tsk_thread_flag(child, TIF_BTS_TRACE_TS);
+               flags |= BTS_TIMESTAMPS;
 
-       error = sizeof(cfg);
+       child->bts = ds_request_bts(child, child->bts_buffer, child->bts_size,
+                                   /* ovfl = */ NULL, /* th = */ (size_t)-1,
+                                   flags);
+       if (IS_ERR(child->bts)) {
+               int error = PTR_ERR(child->bts);
 
-out:
-       if (child->thread.debugctlmsr)
-               set_tsk_thread_flag(child, TIF_DEBUGCTLMSR);
-       else
-               clear_tsk_thread_flag(child, TIF_DEBUGCTLMSR);
+               ptrace_bts_free_buffer(child);
+               child->bts = NULL;
 
-       return error;
+               return error;
+       }
 
-errout:
-       child->thread.debugctlmsr &= ~bts_cfg.debugctl_mask;
-       clear_tsk_thread_flag(child, TIF_BTS_TRACE_TS);
-       goto out;
+       return sizeof(cfg);
 }
 
 static int ptrace_bts_status(struct task_struct *child,
                             long cfg_size,
                             struct ptrace_bts_config __user *ucfg)
 {
+       const struct bts_trace *trace;
        struct ptrace_bts_config cfg;
-       size_t end;
-       const void *base, *max;
-       int error;
 
        if (cfg_size < sizeof(cfg))
                return -EIO;
 
-       error = ds_get_bts_end(child, &end);
-       if (error < 0)
-               return error;
-
-       error = ds_access_bts(child, /* index = */ 0, &base);
-       if (error < 0)
-               return error;
-
-       error = ds_access_bts(child, /* index = */ end, &max);
-       if (error < 0)
-               return error;
+       trace = ds_read_bts(child->bts);
+       if (!trace)
+               return -EPERM;
 
        memset(&cfg, 0, sizeof(cfg));
-       cfg.size = (max - base);
+       cfg.size = trace->ds.end - trace->ds.begin;
        cfg.signal = child->thread.bts_ovfl_signal;
        cfg.bts_size = sizeof(struct bts_struct);
 
        if (cfg.signal)
                cfg.flags |= PTRACE_BTS_O_SIGNAL;
 
-       if (test_tsk_thread_flag(child, TIF_DEBUGCTLMSR) &&
-           child->thread.debugctlmsr & bts_cfg.debugctl_mask)
+       if (trace->ds.flags & BTS_USER)
                cfg.flags |= PTRACE_BTS_O_TRACE;
 
-       if (test_tsk_thread_flag(child, TIF_BTS_TRACE_TS))
+       if (trace->ds.flags & BTS_TIMESTAMPS)
                cfg.flags |= PTRACE_BTS_O_SCHED;
 
        if (copy_to_user(ucfg, &cfg, sizeof(cfg)))
@@ -830,110 +756,81 @@ static int ptrace_bts_status(struct task_struct *child,
        return sizeof(cfg);
 }
 
-static int ptrace_bts_write_record(struct task_struct *child,
-                                  const struct bts_struct *in)
+static int ptrace_bts_clear(struct task_struct *child)
 {
-       unsigned char bts_record[BTS_MAX_RECORD_SIZE];
+       const struct bts_trace *trace;
 
-       BUG_ON(BTS_MAX_RECORD_SIZE < bts_cfg.sizeof_bts);
+       trace = ds_read_bts(child->bts);
+       if (!trace)
+               return -EPERM;
 
-       memset(bts_record, 0, bts_cfg.sizeof_bts);
-       switch (in->qualifier) {
-       case BTS_INVALID:
-               break;
+       memset(trace->ds.begin, 0, trace->ds.n * trace->ds.size);
 
-       case BTS_BRANCH:
-               bts_set(bts_record, bts_from, in->variant.lbr.from_ip);
-               bts_set(bts_record, bts_to,   in->variant.lbr.to_ip);
-               break;
+       return ds_reset_bts(child->bts);
+}
 
-       case BTS_TASK_ARRIVES:
-       case BTS_TASK_DEPARTS:
-               bts_set(bts_record, bts_from,    bts_escape);
-               bts_set(bts_record, bts_qual,    in->qualifier);
-               bts_set(bts_record, bts_jiffies, in->variant.jiffies);
-               break;
+static int ptrace_bts_size(struct task_struct *child)
+{
+       const struct bts_trace *trace;
 
-       default:
-               return -EINVAL;
-       }
+       trace = ds_read_bts(child->bts);
+       if (!trace)
+               return -EPERM;
 
-       /* The writing task will be the switched-to task on a context
-        * switch. It needs to write into the switched-from task's BTS
-        * buffer. */
-       return ds_unchecked_write_bts(child, bts_record, bts_cfg.sizeof_bts);
+       return (trace->ds.top - trace->ds.begin) / trace->ds.size;
 }
 
-void ptrace_bts_take_timestamp(struct task_struct *tsk,
-                              enum bts_qualifier qualifier)
+static void ptrace_bts_fork(struct task_struct *tsk)
 {
-       struct bts_struct rec = {
-               .qualifier = qualifier,
-               .variant.jiffies = jiffies_64
-       };
-
-       ptrace_bts_write_record(tsk, &rec);
+       tsk->bts = NULL;
+       tsk->bts_buffer = NULL;
+       tsk->bts_size = 0;
+       tsk->thread.bts_ovfl_signal = 0;
 }
 
-static const struct bts_configuration bts_cfg_netburst = {
-       .sizeof_bts    = sizeof(long) * 3,
-       .sizeof_field  = sizeof(long),
-       .debugctl_mask = (1<<2)|(1<<3)|(1<<5)
-};
-
-static const struct bts_configuration bts_cfg_pentium_m = {
-       .sizeof_bts    = sizeof(long) * 3,
-       .sizeof_field  = sizeof(long),
-       .debugctl_mask = (1<<6)|(1<<7)
-};
+static void ptrace_bts_untrace(struct task_struct *child)
+{
+       if (unlikely(child->bts)) {
+               ds_release_bts(child->bts);
+               child->bts = NULL;
+
+               /* We cannot update total_vm and locked_vm since
+                  child's mm is already gone. But we can reclaim the
+                  memory. */
+               kfree(child->bts_buffer);
+               child->bts_buffer = NULL;
+               child->bts_size = 0;
+       }
+}
 
-static const struct bts_configuration bts_cfg_core2 = {
-       .sizeof_bts    = 8 * 3,
-       .sizeof_field  = 8,
-       .debugctl_mask = (1<<6)|(1<<7)|(1<<9)
-};
+static void ptrace_bts_detach(struct task_struct *child)
+{
+       /*
+        * Ptrace_detach() races with ptrace_untrace() in case
+        * the child dies and is reaped by another thread.
+        *
+        * We only do the memory accounting at this point and
+        * leave the buffer deallocation and the bts tracer
+        * release to ptrace_bts_untrace() which will be called
+        * later on with tasklist_lock held.
+        */
+       release_locked_buffer(child->bts_buffer, child->bts_size);
+}
+#else
+static inline void ptrace_bts_fork(struct task_struct *tsk) {}
+static inline void ptrace_bts_detach(struct task_struct *child) {}
+static inline void ptrace_bts_untrace(struct task_struct *child) {}
+#endif /* CONFIG_X86_PTRACE_BTS */
 
-static inline void bts_configure(const struct bts_configuration *cfg)
+void x86_ptrace_fork(struct task_struct *child, unsigned long clone_flags)
 {
-       bts_cfg = *cfg;
+       ptrace_bts_fork(child);
 }
 
-void __cpuinit ptrace_bts_init_intel(struct cpuinfo_x86 *c)
+void x86_ptrace_untrace(struct task_struct *child)
 {
-       switch (c->x86) {
-       case 0x6:
-               switch (c->x86_model) {
-               case 0xD:
-               case 0xE: /* Pentium M */
-                       bts_configure(&bts_cfg_pentium_m);
-                       break;
-               case 0xF: /* Core2 */
-        case 0x1C: /* Atom */
-                       bts_configure(&bts_cfg_core2);
-                       break;
-               default:
-                       /* sorry, don't know about them */
-                       break;
-               }
-               break;
-       case 0xF:
-               switch (c->x86_model) {
-               case 0x0:
-               case 0x1:
-               case 0x2: /* Netburst */
-                       bts_configure(&bts_cfg_netburst);
-                       break;
-               default:
-                       /* sorry, don't know about them */
-                       break;
-               }
-               break;
-       default:
-               /* sorry, don't know about them */
-               break;
-       }
+       ptrace_bts_untrace(child);
 }
-#endif /* CONFIG_X86_PTRACE_BTS */
 
 /*
  * Called by kernel/ptrace.c when detaching..
@@ -946,15 +843,7 @@ void ptrace_disable(struct task_struct *child)
 #ifdef TIF_SYSCALL_EMU
        clear_tsk_thread_flag(child, TIF_SYSCALL_EMU);
 #endif
-#ifdef CONFIG_X86_PTRACE_BTS
-       (void)ds_release_bts(child);
-
-       child->thread.debugctlmsr &= ~bts_cfg.debugctl_mask;
-       if (!child->thread.debugctlmsr)
-               clear_tsk_thread_flag(child, TIF_DEBUGCTLMSR);
-
-       clear_tsk_thread_flag(child, TIF_BTS_TRACE_TS);
-#endif /* CONFIG_X86_PTRACE_BTS */
+       ptrace_bts_detach(child);
 }
 
 #if defined CONFIG_X86_32 || defined CONFIG_IA32_EMULATION
@@ -1086,7 +975,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                break;
 
        case PTRACE_BTS_SIZE:
-               ret = ds_get_bts_index(child, /* pos = */ 0);
+               ret = ptrace_bts_size(child);
                break;
 
        case PTRACE_BTS_GET:
@@ -1095,7 +984,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                break;
 
        case PTRACE_BTS_CLEAR:
-               ret = ds_clear_bts(child);
+               ret = ptrace_bts_clear(child);
                break;
 
        case PTRACE_BTS_DRAIN:
@@ -1358,6 +1247,14 @@ long compat_arch_ptrace(struct task_struct *child, compat_long_t request,
 
        case PTRACE_GET_THREAD_AREA:
        case PTRACE_SET_THREAD_AREA:
+#ifdef CONFIG_X86_PTRACE_BTS
+       case PTRACE_BTS_CONFIG:
+       case PTRACE_BTS_STATUS:
+       case PTRACE_BTS_SIZE:
+       case PTRACE_BTS_GET:
+       case PTRACE_BTS_CLEAR:
+       case PTRACE_BTS_DRAIN:
+#endif /* CONFIG_X86_PTRACE_BTS */
                return arch_ptrace(child, request, addr, data);
 
        default:
@@ -1384,6 +1281,12 @@ static const struct user_regset x86_64_regsets[] = {
                .size = sizeof(long), .align = sizeof(long),
                .active = xfpregs_active, .get = xfpregs_get, .set = xfpregs_set
        },
+       [REGSET_IOPERM64] = {
+               .core_note_type = NT_386_IOPERM,
+               .n = IO_BITMAP_LONGS,
+               .size = sizeof(long), .align = sizeof(long),
+               .active = ioperm_active, .get = ioperm_get
+       },
 };
 
 static const struct user_regset_view user_x86_64_view = {
@@ -1430,6 +1333,12 @@ static const struct user_regset x86_32_regsets[] = {
                .active = regset_tls_active,
                .get = regset_tls_get, .set = regset_tls_set
        },
+       [REGSET_IOPERM32] = {
+               .core_note_type = NT_386_IOPERM,
+               .n = IO_BITMAP_BYTES / sizeof(u32),
+               .size = sizeof(u32), .align = sizeof(u32),
+               .active = ioperm_active, .get = ioperm_get
+       },
 };
 
 static const struct user_regset_view user_x86_32_view = {
@@ -1451,7 +1360,8 @@ const struct user_regset_view *task_user_regset_view(struct task_struct *task)
 #endif
 }
 
-void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs, int error_code)
+void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs,
+                                        int error_code, int si_code)
 {
        struct siginfo info;
 
@@ -1460,7 +1370,7 @@ void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs, int error_code)
 
        memset(&info, 0, sizeof(info));
        info.si_signo = SIGTRAP;
-       info.si_code = TRAP_BRKPT;
+       info.si_code = si_code;
 
        /* User-mode ip? */
        info.si_addr = user_mode_vm(regs) ? (void __user *) regs->ip : NULL;
@@ -1469,30 +1379,6 @@ void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs, int error_code)
        force_sig_info(SIGTRAP, &info, tsk);
 }
 
-static void syscall_trace(struct pt_regs *regs)
-{
-       if (!(current->ptrace & PT_PTRACED))
-               return;
-
-#if 0
-       printk("trace %s ip %lx sp %lx ax %d origrax %d caller %lx tiflags %x ptrace %x\n",
-              current->comm,
-              regs->ip, regs->sp, regs->ax, regs->orig_ax, __builtin_return_address(0),
-              current_thread_info()->flags, current->ptrace);
-#endif
-
-       ptrace_notify(SIGTRAP | ((current->ptrace & PT_TRACESYSGOOD)
-                               ? 0x80 : 0));
-       /*
-        * this isn't the same as continuing with a signal, but it will do
-        * for normal use.  strace only continues with a signal if the
-        * stopping signal is not SIGTRAP.  -brl
-        */
-       if (current->exit_code) {
-               send_sig(current->exit_code, current, 1);
-               current->exit_code = 0;
-       }
-}
 
 #ifdef CONFIG_X86_32
 # define IS_IA32       1
@@ -1526,8 +1412,9 @@ asmregparm long syscall_trace_enter(struct pt_regs *regs)
        if (unlikely(test_thread_flag(TIF_SYSCALL_EMU)))
                ret = -1L;
 
-       if (ret || test_thread_flag(TIF_SYSCALL_TRACE))
-               syscall_trace(regs);
+       if ((ret || test_thread_flag(TIF_SYSCALL_TRACE)) &&
+           tracehook_report_syscall_entry(regs))
+               ret = -1L;
 
        if (unlikely(current->audit_context)) {
                if (IS_IA32)
@@ -1553,7 +1440,7 @@ asmregparm void syscall_trace_leave(struct pt_regs *regs)
                audit_syscall_exit(AUDITSC_RESULT(regs->ax), regs->ax);
 
        if (test_thread_flag(TIF_SYSCALL_TRACE))
-               syscall_trace(regs);
+               tracehook_report_syscall_exit(regs, 0);
 
        /*
         * If TIF_SYSCALL_EMU is set, we only get here because of
@@ -1569,6 +1456,6 @@ asmregparm void syscall_trace_leave(struct pt_regs *regs)
         * system call instruction.
         */
        if (test_thread_flag(TIF_SINGLESTEP) &&
-           (current->ptrace & PT_PTRACED))
-               send_sigtrap(current, regs, 0);
+           tracehook_consider_fatal_signal(current, SIGTRAP, SIG_DFL))
+               send_sigtrap(current, regs, 0, TRAP_BRKPT);
 }