[PATCH libdrm] amdgpu: fix "add AMDGPU_VA_RANGE_HIGH"

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

 



On 2018-02-28 04:25 PM, Christian König wrote:
> Am 28.02.2018 um 16:21 schrieb Michel Dänzer:
>> On 2018-02-28 03:41 PM, Christian König wrote:
>>> The 32bit address range end is exclusive, not inclusive.
>> The other way around. :) Inclusive means [a,b], i.e. an interval
>> including the endpoints, whereas exclusive means ]a,b[, i.e. an interval
>> not including the endpoints. The patch changes the 32-bit address range
>> endpoint from exclusive to inclusive.
> 
> Actually that's not what I meant. How about this:
> 
> "The range is stored as exclusive, not inclusive. Subtracts one to get
> the inclusive interval for the calculation."

Sounds good to me, but I'll defer to native English speakers for the
final verdict.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux