On Tue, 2012-10-02 at 08:11 +0200, Paolo Bonzini wrote:
> Il 01/10/2012 20:49, Alex Williamson ha scritto:
> >> > @@ -768,7 +768,7 @@ static int vfio_dma_map(VFIOContainer *container,
> >> > target_phys_addr_t iova,
> >> > struct vfio_iommu_type1_dma_map map = {
> >> > .argsz = sizeof(map),
> >> > .flags = VFIO_DMA_MAP_FLAG_READ,
> >> > - .vaddr = (__u64)vaddr,
> >> > + .vaddr = (__u64)(intptr_t)vaddr,
>
> Does this need to be uintptr_t?
With a simple test program, I don't seem to get sign extension either
way. Logically uintptr_t seems preferable to me. I'll add a patch to
my tree. Thanks,
Alex
