diff options
author | Suresh Siddha <suresh.b.siddha@intel.com> | 2008-03-10 18:28:04 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-04-19 13:19:55 -0400 |
commit | 61c4628b538608c1a85211ed8438136adfeb9a95 (patch) | |
tree | 290a695299a363153bc692e6d705ac680d64359e /arch/x86/kernel/traps_64.c | |
parent | fa5c4639419668cbb18ca3d20c1253559a3b43ae (diff) |
x86, fpu: split FPU state from task struct - v5
Split the FPU save area from the task struct. This allows easy migration
of FPU context, and it's generally cleaner. It also allows the following
two optimizations:
1) only allocate when the application actually uses FPU, so in the first
lazy FPU trap. This could save memory for non-fpu using apps. Next patch
does this lazy allocation.
2) allocate the right size for the actual cpu rather than 512 bytes always.
Patches enabling xsave/xrstor support (coming shortly) will take advantage
of this.
Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/kernel/traps_64.c')
-rw-r--r-- | arch/x86/kernel/traps_64.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/arch/x86/kernel/traps_64.c b/arch/x86/kernel/traps_64.c index 6d883b13ef4f..dc0cb497eec3 100644 --- a/arch/x86/kernel/traps_64.c +++ b/arch/x86/kernel/traps_64.c | |||
@@ -1128,7 +1128,7 @@ asmlinkage void math_state_restore(void) | |||
1128 | 1128 | ||
1129 | if (!used_math()) | 1129 | if (!used_math()) |
1130 | init_fpu(me); | 1130 | init_fpu(me); |
1131 | restore_fpu_checking(&me->thread.i387.fxsave); | 1131 | restore_fpu_checking(&me->thread.xstate->fxsave); |
1132 | task_thread_info(me)->status |= TS_USEDFPU; | 1132 | task_thread_info(me)->status |= TS_USEDFPU; |
1133 | me->fpu_counter++; | 1133 | me->fpu_counter++; |
1134 | } | 1134 | } |
@@ -1164,6 +1164,10 @@ void __init trap_init(void) | |||
1164 | #endif | 1164 | #endif |
1165 | 1165 | ||
1166 | /* | 1166 | /* |
1167 | * initialize the per thread extended state: | ||
1168 | */ | ||
1169 | init_thread_xstate(); | ||
1170 | /* | ||
1167 | * Should be a barrier for any external CPU state. | 1171 | * Should be a barrier for any external CPU state. |
1168 | */ | 1172 | */ |
1169 | cpu_init(); | 1173 | cpu_init(); |