[Mesa-dev] [PATCH] radv: compute optimal VM alignment for imported buffers
Samuel Pitoiset
samuel.pitoiset at gmail.com
Thu Dec 20 16:39:37 UTC 2018
On 12/20/18 3:30 PM, Bas Nieuwenhuizen wrote:
> Happy to have this patch, but all descriptions and comments say this
> is an optimization ....
>
> Can we get some comment that this avoids hangs?
Pushed with a comment.
Thanks!
>
> Otherwise
>
> Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
>
> On Thu, Dec 20, 2018 at 3:23 PM Samuel Pitoiset
> <samuel.pitoiset at gmail.com> wrote:
>>
>> This fixes GPU hangs on GFX9 with
>> dEQP-VK.memory.external_memory_host.bind_image_memory_and_render.with_zero_offset.*
>>
>> Copied from RadeonSI.
>>
>> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
>> ---
>> src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c | 28 ++++++++++++++++++-
>> 1 file changed, 27 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
>> index ec126bfc7c..0b0e8850c1 100644
>> --- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
>> +++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c
>> @@ -410,6 +410,28 @@ radv_amdgpu_winsys_bo_unmap(struct radeon_winsys_bo *_bo)
>> amdgpu_bo_cpu_unmap(bo->bo);
>> }
>>
>> +static uint64_t
>> +radv_amdgpu_get_optimal_vm_alignment(struct radv_amdgpu_winsys *ws,
>> + uint64_t size, unsigned alignment)
>> +{
>> + uint64_t vm_alignment = alignment;
>> +
>> + /* Increase the VM alignment for faster address translation. */
>> + if (size >= ws->info.pte_fragment_size)
>> + vm_alignment = MAX2(vm_alignment, ws->info.pte_fragment_size);
>> +
>> + /* Gfx9: Increase the VM alignment to the most significant bit set
>> + * in the size for faster address translation.
>> + */
>> + if (ws->info.chip_class >= GFX9) {
>> + unsigned msb = util_last_bit64(size); /* 0 = no bit is set */
>> + uint64_t msb_alignment = msb ? 1ull << (msb - 1) : 0;
>> +
>> + vm_alignment = MAX2(vm_alignment, msb_alignment);
>> + }
>> + return vm_alignment;
>> +}
>> +
>> static struct radeon_winsys_bo *
>> radv_amdgpu_winsys_bo_from_ptr(struct radeon_winsys *_ws,
>> void *pointer,
>> @@ -420,16 +442,20 @@ radv_amdgpu_winsys_bo_from_ptr(struct radeon_winsys *_ws,
>> struct radv_amdgpu_winsys_bo *bo;
>> uint64_t va;
>> amdgpu_va_handle va_handle;
>> + uint64_t vm_alignment;
>>
>> bo = CALLOC_STRUCT(radv_amdgpu_winsys_bo);
>> if (!bo)
>> return NULL;
>>
>> + vm_alignment = radv_amdgpu_get_optimal_vm_alignment(ws, size,
>> + ws->info.gart_page_size);
>> +
>> if (amdgpu_create_bo_from_user_mem(ws->dev, pointer, size, &buf_handle))
>> goto error;
>>
>> if (amdgpu_va_range_alloc(ws->dev, amdgpu_gpu_va_range_general,
>> - size, 1 << 12, 0, &va, &va_handle,
>> + size, vm_alignment, 0, &va, &va_handle,
>> AMDGPU_VA_RANGE_HIGH))
>> goto error_va_alloc;
>>
>> --
>> 2.20.1
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list