[PATCH libdrm v2] amdgpu: Add memory over allocation test.

[christian.koenig@xxxxxxx (Christian König)]

Am 14.11.2017 um 13:53 schrieb Andrey Grodzovsky:
>
>
> On 11/14/2017 03:44 AM, Christian König wrote:
>> Am 13.11.2017 um 18:01 schrieb Andrey Grodzovsky:
>>> Allocates 1 TB of memory. Test is disabled by default
>>> since it's triggers OOM killer.
>>>
>>> v2:
>>> FIx the test to only alloc the BO and assert if return value
>>> not equal to -ENOMEM and remove test disable on start.
>>>
>>> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
>>
>> I was just about to scream that you need to fix the helper first, but 
>> then I saw that you now directly use amdgpu_bo_alloc().
>>
>> The commit message could be changed and for the unlikely case that we 
>> actually can allocate a BO of 1TB size we should free it.
>
> The patch already does this
>
> if (!r) {
> Â Â Â Â Â Â Â  r = amdgpu_bo_free(bo);
> Â Â Â Â Â Â Â  CU_ASSERT_EQUAL(r, 0);
> Â Â Â  }

Not enough sleep last night.

Going to review and push the patch now.

Thanks,
Christian.

>
>
> Thanks,
> Andrey
>
>>
>> But apart from that the patch looks good to me and is Acked-by: 
>> Christian König <christian.koenig at amd.com>.
>>
>> Regards,
>> Christian.
>>
>>> ---
>>> Â  tests/amdgpu/bo_tests.c | 24 ++++++++++++++++++++++++
>>> Â  1 file changed, 24 insertions(+)
>>>
>>> diff --git a/tests/amdgpu/bo_tests.c b/tests/amdgpu/bo_tests.c
>>> index 4545196..53e76c1 100644
>>> --- a/tests/amdgpu/bo_tests.c
>>> +++ b/tests/amdgpu/bo_tests.c
>>> @@ -47,6 +47,7 @@ static void amdgpu_bo_export_import(void);
>>> Â  static void amdgpu_bo_metadata(void);
>>> Â  static void amdgpu_bo_map_unmap(void);
>>> Â  static void amdgpu_memory_alloc(void);
>>> +static void amdgpu_mem_fail_alloc(void);
>>> Â Â Â  CU_TestInfo bo_tests[] = {
>>>       { "Export/Import",  amdgpu_bo_export_import },
>>> @@ -55,6 +56,7 @@ CU_TestInfo bo_tests[] = {
>>> Â  #endif
>>>       { "CPU map/unmap",  amdgpu_bo_map_unmap },
>>>       { "Memory alloc Test",  amdgpu_memory_alloc },
>>> +    { "Memory fail alloc Test",  amdgpu_mem_fail_alloc },
>>> Â Â Â Â Â  CU_TEST_INFO_NULL,
>>> Â  };
>>> Â  @@ -244,3 +246,25 @@ static void amdgpu_memory_alloc(void)
>>> Â Â Â Â Â  r = gpu_mem_free(bo, va_handle, bo_mc, 4096);
>>> Â Â Â Â Â  CU_ASSERT_EQUAL(r, 0);
>>> Â  }
>>> +
>>> +static void amdgpu_mem_fail_alloc(void)
>>> +{
>>> +Â Â Â  amdgpu_bo_handle bo;
>>> +Â Â Â  int r;
>>> +Â Â Â  struct amdgpu_bo_alloc_request req = {0};
>>> +Â Â Â  amdgpu_bo_handle buf_handle;
>>> +
>>> +Â Â Â  /* Test impossible mem allocation, 1TB */
>>> +Â Â Â  req.alloc_size = 0xE8D4A51000;
>>> +Â Â Â  req.phys_alignment = 4096;
>>> +Â Â Â  req.preferred_heap = AMDGPU_GEM_DOMAIN_VRAM;
>>> +Â Â Â  req.flags = AMDGPU_GEM_CREATE_NO_CPU_ACCESS;
>>> +
>>> +Â Â Â  r = amdgpu_bo_alloc(device_handle, &req, &buf_handle);
>>> +Â Â Â  CU_ASSERT_EQUAL(r, -ENOMEM);
>>> +
>>> +Â Â Â  if (!r) {
>>> +Â Â Â Â Â Â Â  r = amdgpu_bo_free(bo);
>>> +Â Â Â Â Â Â Â  CU_ASSERT_EQUAL(r, 0);
>>> +Â Â Â  }
>>> +}
>>
>>
>