qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 3/3] vfio: Fix 128 bit handling


From: Alexey Kardashevskiy
Subject: Re: [Qemu-devel] [PATCH v3 3/3] vfio: Fix 128 bit handling
Date: Thu, 29 Aug 2013 16:58:05 +1000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7

On 08/29/2013 04:29 PM, Paolo Bonzini wrote:
> Il 29/08/2013 04:26, Alexey Kardashevskiy ha scritto:
>>
>> Right. I was planning to add my IOMMU stuff right before calculating @end.
> 
> But then the non-IOMMU stuff can just use int128_get64, no?  So even if
> this patch simply uses int128_get64, it is still a suitable basis for
> adding IOMMU stuff.

Suitable but ugly. What if before calling int128_get64, I test
section->size if it is <2^64 and only then do RAM part of this function?


> 
> Paolo
> 
>>
>>>> This patch is not maintainable.  We're seemingly
>>>> calculating the same value twice with no comment as to why.  A hwaddr
>>>> type end should be calculated from the Int128 rather than paying
>>>> attention to rollover in one place but not another.  Thanks,
>> Yes, not maintainable... I guess I just have to convert @end to 128 bit as
>> well to have things consistent.
> 


-- 
Alexey



reply via email to

[Prev in Thread] Current Thread [Next in Thread]