x86: x86 user_regset cleanup
authorRoland McGrath <roland@redhat.com>
Wed, 30 Jan 2008 12:31:55 +0000 (13:31 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 30 Jan 2008 12:31:55 +0000 (13:31 +0100)
This removes a bunch of dead code that is no longer needed now
that the user_regset interfaces are being used for all these jobs.

Signed-off-by: Roland McGrath <roland@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/x86/kernel/i387.c
arch/x86/kernel/process_32.c
arch/x86/kernel/process_64.c
arch/x86/math-emu/fpu_entry.c
include/asm-x86/elf.h
include/asm-x86/i387.h
include/asm-x86/math_emu.h

index f7f7568dd7bc194e6c8a0f0f7ab88e97b0641947..26719bd2c77ce01c8e087fca9c86ad5d3f067278 100644 (file)
@@ -454,113 +454,26 @@ int restore_i387_ia32(struct _fpstate_ia32 __user *buf)
        return err;
 }
 
-#endif /* CONFIG_X86_32 || CONFIG_IA32_EMULATION */
-
-#ifdef CONFIG_X86_64
-
-int get_fpregs(struct user_i387_struct __user *buf, struct task_struct *tsk)
-{
-       return xfpregs_get(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-int set_fpregs(struct task_struct *tsk, struct user_i387_struct __user *buf)
-{
-       return xfpregs_set(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-#else
-
-int get_fpregs(struct user_i387_struct __user *buf, struct task_struct *tsk)
-{
-       return fpregs_get(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-int set_fpregs(struct task_struct *tsk, struct user_i387_struct __user *buf)
-{
-       return fpregs_set(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-int get_fpxregs(struct user_fxsr_struct __user *buf, struct task_struct *tsk)
-{
-       return xfpregs_get(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-int set_fpxregs(struct task_struct *tsk, struct user_fxsr_struct __user *buf)
-{
-       return xfpregs_get(tsk, NULL, 0, sizeof(*buf), NULL, buf);
-}
-
-#endif
-
 /*
  * FPU state for core dumps.
+ * This is only used for a.out dumps now.
+ * It is declared generically using elf_fpregset_t (which is
+ * struct user_i387_struct) but is in fact only used for 32-bit
+ * dumps, so on 64-bit it is really struct user_i387_ia32_struct.
  */
-
-static inline void copy_fpu_fsave(struct task_struct *tsk,
-                                 struct user_i387_struct *fpu)
-{
-       memcpy(fpu, &tsk->thread.i387.fsave,
-              sizeof(struct user_i387_struct));
-}
-
-static inline void copy_fpu_fxsave(struct task_struct *tsk,
-                                  struct user_i387_struct *fpu)
-{
-       unsigned short *to;
-       unsigned short *from;
-       int i;
-
-       memcpy(fpu, &tsk->thread.i387.fxsave, 7 * sizeof(long));
-
-       to = (unsigned short *)&fpu->st_space[0];
-       from = (unsigned short *)&tsk->thread.i387.fxsave.st_space[0];
-       for (i = 0; i < 8; i++, to += 5, from += 8)
-               memcpy(to, from, 5 * sizeof(unsigned short));
-}
-
 int dump_fpu(struct pt_regs *regs, struct user_i387_struct *fpu)
 {
        int fpvalid;
        struct task_struct *tsk = current;
 
        fpvalid = !!used_math();
-       if (fpvalid) {
-               unlazy_fpu(tsk);
-               if (cpu_has_fxsr) {
-                       copy_fpu_fxsave(tsk, fpu);
-               } else {
-                       copy_fpu_fsave(tsk, fpu);
-               }
-       }
+       if (fpvalid)
+               fpvalid = !fpregs_get(tsk, NULL,
+                                     0, sizeof(struct user_i387_ia32_struct),
+                                     fpu, NULL);
 
        return fpvalid;
 }
 EXPORT_SYMBOL(dump_fpu);
 
-int dump_task_fpu(struct task_struct *tsk, struct user_i387_struct *fpu)
-{
-       int fpvalid = !!tsk_used_math(tsk);
-
-       if (fpvalid) {
-               if (tsk == current)
-                       unlazy_fpu(tsk);
-               if (cpu_has_fxsr)
-                       copy_fpu_fxsave(tsk, fpu);
-               else
-                       copy_fpu_fsave(tsk, fpu);
-       }
-       return fpvalid;
-}
-
-int dump_task_extended_fpu(struct task_struct *tsk,
-                          struct user32_fxsr_struct *fpu)
-{
-       int fpvalid = tsk_used_math(tsk) && cpu_has_fxsr;
-
-       if (fpvalid) {
-               if (tsk == current)
-                      unlazy_fpu(tsk);
-               memcpy(fpu, &tsk->thread.i387.fxsave, sizeof(*fpu));
-       }
-       return fpvalid;
-}
+#endif /* CONFIG_X86_32 || CONFIG_IA32_EMULATION */
index 40cc29695ebac8c8231930f84c40986c78c0227c..35a6f318c5417eabc937092970c32f7e470a1bf4 100644 (file)
@@ -571,22 +571,6 @@ void dump_thread(struct pt_regs * regs, struct user * dump)
 }
 EXPORT_SYMBOL(dump_thread);
 
-/*
- * Capture the user space registers if the task is not running (in user space)
- */
-int dump_task_regs(struct task_struct *tsk, elf_gregset_t *regs)
-{
-       struct pt_regs ptregs = *task_pt_regs(tsk);
-       ptregs.cs &= 0xffff;
-       ptregs.ds &= 0xffff;
-       ptregs.es &= 0xffff;
-       ptregs.ss &= 0xffff;
-
-       elf_core_copy_regs(regs, &ptregs);
-
-       return 1;
-}
-
 #ifdef CONFIG_SECCOMP
 static void hard_disable_TSC(void)
 {
index e3a3610ade104a522e8378758032ed884ec9b7c2..78d80067b7f90e5b60a164af07219e65560add92 100644 (file)
@@ -544,24 +544,6 @@ out:
  */
 #define loaddebug(thread, r) set_debugreg(thread->debugreg ## r, r)
 
-/*
- * Capture the user space registers if the task is not running (in user space)
- */
-int dump_task_regs(struct task_struct *tsk, elf_gregset_t *regs)
-{
-       struct pt_regs *pp, ptregs;
-
-       pp = task_pt_regs(tsk);
-
-       ptregs = *pp;
-       ptregs.cs &= 0xffff;
-       ptregs.ss &= 0xffff;
-
-       elf_core_copy_regs(regs, &ptregs);
-
-       return 1;
-}
-
 static inline void __switch_to_xtra(struct task_struct *prev_p,
                                    struct task_struct *next_p,
                                    struct tss_struct *tss)
@@ -929,4 +911,3 @@ unsigned long arch_randomize_brk(struct mm_struct *mm)
        unsigned long range_end = mm->brk + 0x02000000;
        return randomize_range(mm->brk, range_end, 0) ? : mm->brk;
 }
-
index cfbdaa1532ce98af8997198d7f71f8a744814ad1..760baeea5f07c9ad13858d19faa1060a7de3d849 100644 (file)
@@ -761,17 +761,3 @@ int fpregs_soft_get(struct task_struct *target,
 
        return ret;
 }
-
-int save_i387_soft(void *s387, struct _fpstate __user *buf)
-{
-       return fpregs_soft_get(current, NULL,
-                              0, sizeof(struct user_i387_struct),
-                              NULL, buf) ? -1 : 1;
-}
-
-int restore_i387_soft(void *s387, struct _fpstate __user *buf)
-{
-       return fpregs_soft_set(current, NULL,
-                              0, sizeof(struct user_i387_struct),
-                              NULL, buf) ? -1 : 1;
-}
index 123e2d599c6d407c63ca86ad5e340b1de76976c6..d6bf7421d7b09bb5e01e72701cd5496ccd559884 100644 (file)
@@ -104,28 +104,6 @@ typedef struct user_fxsr_struct elf_fpxregset_t;
        _r->ax = 0; \
 } while (0)
 
-/* regs is struct pt_regs, pr_reg is elf_gregset_t (which is
-   now struct_user_regs, they are different) */
-
-#define ELF_CORE_COPY_REGS(pr_reg, regs)               \
-       pr_reg[0] = regs->bx;                           \
-       pr_reg[1] = regs->cx;                           \
-       pr_reg[2] = regs->dx;                           \
-       pr_reg[3] = regs->si;                           \
-       pr_reg[4] = regs->di;                           \
-       pr_reg[5] = regs->bp;                           \
-       pr_reg[6] = regs->ax;                           \
-       pr_reg[7] = regs->ds & 0xffff;                  \
-       pr_reg[8] = regs->es & 0xffff;                  \
-       pr_reg[9] = regs->fs & 0xffff;                  \
-       savesegment(gs,pr_reg[10]);                     \
-       pr_reg[11] = regs->orig_ax;                     \
-       pr_reg[12] = regs->ip;                          \
-       pr_reg[13] = regs->cs & 0xffff;                 \
-       pr_reg[14] = regs->flags;                       \
-       pr_reg[15] = regs->sp;                          \
-       pr_reg[16] = regs->ss & 0xffff;
-
 #define ELF_PLATFORM   (utsname()->machine)
 #define set_personality_64bit()        do { } while (0)
 extern unsigned int vdso_enabled;
@@ -159,41 +137,6 @@ extern unsigned int vdso_enabled;
        clear_thread_flag(TIF_IA32);                      \
 } while (0)
 
-/* regs is struct pt_regs, pr_reg is elf_gregset_t (which is
-   now struct_user_regs, they are different). Assumes current is the process
-   getting dumped. */
-
-#define ELF_CORE_COPY_REGS(pr_reg, regs)  do {                 \
-       unsigned v;                                             \
-       (pr_reg)[0] = (regs)->r15;                              \
-       (pr_reg)[1] = (regs)->r14;                              \
-       (pr_reg)[2] = (regs)->r13;                              \
-       (pr_reg)[3] = (regs)->r12;                              \
-       (pr_reg)[4] = (regs)->bp;                               \
-       (pr_reg)[5] = (regs)->bx;                               \
-       (pr_reg)[6] = (regs)->r11;                              \
-       (pr_reg)[7] = (regs)->r10;                              \
-       (pr_reg)[8] = (regs)->r9;                               \
-       (pr_reg)[9] = (regs)->r8;                               \
-       (pr_reg)[10] = (regs)->ax;                              \
-       (pr_reg)[11] = (regs)->cx;                              \
-       (pr_reg)[12] = (regs)->dx;                              \
-       (pr_reg)[13] = (regs)->si;                              \
-       (pr_reg)[14] = (regs)->di;                              \
-       (pr_reg)[15] = (regs)->orig_ax;                 \
-       (pr_reg)[16] = (regs)->ip;                              \
-       (pr_reg)[17] = (regs)->cs;                              \
-       (pr_reg)[18] = (regs)->flags;                           \
-       (pr_reg)[19] = (regs)->sp;                              \
-       (pr_reg)[20] = (regs)->ss;                              \
-       (pr_reg)[21] = current->thread.fs;                      \
-       (pr_reg)[22] = current->thread.gs;                      \
-       asm("movl %%ds,%0" : "=r" (v)); (pr_reg)[23] = v;       \
-       asm("movl %%es,%0" : "=r" (v)); (pr_reg)[24] = v;       \
-       asm("movl %%fs,%0" : "=r" (v)); (pr_reg)[25] = v;       \
-       asm("movl %%gs,%0" : "=r" (v)); (pr_reg)[26] = v;       \
-} while(0);
-
 /* I'm not sure if we can use '-' here */
 #define ELF_PLATFORM       ("x86_64")
 extern void set_personality_64bit(void);
@@ -236,18 +179,7 @@ extern int vdso_enabled;
 
 struct task_struct;
 
-extern int dump_task_regs (struct task_struct *, elf_gregset_t *);
-extern int dump_task_fpu (struct task_struct *, elf_fpregset_t *);
-
-#define ELF_CORE_COPY_TASK_REGS(tsk, elf_regs) dump_task_regs(tsk, elf_regs)
-#define ELF_CORE_COPY_FPREGS(tsk, elf_fpregs) dump_task_fpu(tsk, elf_fpregs)
-
 #ifdef CONFIG_X86_32
-extern int dump_task_extended_fpu (struct task_struct *,
-                                  struct user_fxsr_struct *);
-#define ELF_CORE_COPY_XFPREGS(tsk, elf_xfpregs) \
-       dump_task_extended_fpu(tsk, elf_xfpregs)
-#define ELF_CORE_XFPREG_TYPE NT_PRXFPREG
 
 #define VDSO_HIGH_BASE         (__fix_to_virt(FIX_VDSO))
 
index de435b9114dfbc91bef129bcc2893c94edddb5a9..ba8105ca822b99840f6bcf91b23e9d7f0c34902f 100644 (file)
@@ -327,20 +327,6 @@ static inline void clear_fpu(struct task_struct *tsk)
 
 #endif /* CONFIG_X86_64 */
 
-/*
- * ptrace request handlers...
- */
-extern int get_fpregs(struct user_i387_struct __user *buf,
-                     struct task_struct *tsk);
-extern int set_fpregs(struct task_struct *tsk,
-                     struct user_i387_struct __user *buf);
-
-struct user_fxsr_struct;
-extern int get_fpxregs(struct user_fxsr_struct __user *buf,
-                      struct task_struct *tsk);
-extern int set_fpxregs(struct task_struct *tsk,
-                      struct user_fxsr_struct __user *buf);
-
 /*
  * i387 state interaction
  */
index a4b0aa3320e681f802cb7b297766ec58f26d7f9c..9bf4ae93ab10b3a549efaac717fcdce0f81a158d 100644 (file)
@@ -1,11 +1,6 @@
 #ifndef _I386_MATH_EMU_H
 #define _I386_MATH_EMU_H
 
-#include <asm/sigcontext.h>
-
-int restore_i387_soft(void *s387, struct _fpstate __user *buf);
-int save_i387_soft(void *s387, struct _fpstate __user *buf);
-
 /* This structure matches the layout of the data saved to the stack
    following a device-not-present interrupt, part of it saved
    automatically by the 80386/80486.