uml: fix timer switching

Fix up the switching between virtual and real timers.  The idle loop sleeps,
so the timer at that point must be real time.  At all other times, the timer
must be virtual.  Even when userspace is running, and the kernel is asleep,
the virtual timer is correct because the process timer will be running and the
process timer will be firing.

The timer switch used to be in the context switch and timer handler code.
This is moved to the idle loop and the signal handler, making it much more
clear why it is happening.

switch_timers now returns the old timer type so that it may be restored.  The
signal handler uses this in order to restore the previous timer type when it
returns.

Signed-off-by: Jeff Dike <jdike@linux.intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

1 files changed, 2 insertions, 7 deletions

diff --git a/arch/um/kernel/process.c b/arch/um/kernel/process.c
index 8a6882dfba01..56d75afedbf7 100644
--- a/arch/um/kernel/process.c
+++ b/arch/um/kernel/process.c
@@ -95,18 +95,11 @@ void *_switch_to(void *prev, void *next, void *last)
         do {
                 current->thread.saved_task = NULL;
 
-                /* XXX need to check runqueues[cpu].idle */
-                if (current->pid == 0)
-                        switch_timers(0);
-
                 switch_threads(&from->thread.switch_buf,
                                &to->thread.switch_buf);
 
                 arch_switch_to(current->thread.prev_sched, current);
 
-                if (current->pid == 0)
-                        switch_timers(1);
-
                 if (current->thread.saved_task)
                         show_regs(&(current->thread.regs));
                 next= current->thread.saved_task;
@@ -251,7 +244,9 @@ void default_idle(void)
                 if (need_resched())
                         schedule();
 
+                switch_timers(1);
                 idle_sleep(10);
+                switch_timers(0);
         }
 }