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

[xen stable-4.14] x86/amd: Fix DE_CFG truncation in amd_check_zenbleed()



commit 602ee4c2954ed3f9f8552e08043146bee60ec012
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Fri Jul 28 18:42:12 2023 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Mon Jul 31 13:55:36 2023 +0100

    x86/amd: Fix DE_CFG truncation in amd_check_zenbleed()
    
    This line:
    
            val &= ~chickenbit;
    
    ends up truncating val to 32 bits, and turning off various errata 
workarounds
    in Zen2 systems.
    
    Fixes: f91c5ea97067 ("x86/amd: Mitigations for Zenbleed")
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    (cherry picked from commit c0dd53b8cbd1e47e9c89873a9265a7170bdc6b4c)
---
 xen/arch/x86/cpu/amd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/xen/arch/x86/cpu/amd.c b/xen/arch/x86/cpu/amd.c
index 9db79f409a..a1447f37ec 100644
--- a/xen/arch/x86/cpu/amd.c
+++ b/xen/arch/x86/cpu/amd.c
@@ -760,8 +760,8 @@ void amd_init_spectral_chicken(void)
 void amd_check_zenbleed(void)
 {
        const struct cpu_signature *sig = &this_cpu(cpu_sig);
-       unsigned int good_rev, chickenbit = (1 << 9);
-       uint64_t val, old_val;
+       unsigned int good_rev;
+       uint64_t val, old_val, chickenbit = (1 << 9);
 
        /*
         * If we're virtualised, we can't do family/model checks safely, and
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.14



 


Rackspace

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