![Ingo van Lil](/assets/img/avatar_default.png)
Some time ago Jeff prepared 42daba3165
("uml: stop saving process FP
state") for UML to stop saving the process FP state between task
switches. The assumption was that since with SKAS0 every guest process
runs inside a host process context the host OS will take care of keeping
the proper FP state.
Unfortunately this is not true for multi-threaded applications, where
all guest threads share a single host process context yet all may use
the FPU on their own. Although I haven't verified it I suspect things
to be even worse in SKAS3 mode where all guest processes run inside a
single host process.
The patch reintroduces the saving and restoring of the FP context
between task switches.
[richard@nod.at: Ingo posted this patch in 2009, sadly it was never applied
and got lost. Now in 2011 the problem was reported by Gunnar.]
Signed-off-by: Ingo van Lil <inguin@gmx.de>
Signed-off-by: Richard Weinberger <richard@nod.at>
Reported-by: <gunnarlindroth@hotmail.com>
Tested-by: <gunnarlindroth@hotmail.com>
Cc: Stanislav Meduna <stano@meduna.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
59 lines
1.1 KiB
C
59 lines
1.1 KiB
C
/*
|
|
* Copyright (C) 2004 PathScale, Inc
|
|
* Copyright (C) 2004 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
|
|
* Licensed under the GPL
|
|
*/
|
|
|
|
#include <errno.h>
|
|
#include <string.h>
|
|
#include <sys/ptrace.h>
|
|
#include "sysdep/ptrace.h"
|
|
#include "sysdep/ptrace_user.h"
|
|
#include "registers.h"
|
|
|
|
int save_registers(int pid, struct uml_pt_regs *regs)
|
|
{
|
|
int err;
|
|
|
|
err = ptrace(PTRACE_GETREGS, pid, 0, regs->gp);
|
|
if (err < 0)
|
|
return -errno;
|
|
return 0;
|
|
}
|
|
|
|
int restore_registers(int pid, struct uml_pt_regs *regs)
|
|
{
|
|
int err;
|
|
|
|
err = ptrace(PTRACE_SETREGS, pid, 0, regs->gp);
|
|
if (err < 0)
|
|
return -errno;
|
|
return 0;
|
|
}
|
|
|
|
/* This is set once at boot time and not changed thereafter */
|
|
|
|
static unsigned long exec_regs[MAX_REG_NR];
|
|
static unsigned long exec_fp_regs[FP_SIZE];
|
|
|
|
int init_registers(int pid)
|
|
{
|
|
int err;
|
|
|
|
err = ptrace(PTRACE_GETREGS, pid, 0, exec_regs);
|
|
if (err < 0)
|
|
return -errno;
|
|
|
|
arch_init_registers(pid);
|
|
get_fp_registers(pid, exec_fp_regs);
|
|
return 0;
|
|
}
|
|
|
|
void get_safe_registers(unsigned long *regs, unsigned long *fp_regs)
|
|
{
|
|
memcpy(regs, exec_regs, sizeof(exec_regs));
|
|
|
|
if (fp_regs)
|
|
memcpy(fp_regs, exec_fp_regs, sizeof(exec_fp_regs));
|
|
}
|