]> pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - arch/um/kernel/ptrace.c
arch/um/kernel/um_arch.c: some small improvements
[linux-2.6-omap-h63xx.git] / arch / um / kernel / ptrace.c
index 627742d894347c7ca6eb51f989b1e4c259de0909..47b57b497d5529e570591e97a0f041e0d55d842e 100644 (file)
@@ -1,35 +1,27 @@
-/* 
- * Copyright (C) 2000 Jeff Dike (jdike@karaya.com)
+/*
+ * Copyright (C) 2000 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
  * Licensed under the GPL
  */
 
-#include "linux/sched.h"
-#include "linux/mm.h"
-#include "linux/errno.h"
-#include "linux/smp_lock.h"
-#include "linux/security.h"
-#include "linux/ptrace.h"
 #include "linux/audit.h"
+#include "linux/ptrace.h"
+#include "linux/sched.h"
+#include "asm/uaccess.h"
 #ifdef CONFIG_PROC_MM
-#include "linux/proc_mm.h"
+#include "proc_mm.h"
 #endif
-#include "asm/ptrace.h"
-#include "asm/uaccess.h"
-#include "kern_util.h"
 #include "skas_ptrace.h"
-#include "sysdep/ptrace.h"
-#include "os.h"
 
 static inline void set_singlestepping(struct task_struct *child, int on)
 {
-        if (on)
-                child->ptrace |= PT_DTRACE;
-        else
-                child->ptrace &= ~PT_DTRACE;
-        child->thread.singlestep_syscall = 0;
+       if (on)
+               child->ptrace |= PT_DTRACE;
+       else
+               child->ptrace &= ~PT_DTRACE;
+       child->thread.singlestep_syscall = 0;
 
 #ifdef SUBARCH_SET_SINGLESTEPPING
-        SUBARCH_SET_SINGLESTEPPING(child, on);
+       SUBARCH_SET_SINGLESTEPPING(child, on);
 #endif
 }
 
@@ -37,8 +29,8 @@ static inline void set_singlestepping(struct task_struct *child, int on)
  * Called by kernel/ptrace.c when detaching..
  */
 void ptrace_disable(struct task_struct *child)
-{ 
-        set_singlestepping(child,0);
+{
+       set_singlestepping(child,0);
 }
 
 extern int peek_user(struct task_struct * child, long addr, long data);
@@ -50,52 +42,40 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
        unsigned long __user *p = (void __user *)(unsigned long)data;
 
        switch (request) {
-               /* when I and D space are separate, these will need to be fixed. */
-       case PTRACE_PEEKTEXT: /* read word at location addr. */ 
-       case PTRACE_PEEKDATA: {
-               unsigned long tmp;
-               int copied;
-
-               ret = -EIO;
-               copied = access_process_vm(child, addr, &tmp, sizeof(tmp), 0);
-               if (copied != sizeof(tmp))
-                       break;
-               ret = put_user(tmp, p);
+       /* read word at location addr. */
+       case PTRACE_PEEKTEXT:
+       case PTRACE_PEEKDATA:
+               ret = generic_ptrace_peekdata(child, addr, data);
                break;
-       }
 
        /* read the word at location addr in the USER area. */
-        case PTRACE_PEEKUSR:
-                ret = peek_user(child, addr, data);
-                break;
+       case PTRACE_PEEKUSR:
+               ret = peek_user(child, addr, data);
+               break;
 
-       /* when I and D space are separate, this will have to be fixed. */
-       case PTRACE_POKETEXT: /* write the word at location addr. */
+       /* write the word at location addr. */
+       case PTRACE_POKETEXT:
        case PTRACE_POKEDATA:
-               ret = -EIO;
-               if (access_process_vm(child, addr, &data, sizeof(data), 
-                                     1) != sizeof(data))
-                       break;
-               ret = 0;
+               ret = generic_ptrace_pokedata(child, addr, data);
                break;
 
-       case PTRACE_POKEUSR: /* write the word at location addr in the USER area */
-                ret = poke_user(child, addr, data);
-                break;
+       /* write the word at location addr in the USER area */
+       case PTRACE_POKEUSR:
+               ret = poke_user(child, addr, data);
+               break;
 
-       case PTRACE_SYSCALL: /* continue and stop at next (return from) syscall */
-       case PTRACE_CONT: { /* restart after signal. */
+       /* continue and stop at next (return from) syscall */
+       case PTRACE_SYSCALL:
+       /* restart after signal. */
+       case PTRACE_CONT: {
                ret = -EIO;
                if (!valid_signal(data))
                        break;
 
-                set_singlestepping(child, 0);
-               if (request == PTRACE_SYSCALL) {
+               set_singlestepping(child, 0);
+               if (request == PTRACE_SYSCALL)
                        set_tsk_thread_flag(child, TIF_SYSCALL_TRACE);
-               }
-               else {
-                       clear_tsk_thread_flag(child, TIF_SYSCALL_TRACE);
-               }
+               else clear_tsk_thread_flag(child, TIF_SYSCALL_TRACE);
                child->exit_code = data;
                wake_up_process(child);
                ret = 0;
@@ -103,8 +83,8 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
        }
 
 /*
- * make the child exit.  Best I can do is send it a sigkill. 
- * perhaps it should be put in the status that it wants to 
+ * make the child exit.  Best I can do is send it a sigkill.
+ * perhaps it should be put in the status that it wants to
  * exit.
  */
        case PTRACE_KILL: {
@@ -112,7 +92,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                if (child->exit_state == EXIT_ZOMBIE)   /* already dead */
                        break;
 
-                set_singlestepping(child, 0);
+               set_singlestepping(child, 0);
                child->exit_code = SIGKILL;
                wake_up_process(child);
                break;
@@ -123,7 +103,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                if (!valid_signal(data))
                        break;
                clear_tsk_thread_flag(child, TIF_SYSCALL_TRACE);
-                set_singlestepping(child, 1);
+               set_singlestepping(child, 1);
                child->exit_code = data;
                /* give it a chance to run. */
                wake_up_process(child);
@@ -131,11 +111,6 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                break;
        }
 
-       case PTRACE_DETACH:
-               /* detach a process that was attached. */
-               ret = ptrace_detach(child, data);
-               break;
-
 #ifdef PTRACE_GETREGS
        case PTRACE_GETREGS: { /* Get all gp regs from the child. */
                if (!access_ok(VERIFY_WRITE, p, MAX_REG_OFFSET)) {
@@ -168,22 +143,14 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
 #endif
 #ifdef PTRACE_GETFPREGS
        case PTRACE_GETFPREGS: /* Get the child FPU state. */
-               ret = get_fpregs(data, child);
+               ret = get_fpregs((struct user_i387_struct __user *) data,
+                                child);
                break;
 #endif
 #ifdef PTRACE_SETFPREGS
        case PTRACE_SETFPREGS: /* Set the child FPU state. */
-               ret = set_fpregs(data, child);
-               break;
-#endif
-#ifdef PTRACE_GETFPXREGS
-       case PTRACE_GETFPXREGS: /* Get the child FPU state. */
-               ret = get_fpxregs(data, child);
-               break;
-#endif
-#ifdef PTRACE_SETFPXREGS
-       case PTRACE_SETFPXREGS: /* Set the child FPU state. */
-               ret = set_fpxregs(data, child);
+               ret = set_fpregs((struct user_i387_struct __user *) data,
+                                child);
                break;
 #endif
        case PTRACE_GET_THREAD_AREA:
@@ -197,14 +164,13 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                break;
 
        case PTRACE_FAULTINFO: {
-               /* Take the info from thread->arch->faultinfo,
+               /*
+                * Take the info from thread->arch->faultinfo,
                 * but transfer max. sizeof(struct ptrace_faultinfo).
                 * On i386, ptrace_faultinfo is smaller!
                 */
                ret = copy_to_user(p, &child->thread.arch.faultinfo,
                                   sizeof(struct ptrace_faultinfo));
-               if(ret)
-                       break;
                break;
        }
 
@@ -212,12 +178,13 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
        case PTRACE_LDT: {
                struct ptrace_ldt ldt;
 
-               if(copy_from_user(&ldt, p, sizeof(ldt))){
+               if (copy_from_user(&ldt, p, sizeof(ldt))) {
                        ret = -EIO;
                        break;
                }
 
-               /* This one is confusing, so just punt and return -EIO for 
+               /*
+                * This one is confusing, so just punt and return -EIO for
                 * now
                 */
                ret = -EIO;
@@ -229,7 +196,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
                struct mm_struct *old = child->mm;
                struct mm_struct *new = proc_mm_get_mm(data);
 
-               if(IS_ERR(new)){
+               if (IS_ERR(new)) {
                        ret = PTR_ERR(new);
                        break;
                }
@@ -243,20 +210,22 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
        }
 #endif
 #ifdef PTRACE_ARCH_PRCTL
-        case PTRACE_ARCH_PRCTL:
-                /* XXX Calls ptrace on the host - needs some SMP thinking */
-                ret = arch_prctl_skas(child, data, (void *) addr);
-                break;
+       case PTRACE_ARCH_PRCTL:
+               /* XXX Calls ptrace on the host - needs some SMP thinking */
+               ret = arch_prctl(child, data, (void *) addr);
+               break;
 #endif
        default:
                ret = ptrace_request(child, request, addr, data);
+               if (ret == -EIO)
+                       ret = subarch_ptrace(child, request, addr, data);
                break;
        }
 
        return ret;
 }
 
-void send_sigtrap(struct task_struct *tsk, union uml_pt_regs *regs,
+void send_sigtrap(struct task_struct *tsk, struct uml_pt_regs *regs,
                  int error_code)
 {
        struct siginfo info;
@@ -268,14 +237,15 @@ void send_sigtrap(struct task_struct *tsk, union uml_pt_regs *regs,
        /* User-mode eip? */
        info.si_addr = UPT_IS_USER(regs) ? (void __user *) UPT_IP(regs) : NULL;
 
-       /* Send us the fakey SIGTRAP */
+       /* Send us the fake SIGTRAP */
        force_sig_info(SIGTRAP, &info, tsk);
 }
 
-/* XXX Check PT_DTRACE vs TIF_SINGLESTEP for singlestepping check and
+/*
+ * XXX Check PT_DTRACE vs TIF_SINGLESTEP for singlestepping check and
  * PT_PTRACED vs TIF_SYSCALL_TRACE for syscall tracing check
  */
-void syscall_trace(union uml_pt_regs *regs, int entryexit)
+void syscall_trace(struct uml_pt_regs *regs, int entryexit)
 {
        int is_singlestep = (current->ptrace & PT_DTRACE) && entryexit;
        int tracesysgood;
@@ -289,7 +259,7 @@ void syscall_trace(union uml_pt_regs *regs, int entryexit)
                                            UPT_SYSCALL_ARG3(regs),
                                            UPT_SYSCALL_ARG4(regs));
                else audit_syscall_exit(AUDITSC_RESULT(UPT_SYSCALL_RET(regs)),
-                                        UPT_SYSCALL_RET(regs));
+                                       UPT_SYSCALL_RET(regs));
        }
 
        /* Fake a debug trap */
@@ -302,15 +272,18 @@ void syscall_trace(union uml_pt_regs *regs, int entryexit)
        if (!(current->ptrace & PT_PTRACED))
                return;
 
-       /* the 0x80 provides a way for the tracing parent to distinguish
-          between a syscall stop and SIGTRAP delivery */
+       /*
+        * the 0x80 provides a way for the tracing parent to distinguish
+        * between a syscall stop and SIGTRAP delivery
+        */
        tracesysgood = (current->ptrace & PT_TRACESYSGOOD);
        ptrace_notify(SIGTRAP | (tracesysgood ? 0x80 : 0));
 
        if (entryexit) /* force do_signal() --> is_syscall() */
                set_thread_flag(TIF_SIGPENDING);
 
-       /* this isn't the same as continuing with a signal, but it will do
+       /*
+        * 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
         */