Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 15ee2d00 authored Jan 12, 2012 by Mathias Krause Committed by Linus Torvalds Jan 12, 2012

parisc, exec: remove redundant set_fs(USER_DS)



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: Kyle McMartin <kyle@mcmartin.ca>
Cc: Helge Deller <deller@gmx.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent 01fa310c

arch/parisc/include/asm/processor.h

+0 −2

Original line number	Diff line number	Diff line
		@@ -196,7 +196,6 @@ typedef unsigned int elf_caddr_t;
		/* offset pc for priv. level */ \
		pc \|= 3; \
		\
		set_fs(USER_DS); \
		regs->iasq[0] = spaceid; \
		regs->iasq[1] = spaceid; \
		regs->iaoq[0] = pc; \
		@@ -299,7 +298,6 @@ on downward growing arches, it looks like this:
		elf_addr_t pc = (elf_addr_t)new_pc \| 3; \
		elf_caddr_t argv = (elf_caddr_t )bprm->exec + 1; \
		\
		set_fs(USER_DS); \
		regs->iasq[0] = spaceid; \
		regs->iasq[1] = spaceid; \
		regs->iaoq[0] = pc; \

arch/parisc/kernel/process.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -192,7 +192,6 @@ void flush_thread(void)
		/* Only needs to handle fpu stuff or perf monitors.
		** REVISIT: several arches implement a "lazy fpu state".
		*/
		set_fs(USER_DS);
		}

		void release_thread(struct task_struct *dead_task)