[PATCH] Drivers: hv: vmbus: Fix virt_to_hvpfn() for X86_PAE

Dexuan Cui decui at microsoft.com
Tue May 7 07:46:55 UTC 2019


In the case of X86_PAE, unsigned long is u32, but the physical address type
should be u64. Due to the bug here, the netvsc driver can not load
successfully, and sometimes the VM can panic due to memory corruption (the
hypervisor writes data to the wrong location).

Fixes: 6ba34171bcbd ("Drivers: hv: vmbus: Remove use of slow_virt_to_phys()")
Cc: stable at vger.kernel.org
Cc: Michael Kelley <mikelley at microsoft.com>
Reported-and-tested-by: Juliana Rodrigueiro <juliana.rodrigueiro at intra2net.com>
Signed-off-by: Dexuan Cui <decui at microsoft.com>
---
 drivers/hv/channel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c
index 23381c41d087..aaaee5f93193 100644
--- a/drivers/hv/channel.c
+++ b/drivers/hv/channel.c
@@ -38,7 +38,7 @@
 
 static unsigned long virt_to_hvpfn(void *addr)
 {
-	unsigned long paddr;
+	phys_addr_t paddr;
 
 	if (is_vmalloc_addr(addr))
 		paddr = page_to_phys(vmalloc_to_page(addr)) +
-- 
2.17.1



More information about the devel mailing list