[PATCH libdrm] tests/amdgpu: add unaligned VM test

[Christian König]

Am 10.09.2018 um 04:44 schrieb Zhang, Jerry (Junwei):
> On 09/10/2018 02:04 AM, Christian König wrote:
>> Make a VM mapping which is as unaligned as possible.
>
> Is it going to test unaligned address between BO allocation and BO 
> mapping
> and skip huge page mapping?

Yes and no.

Huge page handling works by mapping at least 2MB of continuous memory on 
a 2MB aligned address.

What I do here is I allocate 4GB of VRAM and try to map it to an address 
which is aligned to 1GB + 4KB.

In other words the VM subsystem will add a single PTE to align the entry 
to 8KB, then it add two PTEs to align it to 16KB, then four to get to 
32KB and so on until we have the maximum alignment of 2GB which 
Vega/Raven support in the L1.

Regards,
Christian.

>
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>>   tests/amdgpu/vm_tests.c | 45 
>> ++++++++++++++++++++++++++++++++++++++++++++-
>>   1 file changed, 44 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/amdgpu/vm_tests.c b/tests/amdgpu/vm_tests.c
>> index 7b6dc5d6..fada2987 100644
>> --- a/tests/amdgpu/vm_tests.c
>> +++ b/tests/amdgpu/vm_tests.c
>> @@ -31,8 +31,8 @@ static  amdgpu_device_handle device_handle;
>>   static  uint32_t  major_version;
>>   static  uint32_t  minor_version;
>>
>> -
>>   static void amdgpu_vmid_reserve_test(void);
>> +static void amdgpu_vm_unaligned_map(void);
>>
>>   CU_BOOL suite_vm_tests_enable(void)
>>   {
>> @@ -84,6 +84,7 @@ int suite_vm_tests_clean(void)
>>
>>   CU_TestInfo vm_tests[] = {
>>       { "resere vmid test",  amdgpu_vmid_reserve_test },
>> +    { "unaligned map",  amdgpu_vm_unaligned_map },
>>       CU_TEST_INFO_NULL,
>>   };
>>
>> @@ -167,3 +168,45 @@ static void amdgpu_vmid_reserve_test(void)
>>       r = amdgpu_cs_ctx_free(context_handle);
>>       CU_ASSERT_EQUAL(r, 0);
>>   }
>> +
>> +static void amdgpu_vm_unaligned_map(void)
>> +{
>> +    const uint64_t map_size = (4ULL << 30) - (2 << 12);
>> +    struct amdgpu_bo_alloc_request request = {};
>> +    amdgpu_bo_handle buf_handle;
>> +    amdgpu_va_handle handle;
>> +    uint64_t vmc_addr;
>> +    int r;
>> +
>> +    request.alloc_size = 4ULL << 30;
>> +    request.phys_alignment = 4096;
>> +    request.preferred_heap = AMDGPU_GEM_DOMAIN_VRAM;
>> +    request.flags = AMDGPU_GEM_CREATE_NO_CPU_ACCESS;
>> +
>> +    r = amdgpu_bo_alloc(device_handle, &request, &buf_handle);
>> +    /* Don't let the test fail if the device doesn't have enough 
>> VRAM */
>
> We may print some info to the console here.
>
> Regards,
> Jerry
>
>> +    if (r)
>> +        return;
>> +
>> +    r = amdgpu_va_range_alloc(device_handle, 
>> amdgpu_gpu_va_range_general,
>> +                  4ULL << 30, 1ULL << 30, 0, &vmc_addr,
>> +                  &handle, 0);
>> +    CU_ASSERT_EQUAL(r, 0);
>> +    if (r)
>> +        goto error_va_alloc;
>> +
>> +    vmc_addr += 1 << 12;
>> +
>> +    r = amdgpu_bo_va_op(buf_handle, 0, map_size, vmc_addr, 0,
>> +                AMDGPU_VA_OP_MAP);
>> +    CU_ASSERT_EQUAL(r, 0);
>> +    if (r)
>> +        goto error_va_alloc;
>> +
>> +    amdgpu_bo_va_op(buf_handle, 0, map_size, vmc_addr, 0,
>> +            AMDGPU_VA_OP_UNMAP);
>> +
>> +error_va_alloc:
>> +    amdgpu_bo_free(buf_handle);
>> +
>> +}
>>