h8300, exec: remove redundant set_fs(USER_DS)
authorMathias Krause <minipli@googlemail.com>
Tue, 26 Jul 2011 00:12:38 +0000 (17:12 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 26 Jul 2011 03:57:11 +0000 (20:57 -0700)
The address limit is already set in flush_old_exec() so those calls to
set_fs(USER_DS) are redundant.

Signed-off-by: Mathias Krause <minipli@googlemail.com>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/h8300/include/asm/processor.h

index 69e8a34eb6d589a4fd2f2e8bc950910a8549290f..e834b60188977b0a047df1d95d4e29e2cf9b36b1 100644 (file)
@@ -81,7 +81,6 @@ struct thread_struct {
 #if defined(__H8300H__)
 #define start_thread(_regs, _pc, _usp)                         \
 do {                                                           \
-       set_fs(USER_DS);           /* reads from user space */  \
        (_regs)->pc = (_pc);                                    \
        (_regs)->ccr = 0x00;       /* clear all flags */        \
        (_regs)->er5 = current->mm->start_data; /* GOT base */  \
@@ -91,7 +90,6 @@ do {                                                          \
 #if defined(__H8300S__)
 #define start_thread(_regs, _pc, _usp)                         \
 do {                                                           \
-       set_fs(USER_DS);           /* reads from user space */  \
        (_regs)->pc = (_pc);                                    \
        (_regs)->ccr = 0x00;       /* clear kernel flag */      \
        (_regs)->exr = 0x78;       /* enable all interrupts */  \