Some of the DAWR SPR access is already predicated on dawr_enabled(),
apply this to the remainder of the accesses.

Signed-off-by: Nicholas Piggin <npig...@gmail.com>
---
 arch/powerpc/kvm/book3s_hv_p9_entry.c | 34 ++++++++++++++++-----------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/arch/powerpc/kvm/book3s_hv_p9_entry.c 
b/arch/powerpc/kvm/book3s_hv_p9_entry.c
index 323b692bbfe2..0f341011816c 100644
--- a/arch/powerpc/kvm/book3s_hv_p9_entry.c
+++ b/arch/powerpc/kvm/book3s_hv_p9_entry.c
@@ -666,13 +666,16 @@ int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 
time_limit, unsigned long lpc
 
        host_hfscr = mfspr(SPRN_HFSCR);
        host_ciabr = mfspr(SPRN_CIABR);
-       host_dawr0 = mfspr(SPRN_DAWR0);
-       host_dawrx0 = mfspr(SPRN_DAWRX0);
        host_psscr = mfspr(SPRN_PSSCR);
        host_pidr = mfspr(SPRN_PID);
-       if (cpu_has_feature(CPU_FTR_DAWR1)) {
-               host_dawr1 = mfspr(SPRN_DAWR1);
-               host_dawrx1 = mfspr(SPRN_DAWRX1);
+
+       if (dawr_enabled()) {
+               host_dawr0 = mfspr(SPRN_DAWR0);
+               host_dawrx0 = mfspr(SPRN_DAWRX0);
+               if (cpu_has_feature(CPU_FTR_DAWR1)) {
+                       host_dawr1 = mfspr(SPRN_DAWR1);
+                       host_dawrx1 = mfspr(SPRN_DAWRX1);
+               }
        }
 
        local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
@@ -1006,15 +1009,18 @@ int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 
time_limit, unsigned long lpc
        mtspr(SPRN_HFSCR, host_hfscr);
        if (vcpu->arch.ciabr != host_ciabr)
                mtspr(SPRN_CIABR, host_ciabr);
-       if (vcpu->arch.dawr0 != host_dawr0)
-               mtspr(SPRN_DAWR0, host_dawr0);
-       if (vcpu->arch.dawrx0 != host_dawrx0)
-               mtspr(SPRN_DAWRX0, host_dawrx0);
-       if (cpu_has_feature(CPU_FTR_DAWR1)) {
-               if (vcpu->arch.dawr1 != host_dawr1)
-                       mtspr(SPRN_DAWR1, host_dawr1);
-               if (vcpu->arch.dawrx1 != host_dawrx1)
-                       mtspr(SPRN_DAWRX1, host_dawrx1);
+
+       if (dawr_enabled()) {
+               if (vcpu->arch.dawr0 != host_dawr0)
+                       mtspr(SPRN_DAWR0, host_dawr0);
+               if (vcpu->arch.dawrx0 != host_dawrx0)
+                       mtspr(SPRN_DAWRX0, host_dawrx0);
+               if (cpu_has_feature(CPU_FTR_DAWR1)) {
+                       if (vcpu->arch.dawr1 != host_dawr1)
+                               mtspr(SPRN_DAWR1, host_dawr1);
+                       if (vcpu->arch.dawrx1 != host_dawrx1)
+                               mtspr(SPRN_DAWRX1, host_dawrx1);
+               }
        }
 
        if (vc->dpdes)
-- 
2.23.0

Reply via email to