Paolo Bonzini <pbonz...@redhat.com> writes:

> 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?

I don't think it matters semantically but using uintptr_t can't hurt.

Regards,

Anthony Liguori

>
> Paolo
>
>>> >          .iova = iova,
>>> >          .size = size,
>>> >      };
>> Thanks Anthony
>> 
>> Acked-by: Alex Williamson <alex.william...@redhat.com>


Reply via email to