* [PATCH] um, exec: remove redundant set_fs(USER_DS)
[not found] ` <BANLkTinv1teZEHMK0qymaVdcxPOErpqSyg@mail.gmail.com>
@ 2011-06-10 13:10 ` Mathias Krause
2011-06-10 20:00 ` [uml-devel] " Richard Weinberger
0 siblings, 1 reply; 2+ messages in thread
From: Mathias Krause @ 2011-06-10 13:10 UTC (permalink / raw)
To: Jeff Dike
Cc: Andrew Morton, Linus Torvalds, user-mode-linux-devel,
linux-kernel, Mathias Krause, Richard Weinberger
The address limit is already set in flush_old_exec() so this
set_fs(USER_DS) is redundant.
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Cc: Richard Weinberger <richard@nod.at>
---
arch/um/kernel/exec.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/arch/um/kernel/exec.c b/arch/um/kernel/exec.c
index 09bd7b5..939a4a6 100644
--- a/arch/um/kernel/exec.c
+++ b/arch/um/kernel/exec.c
@@ -38,7 +38,6 @@ void flush_thread(void)
void start_thread(struct pt_regs *regs, unsigned long eip, unsigned long esp)
{
- set_fs(USER_DS);
PT_REGS_IP(regs) = eip;
PT_REGS_SP(regs) = esp;
}
--
1.5.6.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [uml-devel] [PATCH] um, exec: remove redundant set_fs(USER_DS)
2011-06-10 13:10 ` [PATCH] um, exec: remove redundant set_fs(USER_DS) Mathias Krause
@ 2011-06-10 20:00 ` Richard Weinberger
0 siblings, 0 replies; 2+ messages in thread
From: Richard Weinberger @ 2011-06-10 20:00 UTC (permalink / raw)
To: Mathias Krause
Cc: Jeff Dike, Linus Torvalds, Andrew Morton, user-mode-linux-devel,
linux-kernel
Am Freitag 10 Juni 2011, 15:10:57 schrieb Mathias Krause:
> The address limit is already set in flush_old_exec() so this
> set_fs(USER_DS) is redundant.
>
> Signed-off-by: Mathias Krause <minipli@googlemail.com>
> Cc: Richard Weinberger <richard@nod.at>
Applied.
Thanks,
//richard
------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-10 20:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <BANLkTiknCeAxe30MJdVTxDom+ko8+EDQ4A@mail.gmail.com>
[not found] ` <1307642718-22257-1-git-send-email-minipli@googlemail.com>
[not found] ` <20110609155630.0f734351.akpm@linux-foundation.org>
[not found] ` <BANLkTinv1teZEHMK0qymaVdcxPOErpqSyg@mail.gmail.com>
2011-06-10 13:10 ` [PATCH] um, exec: remove redundant set_fs(USER_DS) Mathias Krause
2011-06-10 20:00 ` [uml-devel] " Richard Weinberger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox