[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen staging-4.17] x86/mtrr: avoid system wide rendezvous when setting AP MTRRs



commit afcce3cfc2458efbd3ff9320153c534a82a108c5
Author:     Roger Pau Monné <roger.pau@xxxxxxxxxx>
AuthorDate: Tue May 21 12:02:13 2024 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Tue May 21 12:02:13 2024 +0200

    x86/mtrr: avoid system wide rendezvous when setting AP MTRRs
    
    There's no point in forcing a system wide update of the MTRRs on all 
processors
    when there are no changes to be propagated.  On AP startup it's only the AP
    that needs to write the system wide MTRR values in order to match the rest 
of
    the already online CPUs.
    
    We have occasionally seen the watchdog trigger during `xen-hptool 
cpu-online`
    in one Intel Cascade Lake box with 448 CPUs due to the re-setting of the 
MTRRs
    on all the CPUs in the system.
    
    While there adjust the comment to clarify why the system-wide resetting of 
the
    MTRR registers is not needed for the purposes of mtrr_ap_init().
    
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Release-acked-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    master commit: abd00b037da5ffa4e8c4508a5df0cd6eabb805a4
    master date: 2024-05-15 19:59:52 +0100
---
 xen/arch/x86/cpu/mtrr/main.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/xen/arch/x86/cpu/mtrr/main.c b/xen/arch/x86/cpu/mtrr/main.c
index 990f12286a..099e9f2cf7 100644
--- a/xen/arch/x86/cpu/mtrr/main.c
+++ b/xen/arch/x86/cpu/mtrr/main.c
@@ -573,14 +573,15 @@ void mtrr_ap_init(void)
        if (!mtrr_if || hold_mtrr_updates_on_aps)
                return;
        /*
-        * Ideally we should hold mtrr_mutex here to avoid mtrr entries changed,
-        * but this routine will be called in cpu boot time, holding the lock
-        * breaks it. This routine is called in two cases: 1.very earily time
-        * of software resume, when there absolutely isn't mtrr entry changes;
-        * 2.cpu hotadd time. We let mtrr_add/del_page hold cpuhotplug lock to
-        * prevent mtrr entry changes
+        * hold_mtrr_updates_on_aps takes care of preventing unnecessary MTRR
+        * updates when batch starting the CPUs (see
+        * mtrr_aps_sync_{begin,end}()).
+        *
+        * Otherwise just apply the current system wide MTRR values to this AP.
+        * Note this doesn't require synchronization with the other CPUs, as
+        * there are strictly no modifications of the current MTRR values.
         */
-       set_mtrr(~0U, 0, 0, 0);
+       mtrr_set_all();
 }
 
 /**
--
generated by git-patchbot for /home/xen/git/xen.git#staging-4.17



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.