It might happen that some hyp SynIC pages aren't allocated.

Check for that and only then call iounmap().

Signed-off-by: Roman Kisel <[email protected]>
Reviewed-by: Tianyu Lan <[email protected]>
Reviewed-by: Michael Kelley <[email protected]>
---
 drivers/hv/hv.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c
index 95631c08a71f..8e102bcc0be8 100644
--- a/drivers/hv/hv.c
+++ b/drivers/hv/hv.c
@@ -369,8 +369,10 @@ void hv_synic_disable_regs(unsigned int cpu)
         */
        simp.simp_enabled = 0;
        if (ms_hyperv.paravisor_present || hv_root_partition()) {
-               iounmap(hv_cpu->hyp_synic_message_page);
-               hv_cpu->hyp_synic_message_page = NULL;
+               if (hv_cpu->hyp_synic_message_page) {
+                       iounmap(hv_cpu->hyp_synic_message_page);
+                       hv_cpu->hyp_synic_message_page = NULL;
+               }
        } else {
                simp.base_simp_gpa = 0;
        }
@@ -381,8 +383,10 @@ void hv_synic_disable_regs(unsigned int cpu)
        siefp.siefp_enabled = 0;
 
        if (ms_hyperv.paravisor_present || hv_root_partition()) {
-               iounmap(hv_cpu->hyp_synic_event_page);
-               hv_cpu->hyp_synic_event_page = NULL;
+               if (hv_cpu->hyp_synic_event_page) {
+                       iounmap(hv_cpu->hyp_synic_event_page);
+                       hv_cpu->hyp_synic_event_page = NULL;
+               }
        } else {
                siefp.base_siefp_gpa = 0;
        }
-- 
2.43.0


Reply via email to