diff mbox series

hw/hyperv: Fix 32-bit build error for vmbus.

Message ID 20200907175102.28027-2-richard.henderson@linaro.org
State New
Headers show
Series hw/hyperv: Fix 32-bit build error for vmbus. | expand

Commit Message

Richard Henderson Sept. 7, 2020, 5:50 p.m. UTC
../qemu/hw/hyperv/vmbus.c: In function ‘gpadl_iter_io’:
../qemu/hw/hyperv/vmbus.c:383:13: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
  383 |         p = (void *)(((uintptr_t)iter->map & TARGET_PAGE_MASK) | off_in_page);
      |             ^
cc1: all warnings being treated as errors

Fixes: 0d71f7082d7 ("vmbus: vmbus implementation")
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

---
Cc: Jon Doron <arilou@gmail.com>
---
 hw/hyperv/vmbus.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.25.1

Comments

Richard Henderson Sept. 7, 2020, 5:52 p.m. UTC | #1
On 9/7/20 10:50 AM, Richard Henderson wrote:
> ../qemu/hw/hyperv/vmbus.c: In function ‘gpadl_iter_io’:

> ../qemu/hw/hyperv/vmbus.c:383:13: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]

>   383 |         p = (void *)(((uintptr_t)iter->map & TARGET_PAGE_MASK) | off_in_page);

>       |             ^

> cc1: all warnings being treated as errors

> 

> Fixes: 0d71f7082d7 ("vmbus: vmbus implementation")

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

> ---

> Cc: Jon Doron <arilou@gmail.com>

> ---


Dangit.  Forgot to remove old patches from the directory; this isn't part of
the pull request.


r~
diff mbox series

Patch

diff --git a/hw/hyperv/vmbus.c b/hw/hyperv/vmbus.c
index 75af6b83dd..3482e9c6cb 100644
--- a/hw/hyperv/vmbus.c
+++ b/hw/hyperv/vmbus.c
@@ -380,7 +380,8 @@  static ssize_t gpadl_iter_io(GpadlIter *iter, void *buf, uint32_t len)
             }
         }
 
-        p = (void *)(((uintptr_t)iter->map & TARGET_PAGE_MASK) | off_in_page);
+        p = (void *)(uintptr_t)
+            (((uintptr_t)iter->map & TARGET_PAGE_MASK) | off_in_page);
         if (iter->dir == DMA_DIRECTION_FROM_DEVICE) {
             memcpy(p, buf, cplen);
         } else {