x86/PV: drop pointless conditional from pv_cpuid()'s state leaf logic
authorJan Beulich <jbeulich@suse.com>
Thu, 9 Jun 2016 13:49:52 +0000 (15:49 +0200)
committerJan Beulich <jbeulich@suse.com>
Thu, 9 Jun 2016 13:49:52 +0000 (15:49 +0200)
In the control/hardware domain case without it we simply re-read the
same value that was put into b near the top of the function.

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

index 1b42841c7f494fcff7d6bdafe03c082ae59f01e2..f604b89f12475b89e2bf34b6fb7114da63ed26d2 100644 (file)
@@ -1119,8 +1119,7 @@ void pv_cpuid(struct cpu_user_regs *regs)
              * domain policy.  It varies with enabled xstate, and the correct
              * xcr0 is in context.
              */
-            if ( !is_control_domain(currd) && !is_hardware_domain(currd) )
-                cpuid_count(leaf, subleaf, &tmp, &b, &tmp, &tmp);
+            cpuid_count(leaf, subleaf, &tmp, &b, &tmp, &tmp);
             break;
         }