x86/HPET: drop useless check
authorJan Beulich <jbeulich@suse.com>
Mon, 2 Jul 2018 11:10:19 +0000 (13:10 +0200)
committerJan Beulich <jbeulich@suse.com>
Mon, 2 Jul 2018 11:10:19 +0000 (13:10 +0200)
Commit 9e051a840d ("x86/hpet: Improve handling of timer_deadline")
removed all code between for_each_cpu() and cpumask_test_cpu(),
rendering the latter pointless.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@ctirix.com>
xen/arch/x86/hpet.c

index 13b8a2206ec55b04c4eaeb0f6195b5fde5ac5596..4b08488ef1afa3a31488f60037ab782df423fe38 100644 (file)
@@ -187,12 +187,7 @@ again:
     /* find all expired events */
     for_each_cpu(cpu, ch->cpumask)
     {
-        s_time_t deadline;
-
-        if ( !cpumask_test_cpu(cpu, ch->cpumask) )
-            continue;
-
-        deadline = ACCESS_ONCE(per_cpu(timer_deadline, cpu));
+        s_time_t deadline = ACCESS_ONCE(per_cpu(timer_deadline, cpu));
 
         if ( deadline <= now )
             __cpumask_set_cpu(cpu, &mask);