From patchwork Fri Jun 10 13:10:16 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mathias Krause X-Patchwork-Id: 869532 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.4) with ESMTP id p5ADKAtH012847 for ; Fri, 10 Jun 2011 13:20:11 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755656Ab1FJNUK (ORCPT ); Fri, 10 Jun 2011 09:20:10 -0400 Received: from grimli.r00tworld.net ([83.169.44.195]:56693 "EHLO mail.r00tworld.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755636Ab1FJNUJ (ORCPT ); Fri, 10 Jun 2011 09:20:09 -0400 Received: by mail.r00tworld.net (Postfix, from userid 1000) id CC6F4115901B4; Fri, 10 Jun 2011 15:10:16 +0200 (CEST) From: Mathias Krause To: "James E.J. Bottomley" Cc: Andrew Morton , Linus Torvalds , linux-parisc@vger.kernel.org, linux-kernel@vger.kernel.org, Mathias Krause , Kyle McMartin , Helge Deller Subject: [PATCH] parisc, exec: remove redundant set_fs(USER_DS) Date: Fri, 10 Jun 2011 15:10:16 +0200 Message-Id: <1307711416-14119-1-git-send-email-minipli@googlemail.com> X-Mailer: git-send-email 1.5.6.5 In-Reply-To: References: References: <1307642718-22257-1-git-send-email-minipli@googlemail.com> <20110609155630.0f734351.akpm@linux-foundation.org> Sender: linux-parisc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-parisc@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Fri, 10 Jun 2011 13:20:11 +0000 (UTC) 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 Cc: Kyle McMartin Cc: Helge Deller --- arch/parisc/include/asm/processor.h | 2 -- arch/parisc/kernel/process.c | 1 - 2 files changed, 0 insertions(+), 3 deletions(-) diff --git a/arch/parisc/include/asm/processor.h b/arch/parisc/include/asm/processor.h index 9ce66e9..7213ec9 100644 --- a/arch/parisc/include/asm/processor.h +++ b/arch/parisc/include/asm/processor.h @@ -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; \ diff --git a/arch/parisc/kernel/process.c b/arch/parisc/kernel/process.c index 4b4b918..62c60b8 100644 --- a/arch/parisc/kernel/process.c +++ b/arch/parisc/kernel/process.c @@ -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)