|
Ok, I'll fix that, thanks!
From: Christian König <ckoenig.leichtzumerken@xxxxxxxxx>
Sent: Thursday, August 29, 2019 15:13 To: Yin, Tianci (Rico) <Tianci.Yin@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> Cc: Xu, Feifei <Feifei.Xu@xxxxxxx>; Ma, Le <Le.Ma@xxxxxxx>; Xiao, Jack <Jack.Xiao@xxxxxxx>; Zhang, Hawking <Hawking.Zhang@xxxxxxx> Subject: Re: [PATCH 2/2] drm/amdgpu: keep the stolen memory in visible vram region Am 29.08.19 um 05:05 schrieb Tianci Yin:
> From: "Tianci.Yin" <tianci.yin@xxxxxxx> > > stolen memory should be fixed in visible region. > > Change-Id: Icbbbd39fd113e93423aad8d2555f4073c08020e5 > Signed-off-by: Tianci.Yin <tianci.yin@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 6 ++++-- > drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 3 ++- > 2 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > index 801f912..dcd32d0 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c > @@ -1733,6 +1733,7 @@ int amdgpu_ttm_init(struct amdgpu_device *adev) > uint64_t gtt_size; > int r; > u64 vis_vram_limit; > + void *stolen_vga_buf; > > mutex_init(&adev->mman.gtt_window_lock); > > @@ -1787,7 +1788,7 @@ int amdgpu_ttm_init(struct amdgpu_device *adev) > r = amdgpu_bo_create_kernel(adev, adev->gmc.stolen_size, PAGE_SIZE, > AMDGPU_GEM_DOMAIN_VRAM, > &adev->stolen_vga_memory, > - NULL, NULL); > + NULL, &stolen_vga_buf); > if (r) > return r; > DRM_INFO("amdgpu: %uM of VRAM memory ready\n", > @@ -1851,8 +1852,9 @@ int amdgpu_ttm_init(struct amdgpu_device *adev) > */ > void amdgpu_ttm_late_init(struct amdgpu_device *adev) > { > + void *stolen_vga_buf; Coding style says we should add a new line between declaration and code. Apart from that the series is Reviewed-by: Christian König <christian.koenig@xxxxxxx>. > /* return the VGA stolen memory (if any) back to VRAM */ > - amdgpu_bo_free_kernel(&adev->stolen_vga_memory, NULL, NULL); > + amdgpu_bo_free_kernel(&adev->stolen_vga_memory, NULL, &stolen_vga_buf); > } > > /** > diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > index f77138b..ab43ae2 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c > @@ -1248,6 +1248,7 @@ static int gmc_v9_0_sw_init(void *handle) > static int gmc_v9_0_sw_fini(void *handle) > { > struct amdgpu_device *adev = (struct amdgpu_device *)handle; > + void *stolen_vga_buf; > > if (amdgpu_ras_is_supported(adev, AMDGPU_RAS_BLOCK__UMC) && > adev->gmc.umc_ras_if) { > @@ -1280,7 +1281,7 @@ static int gmc_v9_0_sw_fini(void *handle) > amdgpu_vm_manager_fini(adev); > > if (gmc_v9_0_keep_stolen_memory(adev)) > - amdgpu_bo_free_kernel(&adev->stolen_vga_memory, NULL, NULL); > + amdgpu_bo_free_kernel(&adev->stolen_vga_memory, NULL, &stolen_vga_buf); > > amdgpu_gart_table_vram_free(adev); > amdgpu_bo_fini(adev); |
_______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
