patches 4-7 are
Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
though I agree with Christian that it would be nice to get a case
where 7 improves things before we submit it.
For patches 1-3 I need to dive into the slab allocator before I have
enough knowledge to review.
On Sat, Nov 24, 2018 at 12:41 AM Marek Olšák <mar...@gmail.com> wrote:
>
> From: Marek Olšák <marek.ol...@amd.com>
>
> ---
> src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
> b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
> index a9271c33ee9..36e2c4ec0dc 100644
> --- a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
> +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
> @@ -449,24 +449,29 @@ static struct amdgpu_winsys_bo *amdgpu_create_bo(struct
> amdgpu_winsys *ws,
> r = amdgpu_bo_alloc(ws->dev, &request, &buf_handle);
> if (r) {
> fprintf(stderr, "amdgpu: Failed to allocate a buffer:\n");
> fprintf(stderr, "amdgpu: size : %"PRIu64" bytes\n", size);
> fprintf(stderr, "amdgpu: alignment : %u bytes\n", alignment);
> fprintf(stderr, "amdgpu: domains : %u\n", initial_domain);
> goto error_bo_alloc;
> }
>
> va_gap_size = ws->check_vm ? MAX2(4 * alignment, 64 * 1024) : 0;
> +
> + unsigned vm_alignment = alignment;
> +
> + /* Increase the VM alignment for faster address translation. */
> if (size > ws->info.pte_fragment_size)
> - alignment = MAX2(alignment, ws->info.pte_fragment_size);
> + vm_alignment = MAX2(vm_alignment, ws->info.pte_fragment_size);
> +
> r = amdgpu_va_range_alloc(ws->dev, amdgpu_gpu_va_range_general,
> - size + va_gap_size, alignment, 0, &va,
> &va_handle,
> + size + va_gap_size, vm_alignment, 0, &va,
> &va_handle,
> (flags & RADEON_FLAG_32BIT ?
> AMDGPU_VA_RANGE_32_BIT : 0) |
> AMDGPU_VA_RANGE_HIGH);
> if (r)
> goto error_va_alloc;
>
> unsigned vm_flags = AMDGPU_VM_PAGE_READABLE |
> AMDGPU_VM_PAGE_EXECUTABLE;
>
> if (!(flags & RADEON_FLAG_READ_ONLY))
> vm_flags |= AMDGPU_VM_PAGE_WRITEABLE;
> --
> 2.17.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
