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 69e8a34..e834b60 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 */  \