[PATCH 1/2] drm/amdgpu: disable the VRAM manager on special placements v2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Christian König
> Sent: Tuesday, November 08, 2016 8:10 AM
> To: amd-gfx at lists.freedesktop.org; Qu, Jim; Zhou, David(ChunMing)
> Subject: 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.

I swear I replied to this.  Anyway, the series is:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> 
> 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


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux