在 2018年10月26日,18:59,Christian König <ckoenig.leichtzumer...@gmail.com> 写道:
> 
> Make sure the kernel doesn't crash if we map something at the minimum/maximum 
> address.
> 
> Signed-off-by: Christian König <christian.koe...@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..bbdeef4d 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_mapping_test(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 },
> +     { "vm mapping test",  amdgpu_vm_mapping_test },
>       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_mapping_test(void)
> +{
> +     struct amdgpu_bo_alloc_request req = {0};
> +     struct drm_amdgpu_info_device dev_info;
> +     const uint64_t size = 4096;
> +     amdgpu_bo_handle buf;
> +     uint64_t addr;
> +     int r;
> +
> +     req.alloc_size = size;
> +     req.phys_alignment = 0;
> +     req.preferred_heap = AMDGPU_GEM_DOMAIN_GTT;
> +     req.flags = 0;
> +
> +     r = amdgpu_bo_alloc(device_handle, &req, &buf);
> +     CU_ASSERT_EQUAL(r, 0);
> +
> +     r = amdgpu_query_info(device_handle, AMDGPU_INFO_DEV_INFO,
> +                           sizeof(dev_info), &dev_info);
> +     CU_ASSERT_EQUAL(r, 0);
> +
> +     addr = dev_info.virtual_address_offset;
> +     r = amdgpu_bo_va_op(buf, 0, size, addr, 0, AMDGPU_VA_OP_MAP);
> +     CU_ASSERT_EQUAL(r, 0);

Please confirm:

We may need to unmap the VA before bo free, although this VA range is unlikely 
to be used by other test cases.

BTW, is it a chance in practice that a process may map different VA ranges to 
the same bo?

Regards,
Jerry

> +
> +     addr = dev_info.virtual_address_max - size;
> +     r = amdgpu_bo_va_op(buf, 0, size, addr, 0, AMDGPU_VA_OP_MAP);
> +     CU_ASSERT_EQUAL(r, 0);
> +
> +     if (dev_info.high_va_offset) {
> +             addr = dev_info.high_va_offset;
> +             r = amdgpu_bo_va_op(buf, 0, size, addr, 0, AMDGPU_VA_OP_MAP);
> +             CU_ASSERT_EQUAL(r, 0);
> +
> +             addr = dev_info.high_va_max - size;
> +             r = amdgpu_bo_va_op(buf, 0, size, addr, 0, AMDGPU_VA_OP_MAP);
> +             CU_ASSERT_EQUAL(r, 0);
> +     }
> +
> +     amdgpu_bo_free(buf);
> +}
> -- 
> 2.17.1
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to