x86/APIC: drop 32-bit days remnants
authorJan Beulich <jbeulich@suse.com>
Mon, 24 Jan 2022 07:40:13 +0000 (08:40 +0100)
committerJan Beulich <jbeulich@suse.com>
Mon, 24 Jan 2022 07:40:13 +0000 (08:40 +0100)
Mercury and Neptune were Pentium chipsets - no need to work around their
errata, even more so that the workaround looks fragile.

Also ditch a Pentium-related and stale part of a comment.

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

index 5d3e1f3952d7c813b6dbfcb2665af3dc2af553ff..583656158532621f74c340ef3efc44f9b6ba3722 100644 (file)
@@ -1042,11 +1042,6 @@ static void __init wait_8254_wraparound(void)
     do {
         prev_count = curr_count;
         curr_count = get_8254_timer_count();
-
-        /* workaround for broken Mercury/Neptune */
-        if (prev_count >= curr_count + 0x100)
-            curr_count = get_8254_timer_count();
-        
     } while (prev_count >= curr_count);
 }
 
@@ -1056,9 +1051,6 @@ static void __init wait_8254_wraparound(void)
  * this function twice on the boot CPU, once with a bogus timeout
  * value, second time for real. The other (noncalibrating) CPUs
  * call this function only once, with the real, calibrated value.
- *
- * We do reads before writes even if unnecessary, to get around the
- * P5 APIC double write bug.
  */
 
 #define APIC_DIVISOR 1