Reviewed-by: JimQu <Jim.Qu at amd.com>
Thanks
JimQu
________________________________________
å??件人: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> 代表 Christian König <deathsimple at vodafone.de>
å??é??æ?¶é?´: 2016å¹´11æ??8æ?¥ 21:09
�件人: amd-gfx at lists.freedesktop.org; Qu, Jim; Zhou, David(ChunMing)
主�: Re: [PATCH 1/2] drm/amdgpu: disable the VRAM manager on special placements v2
Am 04.11.2016 um 12:00 schrieb Christian König:
> From: Christian König <christian.koenig at amd.com>
>
> This disables the VRAM manager when a special placement is requested, otherwise
> we play ping/pong with the buffers on every command submission.
>
> v2: only check lpfn
>
> Signed-off-by: Christian König <christian.koenig at amd.com>
Could anybody give a Reviewed-by for this? I think that is a rather
important performance bug fix.
Regards,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c
> index 180eed7c..d710226 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.c
> @@ -108,7 +108,7 @@ static int amdgpu_vram_mgr_new(struct ttm_mem_type_manager *man,
> lpfn = man->size;
>
> if (bo->flags & AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS ||
> - amdgpu_vram_page_split == -1) {
> + place->lpfn || amdgpu_vram_page_split == -1) {
> pages_per_node = ~0ul;
> num_nodes = 1;
> } else {
_______________________________________________
amd-gfx mailing list
amd-gfx at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
