frv, exec: remove redundant set_fs(USER_DS)
authorMathias Krause <minipli@googlemail.com>
Tue, 26 Jul 2011 23:08:15 +0000 (16:08 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 26 Jul 2011 23:49:41 +0000 (16:49 -0700)
The address limit is already set in flush_old_exec() so those calls to
set_fs(USER_DS) are redundant.

Also removed the dead code in flush_thread().

Signed-off-by: Mathias Krause <minipli@googlemail.com>
Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/frv/include/asm/processor.h
arch/frv/kernel/process.c

index 4b789ab182b0879e008cb4a8b63943e3390c3bb2..81c2e271d620d90af8f55b9199ac3f1de773f850 100644 (file)
@@ -97,7 +97,6 @@ extern struct task_struct *__kernel_current_task;
  */
 #define start_thread(_regs, _pc, _usp)                 \
 do {                                                   \
-       set_fs(USER_DS); /* reads from user space */    \
        __frame = __kernel_frame0_ptr;                  \
        __frame->pc     = (_pc);                        \
        __frame->psr    &= ~PSR_S;                      \
index 9d359752646741b426596f4fb43671e6000cf18d..3901df1213c0f24454f37a755ddb3e83cd9d978b 100644 (file)
@@ -143,10 +143,7 @@ void machine_power_off(void)
 
 void flush_thread(void)
 {
-#if 0 //ndef NO_FPU
-       unsigned long zero = 0;
-#endif
-       set_fs(USER_DS);
+       /* nothing */
 }
 
 inline unsigned long user_stack(const struct pt_regs *regs)