[Xen-changelog] Clean up task-switch path. Now differs more from native code, but

[BitKeeper B. <ri...@su...>]

ChangeSet 1.1159.256.8, 2005/02/22 23:30:56+00:00, ka...@sc...

	Clean up task-switch path. Now differs more from native code, but
	it is a critical path. Also fixed a comment and FPU TS setting.
	Signed-off-by: Keir Fraser <ke...@xe...>



 process.c |   17 ++++-------------
 1 files changed, 4 insertions(+), 13 deletions(-)


diff -Nru a/linux-2.6.10-xen-sparse/arch/xen/i386/kernel/process.c b/linux-2.6.10-xen-sparse/arch/xen/i386/kernel/process.c
--- a/linux-2.6.10-xen-sparse/arch/xen/i386/kernel/process.c	2005-02-22 19:02:31 -05:00
+++ b/linux-2.6.10-xen-sparse/arch/xen/i386/kernel/process.c	2005-02-22 19:02:31 -05:00
@@ -481,11 +481,8 @@
 	asm volatile("movl %%gs,%0":"=m" (*(int *)&prev->gs));
 
 	/*
-	 * We clobber FS and GS here so that we avoid a GPF when
-	 * restoring previous task's FS/GS values in Xen when the LDT
-	 * is switched. If we don't do this then we can end up
-	 * erroneously re-flushing the page-update queue when we
-	 * 'execute_multicall_list'.
+	 * We clobber FS and GS here so that we avoid a GPF when restoring
+	 * previous task's FS/GS values in Xen when the LDT is switched.
 	 */
 	__asm__ __volatile__ ( 
 		"xorl %%eax,%%eax; movl %%eax,%%fs; movl %%eax,%%gs" : : :
@@ -501,7 +498,7 @@
 	 * synchronously trapping to Xen.
 	 */
 	if (prev_p->thread_info->status & TS_USEDFPU) {
-		save_init_fpu(prev_p);
+		__save_init_fpu(prev_p); /* _not_ save_init_fpu() */
 		queue_multicall0(__HYPERVISOR_fpu_taskswitch);
 	}
 
@@ -510,12 +507,6 @@
 	 * This is load_esp0(tss, next) with a multicall.
 	 */
 	tss->esp0 = next->esp0;
-	/* This can only happen when SEP is enabled, no need to test
-	 * "SEP"arately */
-	if (unlikely(tss->ss1 != next->sysenter_cs)) {
-		tss->ss1 = next->sysenter_cs;
-		wrmsr(MSR_IA32_SYSENTER_CS, next->sysenter_cs, 0);
-	}
 	queue_multicall2(__HYPERVISOR_stack_switch, tss->ss0, tss->esp0);
 
 	/*
@@ -549,7 +540,7 @@
 	/*
 	 * Restore %fs and %gs if needed.
 	 */
-	if (unlikely(prev->fs | prev->gs | next->fs | next->gs)) {
+	if (unlikely(next->fs | next->gs)) {
 		loadsegment(fs, next->fs);
 		loadsegment(gs, next->gs);
 	}